feat: add extension point for application menu items

[JammingBen]

Description

Adds an extension point for adding items in the top left application switcher menu.

AppMenuItemExtension's can either link to an internal page (via path), an external page (via url) or have a button handler (via handler). Please refer to the docs for more information.

This deprecates the applicationMenu property of the appinfo object.

Also removes ancient menu logic from oC10 times, such as the user menu.

Related Issue

• works towards External Sites alternative for Web #7260

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests
• Documentation
• Maintenance (e.g. dependency updates or tooling)

[update-docs]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

[JammingBen]

This currently only supports remix icons. Do we need support for icon urls?

[kulmann]

When I do a production build there is a vue-router error when navigating into an embedded wikipedia.com. doesn't happen with vite 🤔 (tested together with a production build of owncloud/web-extensions#13 in my ocis apps folder)

[JammingBen]

When I do a production build there is a vue-router error when navigating into an embedded wikipedia.com. doesn't happen with vite 🤔 (tested together with a production build of owncloud/web-extensions#13 in my ocis apps folder)

Huh, there was an issue with the way I've added the router component. I don't get why this didn't pop up in dev mode... anyway, now it should work (although you need to edit your server's CSP rules to allow an external site being embedded via an iFrame).

[kulmann]

Hmmm... did I overlook something or is an isVisible callback missing? There are apps, e.g. the admin-settings app, which are only visible to certain users.

[JammingBen]

Hmmm... did I overlook something or is an isVisible callback missing? There are apps, e.g. the admin-settings app, which are only visible to certain users.

Do we need it though? Since it will be the app's duty to register menu items, it can just not register an item if the user doesn't have permissions to see it.

[kulmann]

Hmmm... did I overlook something or is an isVisible callback missing? There are apps, e.g. the admin-settings app, which are only visible to certain users.

Do we need it though? Since it will be the app's duty to register menu items, it can just not register an item if the user doesn't have permissions to see it.

Hm true...

[sonarcloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud