8000 Key modifiers to open transaction in new tab from visualization by mononaut · Pull Request #3622 · mempool/mempool · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Key modifiers to open transaction in new tab from visualization #3622

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

Merged
merged 1 commit into from
Apr 3, 2023

Conversation

mononaut
Copy link
Collaborator
@mononaut mononaut commented Apr 1, 2023

Resolves #3563.

Clicking on a visualization while holding the shift, ctrl, or command keys, or clicking with the middle mouse button will now open the transaction in a new window/tab.

In Chrome on MacOS:

Combo Action
Left-Click Navigate in current tab
Shift + Left-Click Open in new window
Ctrl + Left-Click Doesn't work
Command + Left-Click Open in new tab
Shift + Command + Left-Click Open in new tab & switch to that tab
Middle-Click Open in new tab
Shift + Middle-Click Open in new tab & switch to that tab

This behavior is browser-dependent, so may vary on other platforms.

@cla-bot cla-bot bot added the cla-signed label Apr 1, 2023
@nymkappa nymkappa self-requested a review April 2, 2023 01:08
Copy link
Member
@nymkappa nymkappa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All the described shortcuts lead to opening a new tab and focus on it using Firefox MacOS. This does not matches the behavior described in https://support.mozilla.org/en-US/kb/mouse-shortcuts-perform-common-tasks?redirectslug=Mouse+shortcuts&redirectlocale=en-US but it does fixes issue #3563 anyway so that's fine. The main goal was to be able to open tx in a new tab which this PR achieves.

Tested ACK @ v2.6.0-dev [a91a8d2a]

@softsimon softsimon self-requested a review April 3, 2023 10:04
Copy link
Member
@softsimon softsimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested ACK @ [a91a8d2]

@softsimon softsimon merged commit 6b6dc9f into master Apr 3, 2023
@softsimon softsimon deleted the mononaut/shift-click branch April 3, 2023 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make it possible to Command-Click to open a TX in a new tab from the transaction visualizer
3 participants
0