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

[bugfix] Restore shift-click branch behaviour #1901

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

jameshadfield
Copy link
Member

Clicking a branch while holding the shift-key should show an info modal.

Bisecting indicates that this regression occurred via

25294e899be20256844e26cdfb5d6b550616b6eb is the first bad commit
commit 25294e899be20256844e26cdfb5d6b550616b6eb
Author: Victor Lin <[email protected]>
Date:   Fri Oct 25 16:11:34 2024 -0700

    Properly check window.event.shiftKey

    Only KeyboardEvent has the shiftKey property.

 src/components/tree/reactD3Interface/callbacks.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(See #1900 for a somewhat related bug discovered while fixing this.)

Clicking a branch while holding the shift-key should show an info modal.

Bisecting indicates that this regression occurred via

```
25294e8 is the first bad commit
commit 25294e8
Author: Victor Lin <[email protected]>
Date:   Fri Oct 25 16:11:34 2024 -0700

    Properly check window.event.shiftKey

    Only KeyboardEvent has the shiftKey property.

 src/components/tree/reactD3Interface/callbacks.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
```
@nextstrain-bot nextstrain-bot temporarily deployed to auspice-james-branch-sh-phuvv3 November 14, 2024 04:59 Inactive
@jameshadfield
Copy link
Member Author

Merging sans review

@jameshadfield jameshadfield merged commit 628a6b5 into master Nov 14, 2024
25 checks passed
@jameshadfield jameshadfield deleted the james/branch-shift-click-bugfix branch November 14, 2024 05:03
@victorlin
Copy link
Member

Only KeyboardEvent has the shiftKey property.

Man I was wrong. Should have tested this in #1864. Thanks for catching!

@victorlin victorlin mentioned this pull request Dec 4, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants