<?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>224026</bug_id>
          
          <creation_ts>2021-03-31 16:56:56 -0700</creation_ts>
          <short_desc>Web Inspector: adapt :focus-visible internally</short_desc>
          <delta_ts>2022-02-08 12:51:07 -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>Web Inspector</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>ASSIGNED</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Nikita Vasilyev">nvasilyev</reporter>
          <assigned_to name="Nikita Vasilyev">nvasilyev</assigned_to>
          <cc>firefoxic.dev</cc>
    
    <cc>inspector-bugzilla-changes</cc>
    
    <cc>rego</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1745856</commentid>
    <comment_count>0</comment_count>
    <who name="Nikita Vasilyev">nvasilyev</who>
    <bug_when>2021-03-31 16:56:56 -0700</bug_when>
    <thetext>:focus-visible is implemented behind an experimental flag (&quot;:focus-visible pseudo class&quot;).
Once it&apos;s on by default, we should add:

    :focus:not(:focus-visible) {
        outline: none;
    }

This will allow to closer match macOS focus behavior. For example, currently, clicking on &quot;Hover&quot; checkbox (to force `:hover` state) displays the focus outline around it.
With the CSS rule above, it would only draw the outline when it&apos;s focused by using keyboard navigation, which matches native apps behavior.

https://twitter.com/leaverou/status/1045768279753666562?lang=en</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1745857</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-03-31 16:57:13 -0700</bug_when>
    <thetext>&lt;rdar://problem/76078422&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1750557</commentid>
    <comment_count>2</comment_count>
    <who name="Manuel Rego Casasnovas">rego</who>
    <bug_when>2021-04-15 03:24:20 -0700</bug_when>
    <thetext>Ideally the default UA should change to use :focus-visible directly, so that kind of rule won&apos;t be needed.

See bug #221925.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1833536</commentid>
    <comment_count>3</comment_count>
    <who name="Nikita Vasilyev">nvasilyev</who>
    <bug_when>2022-01-24 15:41:24 -0800</bug_when>
    <thetext>(In reply to Manuel Rego Casasnovas from comment #2)
&gt; Ideally the default UA should change to use :focus-visible directly, so that
&gt; kind of rule won&apos;t be needed.
&gt; 
&gt; See bug #221925.

Thanks for the heads up. That&apos;s great!

This bug then is about removing obscure focus management logic that predates :focus-visible. One example:
https://github.com/WebKit/WebKit/blob/37d73f68d97e0a6c88cb19572874d140233ffbeb/Source/WebInspectorUI/UserInterface/Views/NavigationBar.js#L290-L294</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>