feat(UserMenu): add target property to usermenu component

[fitrahmunir]

• Resolves: Please add 'target = "_blank"' to the link 'List of invalid files' in Settings > Admin > Overview #41050

Summary

Add target property to UserMenu and UserMenuEntry components and add target "_blank" to Administration settings

TODO

• Add target property to UserMenu and UserMenuEntry components
• Add target "_blank" to admin_settings

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[nickvergessen]

Failing CI is related:

[nickvergessen]

I don't think the admin settings should open in a new tab all the time?
Neither the user menu links...

cc @nextcloud/designers

[marcoambrosini]

This is an old one.. I agree with the proposed change. I think that admin settings and personal settings, and all apps for that matter, should open in another tab. This solution:

• Reduces friction: once I'm done with the admin settings work, I don't have to remember where I was and what I was doing. I just close the tab and the first thing I see is my work exactly where I left it.
  Moreover, had open settings in the same tab, the state of the app that I navigated away from is not saved, and will likely never be saved:
  It has been suggested in the past that we could "remember" the state of each app, but this is hard to implement and maybe not even desirable in most cases. In any event, we'd be reinventing something that's already there at the browser level: multitasking. So in my opinion it's not a wise allocation of resources.

• Cuts on lading times: I don't have to load the app that I was using again.

So by insisting on loading different things in the same tab in pursuit of the big single page application feel, we're effectively discouraging users from using the multitasking feature that comes with a web browser.

@nextcloud/designers please go and take a look at other (multi-app) projects, the usual suspects. All of them open apps and global settings in new tabs, despite the fact that their loading times are in general faster than ours. (Testing their loading times vs. my local docker loading times).

[szaimen]

Hm... I think I don't like this idea. New tabs usually should only be opened for links to external services or if it is really necessary IIRC...

[marcoambrosini]

New tabs usually should only be opened for links to external services

That is just a convention that seems to be born out of not wanting users to navigate away from your site. Not a UX rule.

As per the necessity of it, I think here it's warranted because of the reasons I mentioned above. And our friends at G & M, that have all the UX research resources that they want, reached that conclusion too.

[szaimen]

And our friends at G & M, that have all the UX research resources that they want, reached that conclusion too.

Fine by me then 👍

[fitrahmunir]

@nickvergessen please review this PR

[come-nc]

👎 I do not think this is a good idea. It forces the user to open those pages in new tabs while it’s usually not what I want.
See https://sensiolabs.com/blog/2025/the-tab-trap-why-forcing-new-tabs-is-bad-ux for why this is a bad practice.

[ShGKme]

I agree with szaimen and come-nc.

If a user wants to open admin settings in a new tab, the user can user CTRL+Click, middle mouse button click, or the context menu while the default is opening in the current tab. We do not prevent the choice here.

If we change the default, keeping it open in the same tab is not so straightforward, and we are giving no choice to the user.

Especially if a user initially opened the instance to navigate to the admin settings.

[ShGKme]

Maybe it was changed in the meanwhile, but the linked issue seems about a different problem.

[susnux]

@fitrahmunir thank you very much for your contribution!
As outlined by the others opening every user menu entry in a new tab is not good UX.
Moreover the bug report was about the setup checks for the integrity results where such behavior probably makes sense because it redirects to a plain text file and not a webpage.