cc @DragaDoncila @psobolewskiPhD

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.43%. Comparing base (ded87bf) to head (fef1476).
Report is 2 commits behind head on main.

@@            Coverage Diff             @@
##             main    #6848      +/-   ##
==========================================
- Coverage   92.48%   92.43%   -0.06%     
==========================================
  Files         612      611       -1     
  Lines       55153    55151       -2     
==========================================
- Hits        51009    50977      -32     
- Misses       4144     4174      +30     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

After discussing with @DragaDoncila, we think all the menu bar actions should be defined in one file in _qt - currently the only non-qt actions are some help menu actions that open a URL. Although these actions technically do not require QT as they are defined for the purpose of living in a menu, it should be considered a 'GUI' action.

The only non-qt actions are the layer actions, which are not in the menubar.

This resolves any previous conflict for this PR as I think submenus can live with their actions, as only one file will exist for each menubar menu.

one of these files in two spots right?

2 spots only if they want to add a submenu as well. Only one spot if they are just adding an action.

Note if the action is a method, you'd add/amend the method in the file where the class is defined and update the action file.
Without the decorator we still have one 'extra' file to change. Note if you want to be able to add a submenu in the decorator as well, it makes things complicated as submenus may be re-used so adding it in a decorator is not ideal. And if we don't allow submenu addition it in a decorator you would still have to change 2 files to add an action AND a submenu.

But do like that we can see all the actions for a menu in one file, and its useful for checking how items are grouped inside the menu.

The failing test does not seem to be related? I've re-run it.

Some are timeout but e.g., test_last_point_is_visible_in_viewport may be legit? Passed on re-run

merge conflict

merge conflict resolved, cc @Czaki

Merged in main, which included removing the HELP menu actions from napari/_app_model/_app.py. I think that's correct, based on the changes in main. (btw in case folks haven't found it yet, I highly recommend git config --global merge.conflictStyle diff3.) If tests pass, any core dev please feel free to merge this!