Closed Bug 1265642 Opened 9 years ago Closed 9 years ago

Image display issues after setting "Show images" as "Blocked"

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: bli, Unassigned)

References

Details

Attachments

(3 files)

Attached image CtV-1.png
1. When set "Show images" as "Blocked", on some webpages(e.g. https://m.baidu.com), the default image is too large. Pls refer to the attachment "CtV-1.png". 2. When set "Show images" as "Blocked", on some webpages(e.g. http://m.miaopai.com/show/channel/rIwmI50u5GcdmxuwitTpPQ__), the default images overlapped. Pls refer to the attachment "CtV-2.png".
Attached image CtV-2.png
This does not look good but may be working as designed. NI to the developer who added the feature to have a look at the problems.
Flags: needinfo?(jonalmeida942)
(In reply to Kevin Brosnan [:kbrosnan] from comment #2) > This does not look good but may be working as designed. NI to the developer > who added the feature to have a look at the problems. Kevin could be right. The feature only blocks the image from being downloaded. For example, with data: URIs, we can't block the image. But that's not the issue here, I think. Could be something about the size of the image, or how the image is used in an overlay. Worth investigating.
See Also: → 1276385
I had a look at it and I'm not sure we can do much here. The way the website is constructed is to have two images of the next video thumbnails hidden behind the larger thumbnail of the current video and another image for the play button in the center, hence the lopsided look when we replace the images with the placeholder that has a transparent background. Maybe we could consider an svg with a white background to make it move evident, although we would need to test this out with other websites as well to make sure we don't break the look of the placeholder there as well. I created bug 1276385 to track it, but I don't think there's anything particular I think we can do here.
Flags: needinfo?(jonalmeida942)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: