<?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>14019</bug_id>
          
          <creation_ts>2007-06-06 12:42:43 -0700</creation_ts>
          <short_desc>SVG focus ring support has minor display bugs</short_desc>
          <delta_ts>2010-07-08 23:53:50 -0700</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>523.x (Safari 3)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>10849</dependson>
    
    <dependson>14015</dependson>
    
    <dependson>25532</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>emacemac7</cc>
    
    <cc>zimmermann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>8371</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2007-06-06 12:42:43 -0700</bug_when>
    <thetext>SVG focus ring support has minor display bugs

Now that SVG has focus ring support (thanks to bug 10849), we need to go back and clean up some of the display bugs such support exposes.

Some examples of bugs:
- focus rings for paths do not conform to path shape
- focus rings for paths are 1px larger than necessary (due to antialiasing fuzz in the absoluteBounds method)
- focus rings do not respect clip when sizing themselves (see &lt;image&gt; example)
- text focus rings do not respect transform= attributes correctly
- image focus rings do not respect transform= attributes correctly</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>8372</commentid>
    <comment_count>1</comment_count>
      <attachid>14887</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2007-06-06 12:44:57 -0700</bug_when>
    <thetext>Created attachment 14887
focus ring tests showing some remaining bugs

This test case has already been landed as svg/custom/focus-ring.svg</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>58351</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2007-10-12 02:23:09 -0700</bug_when>
    <thetext>Focus ring painting will actually improve once we fix bug 14015, because at least RenderSVGImage depends on a proper computeAbsoluteRepaintRect implementation (which we don&apos;t currently have).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119739</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-05-03 15:37:52 -0700</bug_when>
    <thetext>I&apos;ve fixed text focus rings locally.  Will be uploading that patch soon.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248777</commentid>
    <comment_count>4</comment_count>
    <who name="Nikolas Zimmermann">zimmermann</who>
    <bug_when>2010-07-08 23:53:50 -0700</bug_when>
    <thetext>Fixed in trunk.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>14887</attachid>
            <date>2007-06-06 12:44:57 -0700</date>
            <delta_ts>2007-06-06 12:44:57 -0700</delta_ts>
            <desc>focus ring tests showing some remaining bugs</desc>
            <filename>focus-ring.svg</filename>
            <type>image/svg+xml</type>
            <size>1497</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHhtbG5zOnhsaW5rPSJodHRw
Oi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIj4KICA8c3R5bGU+CiAgICAucmluZyB7CiAgICAgIG91
dGxpbmU6IC13ZWJraXQtZm9jdXMtcmluZy1jb2xvciA1cHggYXV0bzsKICAgIH0KICA8L3N0eWxl
PgogICAgCiAgPGNpcmNsZSBjeD0iNjUiIGN5PSI2MCIgcj0iNTAiIGZpbGw9Im9yYW5nZSIgY2xh
c3M9InJpbmciIC8+CiAgPHBhdGggZD0iTSAxNDAgMTAgbCA1MCAxMDAgbCA1MCAtNTAgbCAtMjAg
MCBsIC0yMCAtMzAgWiIgZmlsbD0ib3JhbmdlIiBjbGFzcz0icmluZyIgLz4KICA8ZyBjbGFzcz0i
cmluZyI+CiAgICA8cmVjdCB4PSIyNzAiIHk9IjEwIiB3aWR0aD0iMTAwIiBoZWlnaHQ9IjEwMCIg
ZmlsbD0ib3JhbmdlIiAvPgogIDwvZz4KICAKICA8ZyB0cmFuc2Zvcm09InRyYW5zbGF0ZSg2MCwg
MTQwKSByb3RhdGUoMzApIj4KICAgIDxyZWN0IHdpZHRoPSIxMDAiIGhlaWdodD0iMTAwIiBmaWxs
PSJvcmFuZ2UiIGNsYXNzPSJyaW5nIiAvPgogIDwvZz4KICA8cmVjdCB3aWR0aD0iMTAwIiBoZWln
aHQ9IjEwMCIgZmlsbD0ib3JhbmdlIiB0cmFuc2Zvcm09InRyYW5zbGF0ZSgyMDAsIDE0MCkgcm90
YXRlKDMwKSIgY2xhc3M9InJpbmciIC8+CiAgPGcgY2xhc3M9InJpbmciPgogICAgPHJlY3Qgd2lk
dGg9IjEwMCIgaGVpZ2h0PSIxMDAiIGZpbGw9Im9yYW5nZSIgdHJhbnNmb3JtPSJ0cmFuc2xhdGUo
MzYwLCAxNDApIHJvdGF0ZSgzMCkiIC8+CiAgPC9nPgogIAogIDx0ZXh0IHg9IjEwIiB5PSIzMjAi
IGNsYXNzPSJyaW5nIj5mb2N1c2VkIHRleHQ8L3RleHQ+CiAgPHRleHQgdHJhbnNmb3JtPSJ0cmFu
c2xhdGUoMTAwLCAzMjApIHJvdGF0ZSgzMCkiIGNsYXNzPSJyaW5nIj5mb2N1c2VkIHRleHQ8L3Rl
eHQ+CiAgPGcgY2xhc3M9InJpbmciPgogICAgPHRleHQgdHJhbnNmb3JtPSJ0cmFuc2xhdGUoMjAw
LCAzMjApIHJvdGF0ZSgzMCkiIGNsYXNzPSJyaW5nIj5mb2N1c2VkIHRleHQ8L3RleHQ+CiAgPC9n
PgogIAogIDxjbGlwUGF0aCBpZD0iY2xpcCI+CiAgICA8cmVjdCB3aWR0aD0iMTAwIiBoZWlnaHQ9
IjEwMCIgLz4KICA8L2NsaXBQYXRoPgogIDxpbWFnZSB4bGluazpocmVmPSJyZXNvdXJjZXMvZ3Jl
ZW4tY2hlY2tlci5wbmciIHg9IjEwIiB5PSI0MDAiIGNsaXAtcGF0aD0idXJsKCNjbGlwKSIgY2xh
c3M9InJpbmciIC8+CiAgPGltYWdlIHhsaW5rOmhyZWY9InJlc291cmNlcy9ncmVlbi1jaGVja2Vy
LnBuZyIgdHJhbnNmb3JtPSJ0cmFuc2xhdGUoMjAwLCA0MDApIHJvdGF0ZSgzMCkiIGNsYXNzPSJy
aW5nIiBjbGlwLXBhdGg9InVybCgjY2xpcCkiLz4KICA8ZyBjbGFzcz0icmluZyIgdHJhbnNmb3Jt
PSJ0cmFuc2xhdGUoMzYwLCA0MDApIHJvdGF0ZSgzMCkiPgogICAgPGltYWdlIHhsaW5rOmhyZWY9
InJlc291cmNlcy9ncmVlbi1jaGVja2VyLnBuZyIgY2xpcC1wYXRoPSJ1cmwoI2NsaXApIi8+CiAg
PC9nPgogIAo8L3N2Zz4K
</data>

          </attachment>
      

    </bug>

</bugzilla>