1686929 - [Proton] Only show "Inspect Accessibility Properties" context menu item if the user has opened devtools at least once

Status: VERIFIED FIXED

(Firefox :: Menus, enhancement, P1)

Description

[:Gijs (he/him)]

Right now there are 2 "Inspect" items in the context menu. By default, we should only show one, and we should only show the other one if users actually use the "inspect" item that's there, as most users won't use either.

Comment 1

[Yura Zenevich [:yzen] (please use "needinfo")]

From Matrix:
We should add method to DevToolsShim (used in nsContextMenu) that simply calls and returns DevToolsStartup.isDevToolsUser

DevToolsShim.isDevToolsUser would be an additional check (or replace some of the prefs) when checking to build the context menu item for a11y inspection.

Comment 2

[:Gijs (he/him)]

Attachment: Bug 1686929 - only show the inspect a11y properties menuitem for devtools users, r?jdescottes

Comment 3

[Blake Winton (:bwinton) (:☕️)]

As per guidance from Vicky, for tracking, we're marking all the bugs that people are working on as P1.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6b2d3a1aea51
only show the inspect a11y properties menuitem for devtools users, r=jdescottes

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/6b2d3a1aea51

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.

Comment 7

[Bogdan Maris, Desktop Test Engineering]

Verified that this works as expected in latest Nightly across platforms (Windows 10, macOS 11.3 and Ubuntu 18.04).