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

Support for opening elements in new tabs#3223

Merged
bluemarvin merged 1 commit intomasterfrom
v11/conte_menu_open_link
May 29, 2020
Merged

Support for opening elements in new tabs#3223
bluemarvin merged 1 commit intomasterfrom
v11/conte_menu_open_link

Conversation

@keianhzo
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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