WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
120784
REGRESSION (
r122806
): A different favicon is chosen from multiple ones
https://bugs.webkit.org/show_bug.cgi?id=120784
Summary
REGRESSION (r122806): A different favicon is chosen from multiple ones
Maxim
Reported
2013-09-05 11:58:25 PDT
The 32x32 pixels favicons are not displayed properly anymore. The icon is squished and pushed to the right and it looks like there is something added to it while page loads. The issue occurs on latest WebKit
r154944
and on Safari 6.1 seed 7. However older Safari 6.0 does not have this issue. The "sizes" attribute does not seem to make any effect on it either.
Attachments
Screenshots of the issue in different browsers
(263.92 KB, image/png)
2013-09-09 09:15 PDT
,
Maxim
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1
2013-09-05 12:54:36 PDT
Thank you for the report. Do you have a test URL or a test case that you could attach?
Maxim
Comment 2
2013-09-09 09:15:21 PDT
Created
attachment 211049
[details]
Screenshots of the issue in different browsers Now when I had some free time for more thorough tests it looks like the issues is not related to 32x32 pixel icon, but rather to the fact how it is being loaded by: <link rel="icon" href="logo.png" type="image/png" /> <link rel="shortcut icon" href="favicon.ico?v=4" type="image/x-icon" /> Webkit uses only first tag and loads logo.png, although other browsers load favicon.ico . Perhaps my markup is wrong, but I cant seem to recall any problems. Feel free to check demo at
http://dev.visuallizard.com/demo/favicon/
and screenshot attached. I should note that IE11 has similar results as Webkit. Could someone with better understanding of this compare test case with the specs please? PS. Should I close the bug report, since it's title isn't relevant anymore? Thanks, and sorry for confusion, Max
Alexey Proskuryakov
Comment 3
2013-09-09 10:02:50 PDT
This changed with <
http://trac.webkit.org/changeset/122806
> (
bug 88665
). Reading that bug, this is clearly an unintended consequence of that change, but I don't know if the new behavior is correct or not. Could you please keep the demo at <
http://dev.visuallizard.com/demo/favicon/
> online while we are investigating?
Maxim
Comment 4
2013-09-09 10:09:03 PDT
Sure, no problem. let me know if you need more tests or changes from my end. Thanks Alexey
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug