Bug 369207 - Mode & Highlighting menus remain empty on Mac OS X
Summary: Mode & Highlighting menus remain empty on Mac OS X
Status: RESOLVED MOVED
Alias: None
Product: kate
Classification: Applications
Component: part (show other bugs)
Version: 16.08
Platform: Compiled Sources macOS
: NOR major
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-09-22 20:54 UTC by RJVB
Modified: 2020-11-21 17:44 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
screenshot of KDevelop running in native mode on OS X, showing the crippled Tools/Mode menu (76.89 KB, image/png)
2016-09-22 20:55 UTC, RJVB
Details
screenshot of KDevelop running in X11 mode on OS X, showing a normal Tools/Mode menu (with an additional item) (92.72 KB, image/png)
2016-09-22 20:56 UTC, RJVB
Details
Patch to work around the issue on OS X (2.96 KB, patch)
2016-09-23 11:57 UTC, RJVB
Details

Note You need to log in before you can comment on or make changes to this bug.
Description RJVB 2016-09-22 20:54:15 UTC
I noticed that the Tools/Mode and Tools/Highlighting menus remain empty, except for the *disabled* submenus for the various known categories and the items:

"Normal" & "New Filetype" in Tools/Mode
and
"None" in Tools/Highlighting

Both Kate5 and KDevelop5 are affected.

Curiously this happens only when I use Qt's normal/default Cocoa QPA; when I start the applications in X11 mode (`-platform xcb`) the menus behave normally.

See the screenshots I'll be attaching.

I'm currently using Kate 16.08.0 and KTextEditor 5.24.0

Reproducible: Always

Steps to Reproduce:
1. Start Kate5 or KDevelop5
2. Open a document of a supported type for which highlighting definitions exist
3. Check the Tools/Mode and Tools/Highlighting menus

Actual Results:  
See the Cocoa screenshot and notice how the submenus are present but disabled.
The New Filetype menu item has no effect.

Expected Results:  
See the XCB screenshot: populated submenus

The highlighting mode can be selected normally via the status bar widget, so the issue cannot be explained by missing highlighting definitions.

The same software versions do NOT have the "New Filetype" Tools/Mode menu item on Linux.
Comment 1 RJVB 2016-09-22 20:55:33 UTC
Created attachment 101235 [details]
screenshot of KDevelop running in native mode on OS X, showing the crippled Tools/Mode menu
Comment 2 RJVB 2016-09-22 20:56:56 UTC
Created attachment 101236 [details]
screenshot of KDevelop running in X11 mode on OS X, showing a normal Tools/Mode menu (with an additional item)
Comment 3 RJVB 2016-09-22 22:53:32 UTC
So this appears to be related to using the Mac toplevel menubar: adding 

    QCoreApplication::setAttribute(Qt::AA_DontUseNativeMenuBar);

to main() gives functional and populated Mode an Highlighting menus too.
Comment 4 RJVB 2016-09-23 10:22:28 UTC
According to QMenu::isEnabled() and QAction::isEnabled(), the submenus and their actions are all enabled.

However, according to the QMenu documentation:

    QMenu on OS X with Qt Build Against Cocoa

    QMenu can be inserted only once in a menu/menubar. Subsequent insertions will have no effect or will result in a disabled menu item.

    See the Menus example for an example of how to use QMenuBar and QMenu in your application.

and according to the QWidgetAction docs:

    OS X: If you add a widget to a menu in the application's menu bar on OS X, the widget will be added and it will function but with some limitations:

        1. The widget is reparented away from the QMenu to the native menu view. If you show the menu in some other place (e.g. as a popup menu), the widget will not be there.

So it seems we're looking at a similar error/limitation as the one that previously led to error messages about menu items being added to or removed from more than 1 menu. Those messages no longer appear in Qt 5.6, but clearly the underlying reason is still there.
Comment 5 RJVB 2016-09-23 11:57:48 UTC
Created attachment 101240 [details]
Patch to work around the issue on OS X

Moving the Mode and Highlighting menus into the popup/context menu makes them behave as expected. It seems at least the Mode menu becomes better behaved when a copy is preserved in the toplevel menubar, but there's little point in keeping potentially broken copies around.

The patch also provides a number of more Mac-like shortcuts for typical menu actions.
Comment 6 Justin Zobel 2020-11-13 03:25:46 UTC
(In reply to RJVB from comment #5)
> Created attachment 101240 [details]
> Patch to work around the issue on OS X
> 
> Moving the Mode and Highlighting menus into the popup/context menu makes
> them behave as expected. It seems at least the Mode menu becomes better
> behaved when a copy is preserved in the toplevel menubar, but there's little
> point in keeping potentially broken copies around.
> 
> The patch also provides a number of more Mac-like shortcuts for typical menu
> actions.

If this is still an issue are you able to please submit your patch to https://invent.kde.org/websites/kate-editor-org/-/merge_requests

Can you please test and confirm if this issue is still occurring or if this bug report can be marked as resolved. I'm setting status to "needsinfo" pending your response, please change back to "reported" or "resolved" when you respond, thanks.