bhdouglass / uappexplorer

Moved to GitLab
https://gitlab.com/bhdouglass/uappexplorer
GNU General Public License v3.0
42 stars 5 forks source link

Allow opening app detail pages in the background #43

Closed nikwen closed 9 years ago

nikwen commented 9 years ago

Open uappexplorer.com/apps in the webbrowser app. It would be nice if the usual context menu with the "Open link in background tab" entry (it might be possible that this feature isn't available on the stable channel yet) showed up when long-pressing an app icon/name. Instead, I only see a "Copy" entry.

bhdouglass commented 9 years ago

Do other websites show the "Open link in background tab"? Its just a normal HTML link so if the feature is available I would assume it would work for those links.

nikwen commented 9 years ago

It works on most pages, yes. Take Github as an example. However, there seem to be some issues when the link consists of an image. I think that even if you don't have that option, you should be able to test this on your device as it should nevertheless display a "Open link in new tab" option and I'm not seeing that when long-pressing on an app. It does work though when clicking on the Webapp, Twitter and RSS links at the bottom of the page.

bhdouglass commented 9 years ago

I think I see the problem, it doesn't like all the extra html elements under the link tag. If you open the context menu to the side of the app (not on the image/text) where it is still part of the link it behaves properly. You may want to open a bug against the web browser. But I'll think about a way to work around it.

nikwen commented 9 years ago

Ok, I'll do that. :)

nikwen commented 9 years ago

Check out https://bugs.launchpad.net/ubuntu/+source/webbrowser-app/+bug/1460809. Since you wanted to work around it, I decided to use another page as an example. ;)

bhdouglass commented 9 years ago

I just added my "me too" over there, thanks!

nikwen commented 9 years ago

Thank you, too.

bhdouglass commented 9 years ago

The bug you filed was a duplicate of https://bugs.launchpad.net/oxide/+bug/1326070, it seems to be close to completion. So I'm going to wait a bit to find a work around.

nikwen commented 9 years ago

Saw that, thanks.

bhdouglass commented 9 years ago

@nikwen the launchpad bug says "Fix released" have you tried using the latest version of oxide yet?

nikwen commented 9 years ago

Just checked it on today's release of rc-proposed. It's working perfectly now!

screenshot20150918_222157552

bhdouglass commented 9 years ago

Awesome!