Open
Bug 1624650
Opened 5 years ago
Updated 2 years ago
Hide the panels when disabling them instead of destroying them
Categories
(DevTools :: General, enhancement, P3)
DevTools
General
Tracking
(Not tracked)
NEW
People
(Reporter: ochameau, Unassigned)
References
Details
This comment highlights that destroying the panels when disabling them from the options panels introduces a specific codepath which is hard to maintain.
This is the only case where we destroy a panel without destroying the whole toolbox.
Doing this require unregistering all listeners, including RDP event listeners, which we don't have to cleanup when the toolbox closes.
So, it would actually help simplifying our codebase if we were only hiding the disabled panels instead of actively trying to destroy them. The next time the toolbox would start, the disabled panels wouldn't even start.
Updated•5 years ago
|
Priority: -- → P3
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•