157183 - Even though XUL Element Tree is focusable, Both the inspect tool and API return wrong value

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[dsirnapalli]

<tree seltype="single" flex="1">
  <treecols>
    <treecol id="sender" label="Sender" flex="1"/>
    <treecol id="subject" label="Subject" flex="2"/>
  </treecols>
  
  <treechildren>
    <treeitem>
      <treerow>
        <treecell label="[e-mail]"/>
        <treecell label="Top secret plans"/>
      </treerow>  
    </treeitem>
    <treeitem>
      <treerow>
        <treecell label="[e-mail]"/>
        <treecell label="Lets do lunch"/>
      </treerow>  
    </treeitem>      
  </treechildren>
</tree>

Enter the above code in xul file and open it in mozilla. Open Inspect tool and
click on "Show Hightlight Rectangle". Now Click somewhere on mozilla and tab
until the focus is on Tree Node. We get the focus on to the tree node. That
means Tree Element is focusable. Check the Inspect tool, you see only "read
only" for state. It should be "read only, focused, focusable".

If i made mistake correct me.
This was working ok previously.

Comment 1

[Kyle Yuan]

Yes, it's my fault. I disabled the focusable state for tree in my last patch.
Will correct soon.

Comment 2

[Kyle Yuan]

Attachment: added focused and focusable state to tree

Comment 3

[Kyle Yuan]

John, could you r=?

Comment 4

[John Gaunt (redfive)]

Comment on attachment 91309 [details] [diff] [review]
added focused and focusable state to tree

r=jgaunt

Comment 5

[jag (Peter Annema)]

Comment on attachment 91309 [details] [diff] [review]
added focused and focusable state to tree

sr=jag

Comment 6

[Asa Dotzler [:asa]]

Comment on attachment 91309 [details] [diff] [review]
added focused and focusable state to tree

a=asa (on behalf of drivers) for checkin to 1.1

Comment 7

[Kyle Yuan]

checked into trunk!