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)

enhancement

Tracking

(Not tracked)

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.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.