You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The problem is that the AccordionPanel extends Panel and uses its children titles to separate the widgets and show/hide the respective child. See screenshot:
I'm trying to update the title, but it doesn't update the widget since the accordion panel is not listening for changes.
Is this a bug in the Panel widget? Should I open a PR to listen for title changes in the Panel widget, or should I overwrite the addWidget and insertWidget in the AccordionPanel to listen for changes?
Reproduce
Expected behavior
Context
The text was updated successfully, but these errors were encountered:
I think that logic should go in AccordionPanel and not Panel if possible, since not every panel needs to concern itself with the titles of its children.
Description
Not sure if this is a bug, I would say it is.
MenuBar and TabBar listen to title changes on every inserted widget, but Panel doesn't.
MenuBar:
lumino/packages/widgets/src/menubar.ts
Line 229 in 705ffa9
TabBar:
lumino/packages/widgets/src/tabbar.ts
Line 430 in 705ffa9
The problem is that the AccordionPanel extends Panel and uses its children titles to separate the widgets and show/hide the respective child. See screenshot:
I'm trying to update the title, but it doesn't update the widget since the accordion panel is not listening for changes.
Is this a bug in the Panel widget? Should I open a PR to listen for title changes in the Panel widget, or should I overwrite the
addWidget
andinsertWidget
in the AccordionPanel to listen for changes?Reproduce
Expected behavior
Context
The text was updated successfully, but these errors were encountered: