Richard Heck wrote:
Well, toolbars have not been designed for several dozens of entries.
calling this a design is very nice
Indeed. But maybe there is a way around that problem, short of reverting
to the math panel.
the attached patch checks only the first action on a panel instead of all
(this won't change the behavior for the panels we ship)
Index: src/frontends/qt4/IconPalette.cpp
===================================================================
--- src/frontends/qt4/IconPalette.cpp (revision 20613)
+++ src/frontends/qt4/IconPalette.cpp (working copy)
@@ -207,12 +207,18 @@
void IconPalette::updateParent()
{
bool enable = false;
- for (int i = 0; i < actions_.size(); ++i)
+ // FIXME: only checking the first is faster than checking all
+ if (actions_.size() > 0)
+ enable = actions_.at(0)->isEnabled();
+
+ // FIXME: so this is commented out for speed considerations
+ // true fix is to repair the updating mechanism of the toolbar
+/* for (int i = 0; i < actions_.size(); ++i)
if (actions_.at(i)->isEnabled()) {
enable = true;
break;
}
-
+*/
parentWidget()->setEnabled(enable);
}
