Skip to content

Full viewer button link#4563

Merged
jburel merged 4 commits intoome:developfrom
will-moore:full_viewer_button_link
Apr 18, 2016
Merged

Full viewer button link#4563
jburel merged 4 commits intoome:developfrom
will-moore:full_viewer_button_link

Conversation

@will-moore
Copy link
Copy Markdown
Member

See https://trello.com/c/vRHQSsTs/59-full-viewer-button-to-be-link and #4558

This PR makes the thumbnails and the Full Viewer buttons into html <a> links which means you can right-click and "Open link in new tab".
To test:

  • Right-click on thumbnail -> open link in new tab. Repeat on other thumbnails.
  • Right-click on "Full Viewer" button -> open link in new tab.
  • Test "Full Viewer" button in General tab and Preview tab.
  • Check that images have been opened in a series of new tabs.

Also check that other clicks on thumbnails and button are still working OK - E.g. double-click thumb to open image in new window, selecting images, shift-select range of thumbs etc.

@will-moore will-moore changed the title Use image links for thumbnails and Full Viewer button Full viewer button link Apr 7, 2016
@sbesson sbesson added the develop label Apr 7, 2016
@pwalczysko
Copy link
Copy Markdown
Member

The UI part works as expected.

@will-moore
Copy link
Copy Markdown
Member Author

@will-moore will-moore closed this Apr 10, 2016
@will-moore will-moore reopened this Apr 10, 2016
@will-moore
Copy link
Copy Markdown
Member Author

@jburel jburel merged commit 384709f into ome:develop Apr 18, 2016
@jburel jburel added this to the 5.2.3 milestone May 4, 2016
@will-moore will-moore mentioned this pull request Feb 17, 2017
@will-moore will-moore deleted the full_viewer_button_link branch February 18, 2019 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants