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

New action key: ban an image, video or audio track #1505

Merged
merged 1 commit into from
Dec 15, 2024

Conversation

GrosPoulet
Copy link
Contributor

A new action key (default value: "B") can be used to prevent an image, video or audio track from being zoomed (and played) next time it is hovered.

image

Ban is not lifted by just restarting browser. So in Advanced options, a new button is available in order to reset banned items so they can be zoomed again.

image

A new action key (default value: "B") can be used to prevent an image, video or audio track from being zoomed (and played) next time it is hovered.
Ban is not lifted by just restarting browser. So in Advanced options, a new button is available in order to reset banned items so they can be zoomed again.
Copy link

@GrosPoulet GrosPoulet merged commit fa078b3 into extesy:master Dec 15, 2024
3 checks passed
@extesy
Copy link
Owner

extesy commented Dec 15, 2024

@GrosPoulet This is a curious feature. What is the use case? Why would you want to ban a single image but not an entire website?

@GrosPoulet
Copy link
Contributor Author

@extesy This curious feature will be useful when i push improvments that will result in more images being zoomed, including background images. And sometimes, depending on website design, a background image "gets in the way", so user can ban it, et voilà!

@extesy
Copy link
Owner

extesy commented Dec 16, 2024

Be careful with that. Most users won't know about the "ban a single image" feature and when background images suddenly begin to unexpectedly pop up everywhere, it will be very annoying to many users.

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.

2 participants