<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>136059</bug_id>
          
          <creation_ts>2014-08-18 21:45:26 -0700</creation_ts>
          <short_desc>SVG favicon support</short_desc>
          <delta_ts>2022-12-07 08:53:20 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>SVG</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</resolution>
          
          
          <bug_file_loc>http://www.w3.org/TR/html5/links.html#attr-link-sizes-any</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Binyamin">7raivis</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>7raivis</cc>
    
    <cc>anthonyryan1</cc>
    
    <cc>ap</cc>
    
    <cc>beidson</cc>
    
    <cc>brandon</cc>
    
    <cc>dbates</cc>
    
    <cc>dholbert</cc>
    
    <cc>jberlin</cc>
    
    <cc>krit</cc>
    
    <cc>mathias</cc>
    
    <cc>sabouhallawa</cc>
    
    <cc>sierkb</cc>
    
    <cc>syoichi</cc>
    
    <cc>thorton</cc>
    
    <cc>tomac</cc>
    
    <cc>zimmermann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1029945</commentid>
    <comment_count>0</comment_count>
    <who name="Binyamin">7raivis</who>
    <bug_when>2014-08-18 21:45:26 -0700</bug_when>
    <thetext>Add SVG site icon support

Syntax &lt;link rel=icon href=gnome.svg sizes=&quot;any&quot; type=&quot;image/svg+xml&quot;&gt;
Spec http://www.w3.org/TR/html5/links.html#attr-link-sizes-any
Test-case http://xn--dahlstrm-t4a.net/svg/favicon/favicon.html

Implementation status on other browsers:
* Chrome https://code.google.com/p/chromium/issues/detail?id=294179
* Firefox https://bugzilla.mozilla.org/show_bug.cgi?id=366324
* IE https://connect.microsoft.com/IE/feedback/details/782416/svg-favicon-support</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1030209</commentid>
    <comment_count>1</comment_count>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2014-08-19 22:28:01 -0700</bug_when>
    <thetext>A few thoughts:
1 - In the current icon database code, there&apos;s very little unique to any file format (or image format) that prevents any resource data from being saved to the icon database.  Are you sure that an SVG file is *not* saved to the database today?

2 - In the short-to-midterm future, we&apos;re going to be deprecating the icon database altogether and moving away from the WebKit(WebCore) managed database mechanism on disk.  The replacement will be a pure thin client interface up to the embedding application that gives it the url/sizes/type of the proposed icon, asks for a load decision, and delivers the resource data.  In this future, it will truly be format-agnostic and it will be entirely up to the embedding application to manage the fact that it&apos;s SVG and to render it at the appropriate resolution.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1030210</commentid>
    <comment_count>2</comment_count>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2014-08-19 22:29:05 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; A few thoughts:
&gt; ...Are you sure that an SVG file is *not* saved to the database today?

To elaborate on this, in the past we actually successfully stored PDFs as a resolution-agnostic format, for example, and in the Mac port of WebKit at least they would render perfectly fine.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1178507</commentid>
    <comment_count>3</comment_count>
    <who name="Binyamin">7raivis</who>
    <bug_when>2016-03-28 13:23:18 -0700</bug_when>
    <thetext>WHATWG issue https://github.com/whatwg/html/issues/110</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183236</commentid>
    <comment_count>4</comment_count>
    <who name="Daniel Holbert">dholbert</who>
    <bug_when>2016-04-12 14:05:35 -0700</bug_when>
    <thetext>The WHATWG issue linked in comment 3 is actually not directly related to this bug.

(This bug is about SVG as a traditional favicon.  That WHATWG issue is about a fancier thing with an svg mask and a color instead of a favicon.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1431559</commentid>
    <comment_count>5</comment_count>
    <who name="Sierk Bornemann">sierkb</who>
    <bug_when>2018-06-09 07:16:26 -0700</bug_when>
    <thetext>The SVG format can be preferable to more easily support higher resolutions or larger icons and hence favicons.
There&apos;s no reason *not* to use SVG for icons/favicons. SVG scales well for all resolution types and is supported in all browsers back to IE9.
Therefore, please implement full and full-colored SVG favicon support via rel=&quot;icon&quot; rather than having it limited for &quot;pinned tabs&quot; only and rather than requiring a rel=&quot;mask-icon&quot; attribute.

For instance:
&lt;link rel=&quot;icon&quot; href=&quot;favicon.svg&quot;&gt;
or
&lt;link rel=&quot;icon&quot; href=&quot;favicon.svg&quot; type=&quot;image/svg+xml&quot;&gt;

See also:
https://caniuse.com/#feat=link-icon-svg</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1432086</commentid>
    <comment_count>6</comment_count>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2018-06-11 15:36:19 -0700</bug_when>
    <thetext>(In reply to Sierk Bornemann from comment #5)
&gt; The SVG format can be preferable to more easily support higher resolutions
&gt; or larger icons and hence favicons.
&gt; There&apos;s no reason *not* to use SVG for icons/favicons. SVG scales well for
&gt; all resolution types and is supported in all browsers back to IE9.
&gt; Therefore, please implement full and full-colored SVG favicon support via
&gt; rel=&quot;icon&quot; rather than having it limited for &quot;pinned tabs&quot; only and rather
&gt; than requiring a rel=&quot;mask-icon&quot; attribute.
&gt; 
&gt; For instance:
&gt; &lt;link rel=&quot;icon&quot; href=&quot;favicon.svg&quot;&gt;
&gt; or
&gt; &lt;link rel=&quot;icon&quot; href=&quot;favicon.svg&quot; type=&quot;image/svg+xml&quot;&gt;
&gt; 
&gt; See also:
&gt; https://caniuse.com/#feat=link-icon-svg

All of those points are clearly understood.

At this point, however, favicons are no longer a WebKit feature, but rather a Safari feature.

(I alluded to this upcoming change in 2014 https://bugs.webkit.org/show_bug.cgi?id=136059#c1)

So if you&apos;re looking for SVG favicon support in Safari, you&apos;ll need to file with Apple at developer.apple.com.

If you&apos;re looking for them in some other browser, you&apos;ll need to file in that browser&apos;s bug tracker.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1840048</commentid>
    <comment_count>7</comment_count>
    <who name="Sam Sneddon [:gsnedders]">gsnedders</who>
    <bug_when>2022-02-10 16:45:34 -0800</bug_when>
    <thetext>For the sake of Apple people:

&lt;rdar://5697363&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1840055</commentid>
    <comment_count>8</comment_count>
    <who name="Sierk Bornemann">sierkb</who>
    <bug_when>2022-02-10 16:56:34 -0800</bug_when>
    <thetext>(In reply to Sam Sneddon [:gsnedders] from comment #7)
&gt; For the sake of Apple people:
&gt; 
&gt; &lt;rdar://5697363&gt;

FYI, just for the record: also publicly mirrored on Open Radar as

https://openradar.appspot.com/41057732</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1917251</commentid>
    <comment_count>9</comment_count>
    <who name="Sam Sneddon [:gsnedders]">gsnedders</who>
    <bug_when>2022-12-07 08:53:20 -0800</bug_when>
    <thetext>*** Bug 248875 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>