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

Add button to bookmark in jbrowse-web share dialog #3639

Merged
merged 3 commits into from May 18, 2023
Merged

Conversation

cmdcolin
Copy link
Collaborator

This adds a link to the "ShareDialog" that suggests to right-click to bookmark. This is useful because it can otherwise be hard to generate a bookmark from link in the text field

@github-actions github-actions bot added the needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) label Apr 12, 2023
@cmdcolin cmdcolin added enhancement New feature or request and removed needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) labels Apr 12, 2023
@codecov
Copy link

codecov bot commented Apr 12, 2023

Codecov Report

Merging #3639 (01d4f46) into main (001f2cc) will decrease coverage by 0.06%.
The diff coverage is 62.06%.

@@            Coverage Diff             @@
##             main    #3639      +/-   ##
==========================================
- Coverage   64.28%   64.23%   -0.06%     
==========================================
  Files         931      934       +3     
  Lines       29976    29981       +5     
  Branches     7256     7258       +2     
==========================================
- Hits        19271    19258      -13     
- Misses      10540    10558      +18     
  Partials      165      165              
Impacted Files Coverage Δ
...jbrowse-web/src/components/ShareSettingsDialog.tsx 58.33% <58.33%> (ø)
...roducts/jbrowse-web/src/components/ShareDialog.tsx 62.31% <62.31%> (ø)
...roducts/jbrowse-web/src/components/ShareButton.tsx 92.30% <66.66%> (+19.48%) ⬆️
...cts/jbrowse-web/src/components/ShareInfoDialog.tsx 66.66% <66.66%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@carolinebridge
Copy link
Contributor

While "right click to bookmark" is a thing on Firefox, I don't get the same prompt on Chrome...for a Chrome user this might be confusing?

@cmdcolin
Copy link
Collaborator Author

thanks for pointing this out, was hoping chrome would have a similar feature. looks like you can "drag the link to the bookmark bar" in chrome. could use the text "Link for bookmarking" as an alternative.

there is also a method from the igv.js webapp here where you click and then it says to press ctrl+d

image

@cmdcolin cmdcolin changed the title Add link with text "Right-click to bookmark" in share dialog Add link with text "Link for bookmarking" in share dialog Apr 13, 2023
@cmdcolin
Copy link
Collaborator Author

changed the link text to "Link for bookmarking"

@cmdcolin cmdcolin force-pushed the right_click_bookmark branch 2 times, most recently from 9c21b6d to 29a779b Compare April 14, 2023 15:09
@carolinebridge
Copy link
Contributor

I still think it's a little confusing -- I feel like an individual might navigate to the link and then try to bookmark that, without understanding how the share link works.

I looked at it briefly and didn't find anything easy/possible, but a button that does the ctrl+D action on the share link would be perfect...

In lieu of that, maybe have an "info" tooltip that describes copying the link and manually creating a bookmark. It's really annoying (and weird) that Chrome doesn't have a clean equivalent to this.

Also want to point out that single clicking on the text box + right clicking the link we have in there already lets you "bookmark link" on firefox.

@cmdcolin
Copy link
Collaborator Author

I changed the action to try out the "Click to bookmark" and then it updates the text to say "Now hit Ctrl/Cmd+D". I feel like it's a bit awkward and crowded, and the two stage process is a bit hard to explain. I do think this feature is valuable, but, not sure how to best present it. It could be a separate action from the share dialog potentially

@cmdcolin cmdcolin marked this pull request as draft April 14, 2023 19:59
@cmdcolin
Copy link
Collaborator Author

New design

image

It pops up an alert box after clicking "bookmark" instructing the user to hit ctrl+d/cmd+d

@cmdcolin cmdcolin marked this pull request as ready for review May 18, 2023 19:32
@cmdcolin cmdcolin changed the title Add link with text "Link for bookmarking" in share dialog Add button to bookmark in jbrowse-web Share dialog May 18, 2023
@cmdcolin cmdcolin changed the title Add button to bookmark in jbrowse-web Share dialog Add button to bookmark in jbrowse-web share dialog May 18, 2023
@cmdcolin cmdcolin merged commit 5448370 into main May 18, 2023
10 of 12 checks passed
@cmdcolin cmdcolin deleted the right_click_bookmark branch May 18, 2023 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants