Vincent van Ravesteijn wrote:
Hi all,

This patch disables all submenus in the menu bar that have no enabled items.

The code for determining whether a submenu should be enabled is already present, but it is not used because the following line was missing:

           subMenu->setEnabled(m->status().enabled());

However, as there are a lot of submenus (which should be disabled) that have "No Document Open" items, I'm wondering whether this is a work-around or whether it is the design to have these submenus always enabled.

Anyway, this patch also removes all these useless "No Document Open" menuitems.

Any objections ?

None here. I certainly think it's better for the submenus to be disabled.

rh

Reply via email to