391798 - add an UI ability to enable/disable accessibility views

Status: RESOLVED WONTFIX

(Other Applications :: DOM Inspector, defect)

Description

[alexander :surkov (:asurkov)]

Since we have a lot views for accessibility inspection and I think we don't need to keep them turned on by default then we need an ability to enable/disable it.

Comment 1

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 2

[[e-mail]]

Doesn't View->Show Accessible Nodes already turn off those panels?

Comment 3

[alexander :surkov (:asurkov)]

(In reply to comment #2)
> Doesn't View->Show Accessible Nodes already turn off those panels?
> 

No, this option makes/unmakes bold accessible nodes in 'DOMNodes' view and enables/disabled accessible related views in right panel. But views for the left panel aren't depend on this option.

I added an option to enable/disable all accessible views and if they are disabled then it disables 'Show Accessible Nodes' option too.

Comment 4

[[e-mail]]

Well I still fail to see why you need two options that do almost the same...

Comment 5

[alexander :surkov (:asurkov)]

(In reply to comment #4)
> Well I still fail to see why you need two options that do almost the same...
> 

1. Disable completely accessibility support. Accessibility features are used only by those users who care about accessibility of his page or by accessibility developers. We should allow not to know about ally for the rest of users.

2. Show/hide accessible nodes inside DOMNodes view. I often inspect accessibility and I keep enabled the first option always. But sometimes I don't need to see accessibles objects in DOMNodes view. In that case I should be able to hide them because they marked bold and this draws away my attention.

Comment 6

[[e-mail]]

OK, so how does bug 394253 affect this?

Comment 7

[alexander :surkov (:asurkov)]

(In reply to comment #6)
> OK, so how does bug 394253 affect this?
> 

I need to update the patch to trunk.

Comment 8

[[e-mail]]

But I still don't see the point of hiding the views...

Comment 9

[alexander :surkov (:asurkov)]

(In reply to comment #8)
> But I still don't see the point of hiding the views...
> 

I think they make DOMi UI more complicated. I think users who don't want ally stuff shouldn't see those views. No?

Comment 10

[Shawn Wilsher :sdwilsh]

(In reply to comment #9)
> I think they make DOMi UI more complicated. I think users who don't want ally
> stuff shouldn't see those views. No?
Well, there are a lot of viewers that I never use, but are still there.  I'm not sure if it will hurt us :/

Comment 11

[alexander :surkov (:asurkov)]

Ok, if you both do not see usability of this then worksforme?

Comment 12

[Shawn Wilsher :sdwilsh]

Comment on attachment 277914 [details] [diff] [review]
patch

I am indifferent on the resolution.

Comment 13

[alexander :surkov (:asurkov)]

ok, wontfix then