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

fix: do not intercept clicks with modifier keys pressed #200

Merged
merged 1 commit into from May 24, 2022
Merged

fix: do not intercept clicks with modifier keys pressed #200

merged 1 commit into from May 24, 2022

Conversation

kidonng
Copy link
Contributor

@kidonng kidonng commented May 23, 2022

Quick diff should be disabled when any modifier key (Ctrl, Shift, Alt, Meta) is pressed. This allows users to view pages as usual via opening them in new tabs/windows, when IPE is not working due to e.g. rate limit.

Copy link
Member

@bhsd-harry bhsd-harry left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@dragon-fish dragon-fish left a comment

Choose a reason for hiding this comment

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

To be considered: mobile device don't have such keys

@bhsd-harry
Copy link
Member

To be considered: mobile device don't have such keys

I do not think this is an issue. These changes have no effects on mobile device.

Copy link
Member

@dragon-fish dragon-fish left a comment

Choose a reason for hiding this comment

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

LGTM

@dragon-fish
Copy link
Member

Sorry I've been used to using the mmb. So I didn't pay attention to these keys.

You did a great job!

@dragon-fish dragon-fish merged commit 16c4a44 into inpageedit:master May 24, 2022
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.

None yet

3 participants