Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Support for opening elements in new tabs #3223

Merged
merged 1 commit into from
May 29, 2020
Merged

Conversation

keianhzo
Copy link
Contributor

Until now we only supported opening links in new windows/tabs. This PR adds support for also opening elements. ie. you want to open an image element (without a link) in a new window/tab

@keianhzo keianhzo self-assigned this Apr 23, 2020
@keianhzo keianhzo requested a review from MortimerGoro April 23, 2020 08:27
@keianhzo keianhzo added this to the #11 polish milestone Apr 28, 2020
@bluemarvin
Copy link
Contributor

@keianhzo This needs a rebase to land.

@keianhzo keianhzo force-pushed the v11/conte_menu_open_link branch from 29fbd01 to c9ac120 Compare May 29, 2020 07:26
@keianhzo keianhzo force-pushed the v11/conte_menu_open_link branch from c9ac120 to 786ce62 Compare May 29, 2020 07:26
@keianhzo keianhzo requested a review from MortimerGoro May 29, 2020 07:27
@keianhzo
Copy link
Contributor Author

@bluemarvin Rebase issues fixed. I've updated the PR to work more similar to what desktop does and show "View image/video/audio" instead of opening the src in new tab/windows.

@bluemarvin bluemarvin merged commit 5463c83 into master May 29, 2020
@bluemarvin bluemarvin deleted the v11/conte_menu_open_link branch May 29, 2020 19:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants