Reader Mode: Optimize assets used in Reader UI

RESOLVED FIXED in Firefox 16



7 years ago
7 years ago


(Reporter: lucasr, Unassigned)


Firefox 16

Firefox Tracking Flags

(Not tracked)



(3 attachments)



7 years ago
* Only use one size of image for all DPIs?
* Run the images through a pngcrush tool to make them as small as possible?
* Make the background images smaller, so they aren't as big?

Comment 1

7 years ago
* Can we reduce the filesize of the backgrounds? They seem a bit larger and if we could save some space, I'd like to try.

Comment 2

7 years ago
(In reply to Lucas Rocha (:lucasr) from comment #0)
> * Only use one size of image for all DPIs?

About that, I experimented using xhdpi images for all DPIs. The images scale down accordingly depending on the display density. However, the down-scaling brings a slight blurriness to the edges of the icons.

It's not too bad but Ian might not be happy about it. I'll post sample screenshots.

Comment 3

7 years ago
Created attachment 635714 [details]
Screenshot (HDPI using proper images)

Comment 4

7 years ago
Created attachment 635715 [details]
Screenshot (HDPI using downscaled XHDPI images)

Comment 5

7 years ago
Ok, my take on this is that the loss of icon sharpness is not acceptable. I'll let Ian decide. I'll pngcrush all images to try cutting a few bytes from the assets anyway.

Comment 6

7 years ago
Created attachment 635737 [details] [diff] [review]
pngcrush all Reader assets
Attachment #635737 - Flags: review?(mark.finkle)
I agree, we've tried using one image and scaling in other cases, and it just never works right. As much as I don't enjoy making 9 versions of the same graphic (and the resulting extra file weight), it's worth having that pixel-level precision. 

Let's keep PNG-crushing if we can.
Attachment #635737 - Flags: review?(mark.finkle) → review+
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 16
You need to log in before you can comment on or make changes to this bug.