Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#15118 closed enhancement (wontfix)

[patch] Disable menu items when none of its items are enabled

Reported by: bafonins Owned by: team
Priority: normal Milestone:
Component: Core Version:
Keywords: Cc:


This patch disables menu's which have no enabled items in it.

Attachments (1)

menu-disable.patch (4.3 KB) - added by bafonins 6 years ago.
The patch itself

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by bafonins

Attachment: menu-disable.patch added

The patch itself

comment:1 Changed 6 years ago by bastiK

Hi, is it really customary behavior to disable an expandable menu item when all submenus are disabled? At least this is not how it works in LibreOffice and Gimp (if this is even a conscious decision).

It could make sense in some cases, e.g. presets, where menus are deeply nested and all entries are similar. But in general I prefer the way it is at the moment. The reason being that users can explore what tools and options are available and the list of disabled menu entries will give clues, what data to load/select in order to make use of the disabled feature set.

On a technical level, the patch is quite good and I would be glad to get more patches of this quality.

comment:2 Changed 6 years ago by Don-vip

+1 the patch is really good, but I'm not sure this is needed.
I'm afraid it could hurt the usability for people navigating the menu with keyboard arrows: I fear the navigation may be impossible when a disabled entry is found (to check on Windows, mac, Linux, etc.).

comment:3 Changed 6 years ago by stoecker

Resolution: wontfix
Status: newclosed

comment:4 Changed 6 years ago by bafonins

sounds reasonable, can be closed.

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.