Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open in new tab #79

Closed
Luticus opened this issue Feb 7, 2018 · 29 comments · Fixed by #287
Closed

Open in new tab #79

Luticus opened this issue Feb 7, 2018 · 29 comments · Fixed by #287

Comments

@Luticus
Copy link

Luticus commented Feb 7, 2018

It's really great that external sites has the ability to handle sites that can't be embedded, however, it would be really great if there were an option to launch them in a new tab instead of closing/overwriting my Nextcloud tab.

@nickvergessen
Copy link
Member

This is the current decision.
Use CTRL+click or right-click to open the link in a new tab.

@Luticus
Copy link
Author

Luticus commented Feb 7, 2018

Yep, that's what I've been doing up till now. Was hoping for a better more integrated and automatic method but I suppose you can't win them all. Thanks for checking into it.

@Luticus
Copy link
Author

Luticus commented Feb 7, 2018

App is great, btw, loving the per-site access groups/users thing. Made my life a lot better with that alone so I'll take it. :)

@vinzv
Copy link

vinzv commented Feb 20, 2018

Would it be possible to get some checkbox into admin's settings to enable e.g. target="_blank" on a per-link basis?

@nickvergessen
Copy link
Member

The problem is, that the API of the server does not have this option.

@vinzv
Copy link

vinzv commented Feb 20, 2018

I see. We can live with the current behaviour but I'd like to add my thoughts on that so you maybe can consider it for the future:

Default behaviour is embedding sites. If that's not possible, the target page is loaded solely but "overwrites" the nextcloud page. I think it would be more intuitive to have it loaded in another tab/new window instead of killing the user's current view.

@zippytiff
Copy link

Hi,

Any eta on the enhancement ? keen to get this out to users, but they wont cope with any work arounds

ZT

@jancborchardt
Copy link
Member

Not sure how this decision was taken, but I agree with what people mentioned here. If a site can’t be embedded, then clicking it should not overwrite the Nextcloud tab but instead open the site in a new tab as it’s external.

cc @nickvergessen @juliushaertl

@KnownNexus
Copy link

Hi,
I'd just like to clarify the position on this,
i've been developing a change to make it so:
left clicking, ctrl-clicking and right-click: open new tab - all open the file in a new tab

ctrl-click was previously bound to "single select"
and left click resulted in opening the file in the current window (Not ideal for my use case)

Is this something you guys would be interested in?
and if you have any issues, how would you like it changed to make it acceptable?

@juliushaertl
Copy link
Member

Not sure how this decision was taken, but I agree with what people mentioned here. If a site can’t be embedded, then clicking it should not overwrite the Nextcloud tab but instead open the site in a new tab as it’s external.

Sounds good to me, but only for non-embedded sites

left clicking, ctrl-clicking and right-click: open new tab - all open the file in a new tab

Right click should still just open the browser menu, no need to overwrite the default behavior here.

@KnownNexus
Copy link

KnownNexus commented Jun 19, 2019

Right click should still just open the browser menu, no need to overwrite the default behavior here.

Sorry just to clarify, right click opens the context menu, i haven't modified any functionality in that case, i was just confirming that it did still open a new tab

@jmacarthur
Copy link

The current behaviour seems to be a mix of two common behaviours:

For a multiple-selection list box, single click selects one item and deselects others, shift-click selects a range, and ctrl-click toggles one item on or off.
For a HTML link, single click opens in the same window, shift-click opens in new window, and ctrl-click opens in a new tab.

These are just what I consider to be normal behaviour, your browser and experience with UI toolkits may be different of course. But if you agree with the above models, shouldn't we pick one rather than mixing them?

@Luticus
Copy link
Author

Luticus commented Jun 19, 2019

When I originally suggested this, what I thought of was a check box in settings where you declare the external site that says something like "open this site in new tab" and then when you check that box and a user clicks the site icon rather than opening in the same tab and clobbering the users Nextcloud session it opens in a new tab per the browsers settings. I'm not particularly interested in the crtl + click behavior as I'm fairly certain like 99% of users probably don't realize that's a thing. Right click menu options could be interesting. Right click on the icon and you get a menu that says open in new tab or something like that would be interesting, so that other apps could be opened easily in new tabs, like talk, for instance. I'm most interested in external sites though. I embed emby and others in nextcloud and they don't behave right when they are loaded as an app. Full screening a movie doesn't work in emby for instance. So I set them up to clobber nextcloud. Would be really nice if instead it just opened in a new tab.

@jancborchardt
Copy link
Member

@KnownNexus could you open a pull request with just 1 change as described above? That would likely fix the issue, without any need for modification or learning patterns on people’s part:

If a site can’t be embedded, then clicking its icon should not overwrite the Nextcloud tab but instead open the site in a new tab.

Does that work? Thanks a lot! :)


When I originally suggested this, what I thought of was a check box in settings where you declare the external site that says something like "open this site in new tab" and then when you check that box and a user clicks the site icon rather than opening in the same tab and clobbering the users Nextcloud session it opens in a new tab per the browsers settings. I'm not particularly interested in the crtl + click behavior as I'm fairly certain like 99% of users probably don't realize that's a thing.

@Luticus The issue is that the majority of people would not realize that a setting for this exists and then be none the wiser either. :) This needs to simply happen automatically, as said above.

@KnownNexus
Copy link

KnownNexus commented Jun 20, 2019

@jancborchardt

could you open a pull request with just 1 change as described above? That would likely fix the issue, without any need for modification or learning patterns on people’s part:

If a site can’t be embedded, then clicking its icon should not overwrite the Nextcloud tab but instead open the site in a new tab.

What about things like libreoffice? I think it would be good to have that kind of thing open in a new tab, rather than embed in the current one, as that means you aren't losing your place in the file system if you close it.
(Many peoples reaction is to close tabs rather than clicking to return to the previous page)
Something similar to the way google drive opens its docs

@nickvergessen
Copy link
Member

Not sure how this decision was taken…

By technology. The app can not influence the core's navigation handling to tell it to do this.
Either this needs to be extended on the server side. Or we even hard code it there and when the target is not starting with the nextcloud url, we always open in a new tab.

@jancborchardt
Copy link
Member

What about things like libreoffice? I think it would be good to have that kind of thing open in a new tab, rather than embed in the current one, as that means you aren't losing your place in the file system if you close it.

This issue here is about the "External" app, for embedding sites. It’s not about Collabora Online. :) (And Collabora Online doesn’t make you lose your place in the file system, as when you close the Collabora view you are back in Files where you were before.)

@jmacarthur
Copy link

When editing two or more documents, say to copy text from one into another, it's useful to have the file list remain. Otherwise, you would have to open two instances of the same file window before clicking different files in each. If we have to do that via right-click, that will be fine. Ctrl-click to open in a new tab would be nice, but isn't essential.

@Luticus
Copy link
Author

Luticus commented Jun 21, 2019

@jancborchardt

The issue is that the majority of people would not realize that a setting for this exists and then be none the wiser either. :) This needs to simply happen automatically, as said above.

Not sure I follow you here. Maybe we are talking about 2 different things. I'm talking about taking the "default" behavior out of the hands of the users entirely. So as an admin when you are declaring a url as an external site you either add a check box that says "open this site in new tab", or even just modify the behavior of the "this site does not allow embedding" checkbox, which honestly works fine for me. As an admin it's easier to just set the defaults that make the most sense for users rather than explaining, "no on this icon you need to right click and and this one you need to ctrl+click". Just, click the icon and if it works better in a new tab, it opens in a new tab.

ncexternalsites_settings

Maybe even display a popup message, similar to when you send a message over 1000 characters in the talk app, for the user that says that the item opened in a new tab. This way if their browser is configured to not shift immediately to a new tab the user can tell that the icon did something. Might not seem important but as a user who keeps lots of tabs open and doesn't let the browser transition to them immediately on open, I can tell you that it's sometimes easy to get lost and not realize a link opened in a new tab.

@TP75

This comment has been minimized.

@jancborchardt
Copy link
Member

Yes, long discussion is confusing and adding more replies won’t make it better. If there is any progress, this issue will be updated. Contributions are very welcome and what was said above still stands:

#79 (comment)

@nooblag

This comment has been minimized.

@nickvergessen

This comment has been minimized.

@poltrian

This comment has been minimized.

@nickvergessen

This comment has been minimized.

@nooblag

This comment has been minimized.

@solracsf

This comment has been minimized.

@nooblag

This comment has been minimized.

@jancborchardt
Copy link
Member

jancborchardt commented Mar 26, 2020

Can only repeat what I said above:

Yes, long discussion is confusing and adding more replies won’t make it better. If there is any progress, this issue will be updated. Contributions are very welcome and what was said above still stands:
#79 (comment)

And will hereby lock the issue because the same things are said again and again and the comments just obscure the solution which has been proposed. Further discussion can happen in a pull request which is opened by someone who would like to implement this as discussed. :)

@nextcloud nextcloud locked as off-topic and limited conversation to collaborators Mar 26, 2020
@provokateurin provokateurin unpinned this issue Aug 21, 2023
@provokateurin provokateurin pinned this issue Aug 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.