Closed
Bug 914952
Opened 11 years ago
Closed 4 years ago
Correctly decode <link> tags containing multiple sizes and handle multiple <link> tags
Categories
(Firefox for Android Graveyard :: Theme and Visual Design, defect)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: ckitching, Unassigned)
References
Details
A <link> tag, such as one used to specify a Favicon, may specify multiple sizes. For example, if the payload is a container format such as ICO or ICNS, then the tag should (But usually doesn't) list all the different sizes of icon available.
Alternatively, the sizes can be used to specify sizes to which it is valid to scale an otherwise perfectly scalable icon. For example, given an SVG payload (See also: Bug 914025), the website author my stipulate via the sizes attribute that it shall only be scaled to certain sizes (And should not be arbitrarily upscaled). They may then elect to specify an alternative favicon for larger sizes.
Currently, the policy of the Favicon system is to grab the largest, shiniest one we can, and fall back repeatedly on lesser-attractive icons until we either succeed in decoding one or run out of things to try. Continuing with this, when reading <link> tags with multiple sizes we need to parse the complete list to determine which size is the largest available.
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → chriskitching
Reporter | ||
Updated•11 years ago
|
Assignee: chriskitching → nobody
Reporter | ||
Comment 2•10 years ago
|
||
Merging the slight extra twiddle that Bug 1070252 pointed out, since the ensuing patch should straightforwardly resolve both problems (they all just need to feed into the same optimal favicon selection scheme).
Summary: Correctly decode <link> tags containing multiple sizes. → Correctly decode <link> tags containing multiple sizes and handle multiple <link> tags
Comment 3•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•