Critiques pour View Image Context Menu Item
View Image Context Menu Item par Anton G
56 notes
- Noté 5 sur 5par Hollytryx, il y a 4 mois
- Noté 5 sur 5par Mertsch, il y a un anSo useful, does not work in EVERY case, but in so many, that it's really an improvement.
- Noté 5 sur 5par TyDraniu, il y a un an
- Noté 5 sur 5par Milan, il y a 2 ansWorks mostly great. Question: can something be done about reddit images? Simply opening image URL in new tab doesn't work, it opens reddit page with image which means image cannot be easily zoomed in.
Edit: update is working as described in the reply. Thank youRéponse du développeur
mis en ligne : il y a 2 ansPlaying around with it, reddit seems to utilize the Accept header, which with the add-on did not tamper. By default it was accepting "text/html" before any other mime types. Altering the Accept header to instead specify media types first seems to work with reddit at this point. I've updated the add-on (version 3.3) with an option, enabled by default, to override the Accept header for this compatiblity. - Noté 5 sur 5par Taekilla, il y a 2 ans
- Noté 5 sur 5par r2rien, il y a 2 ans
- Noté 5 sur 5par ^5 david slayer, il y a 2 ans
- Noté 5 sur 5par Linkitch, il y a 2 ans
- Noté 5 sur 5par ALH, il y a 2 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 15094302 de Firefox, il y a 2 ansVery helpful. I have no clue why the original menu item was removed, but this add-on restores it without causing any issues. Thank you!
- Noté 5 sur 5par nyuu, il y a 2 ansThe CSS provided in the description uses now-deprecated selectors so your users' view image button is going to be moved back to the middle of the items instead of the top. If people are reporting vague "errors"; this is why.
Below is a corrected userChrome setup for the affected portions:
menuitem[id*="view-image-context-menu-item"] {order: -1 !important;}
menuitem[id*="view-video-context-menu-item"] {order: -1 !important;}
menuitem[id*="view-audio-context-menu-item"] {order: -1 !important;}
#contentAreaContextMenu {display: flex; flex-direction: column-reverse;Réponse du développeur
mis en ligne : il y a 2 ansThanks for the heads up; looks like I was behind the times on my Firefox updates. I've updated to 113 and saw it broke my menu as well.
I've updated the the userChrome in description to start using order instead of the deprecated moz box ordering. Though in my testing i did not see any effects of flex-direction: column-reverse so i have left it out. - Noté 5 sur 5par Aadolf Kärki, il y a 2 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 16596123 de Firefox, il y a 3 answorked great until a recent MANDATORY update from firefox. I fucking hate firefox but this extension is good
- Noté 5 sur 5par ZedEdge, il y a 3 ans
- Noté 5 sur 5par lynx, il y a 3 ans
- Noté 5 sur 5par BluePenguin, il y a 3 ans
- Noté 5 sur 5par Nero14x, il y a 4 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 13630698 de Firefox, il y a 4 ansThanks for doing this, I can't believe they changed something that was working perfectly fine... the only thing I've trouble with is that I can't figure out how to change it's position from the top to lower ones since I tend to right-click the thumbnail and open vids in another tab.
EDIT: That thread was what I was looking for, thanks again :)Réponse du développeur
mis en ligne : il y a 4 ansIf you mean the context menu ordering, by default addon menu items are listed at the bottom. By messing with UserChrome.css you can make them appear in a custom order.
In the addon description i have some basic CSS to always list it first in the context menu, but if you want to have better control over the ordering, check this thread: https://github.com/agoupinets/view-image-context-menu-item/issues/14
And also of note that in the add-on preferences there are options to change the left/middle/shift-click behavior of the context menu item if desired. - Noté 5 sur 5par Utilisateur ou utilisatrice 13689361 de Firefox, il y a 4 ans
- Noté 5 sur 5par asg aesgasgea sg, il y a 4 anskinda sad that one needs an addon for some basic function. shame on you firefox :P
- Noté 5 sur 5par Kathanaddaeus, il y a 4 ansReplaces the old functionality well enough. I did change the userChrome entries to hide the icon though to "visibility: hidden" instead of "display: none" as that moved the text all the way to the left edge of the context menu. In case anyone has a similar problem.
Réponse du développeur
mis en ligne : il y a 4 ansthats a good point - i've been writing the CSS against the Proton interface, but looks like its not compatible with the old UI; in the old UI it needs to be "visibility: hidden" or offsets the text too far left, and in the new UI it needs to be "display: none" otherwise the text remains offset too far right.
i should make a note of that on the description - Noté 5 sur 5par Talon-senpai, il y a 4 ans
- Noté 5 sur 5par fimmwolf, il y a 4 ans
- Noté 5 sur 5par oofergang, il y a 4 ans
- Noté 5 sur 5par TGHI, il y a 4 ans