Instrument inspection of shortcut combination used for toolbox opening with event telemetry

RESOLVED FIXED in Firefox 62

Status

P2
enhancement
RESOLVED FIXED
7 months ago
2 months ago

People

(Reporter: Harald, Assigned: miker)

Tracking

(Blocks: 1 bug)

61 Branch
Firefox 62

Firefox Tracking Flags

(firefox61 wontfix, firefox62 fixed)

Details

Attachments

(2 attachments, 2 obsolete attachments)

(Reporter)

Description

7 months ago
Devtools has arguably accumulated too many shortcuts over time. This would be useful data for the many discussions we have on which shortcuts are useful and which can be given up (and in the future maybe offer customization).

Format TBD, input welcome
(Reporter)

Comment 1

7 months ago
Mike for input on sizing this up; does devtools have this information available already to report?

I would be very interested to get this into 61 as part of M1 depending on hard it is and assuming this can be uplifted. Shortcuts are a major part of investigating accidental usage further.
Flags: needinfo?(mratcliffe)

Updated

7 months ago
Severity: normal → enhancement
Priority: -- → P2
(In reply to :Harald Kirschner :digitarald from comment #1)
> Mike for input on sizing this up; does devtools have this information
> available already to report?
> 
> I would be very interested to get this into 61 as part of M1 depending on
> hard it is and assuming this can be uplifted. Shortcuts are a major part of
> investigating accidental usage further.

No we don't but I am sure we could add something to the toolbox that would register all key events that are function keys or contain a modifier e.g. [cmd], [alt] or [ctrl].

Shall we add that to telemetry M2?
Flags: needinfo?(mratcliffe)
Screenshots would like to take over the Shift + Ctrl + S shortcut, but only if it's not getting much use as a devtools shortcut.

I'd be happy to help with implementation, if developer time is a blocker here.
Flags: needinfo?(mratcliffe)
(Reporter)

Comment 4

6 months ago
M2 works for this, which is scheduled to land in 62.
Flags: needinfo?(mratcliffe)
Summary: Add event property for shortcut combination used for toolbox opening → Instrument inspection of shortcut combination used for toolbox opening with event telemetry
Comment hidden (obsolete)
Attachment #8979194 - Flags: review?(andrei.br92) → review?(francois)
The following keys are used to open the toolbox:

Toggle Toolbox: accel+alt+I, accel+shift+I, F12
Browser Toolbox: accel+alt+shift+I
Inspector: accel+alt+C, accel+shift+C
Web Console: accel+alt+K, accel+shift+K
Debugger: accel+alt+S, accel+shift+S
Net Monitor: accel+alt+E, accel+shift+E
Style Editor: shift+f7
Performance: shift+f5
Storage: shift+f9
DOM: accel+alt+W, accel+shift+W
We already have this:

devtools.main:
  open:
    objects: ["tools"]
    bug_numbers: [1416024]
    notification_emails: ["dev-developer-tools@lists.mozilla.org", "hkirschner@mozilla.com"]
    record_in_processes: ["main"]
    description: User opens devtools toolbox.
    release_channel_collection: opt-out
    expiry_version: never
    extra_keys:
      entrypoint: How was the toolbox opened? CommandLine, ContextMenu, DeveloperToolbar, HamburgerMenu, KeyShortcut, SessionRestore or SystemMenu
      first_panel: The name of the first panel opened.
      host: "Toolbox host (positioning): bottom, side, window or other."
      splitconsole: Indicates whether the split console was open.
      width: Toolbox width rounded up to the nearest 50px.

So we would just need to add a shortcut property for the cases where entrypoint === "KeyShortcut".
Created attachment 8979656 [details]
data-review.txt
Attachment #8979656 - Flags: review?(francois)
Attachment #8979194 - Attachment is obsolete: true
Attachment #8979194 - Flags: review?(francois)
Created attachment 8979854 [details]
data-review.txt
Attachment #8979656 - Attachment is obsolete: true
Attachment #8979656 - Flags: review?(francois)
Attachment #8979854 - Flags: review?(francois)
Services.telemetry.recordEvent("devtools.main", "open", "tools", null, {
  "entrypoint": "KeyShortcut",
  "first_panel": "Inspector",
  "host": "bottom",
  "splitconsole": "false",
  "width": "777",
  "shortcut": "shift+f9"
})
Comment hidden (mozreview-request)
Assignee: nobody → mratcliffe
Status: NEW → ASSIGNED
Has Regression Range: --- → irrelevant
Has STR: --- → irrelevant
Comment hidden (mozreview-request)
Comment on attachment 8979854 [details]
data-review.txt

1) Is there or will there be **documentation** that describes the schema for the ultimate data set available publicly, complete and accurate?

Yes, Events.yaml.

2) Is there a control mechanism that allows the user to turn the data collection on and off?

Yes, telemetry setting.

3) If the request is for permanent data collection, is there someone who will monitor the data over time?**

Yes, Harald Kirschner.

4) Using the **[category system of data types](https://wiki.mozilla.org/Firefox/Data_Collection)** on the Mozilla wiki, what collection type of data do the requested measurements fall under?  **

Category 2

5) Is the data collection request for default-on or default-off?

Default on, all channels.

6) Does the instrumentation include the addition of **any *new* identifiers** (whether anonymous or otherwise; e.g., username, random IDs, etc.  See the appendix for more details)?

No.

7) Is the data collection covered by the existing Firefox privacy notice?

Yes.

8) Does there need to be a check-in in the future to determine whether to renew the data?

No, permanent.
Attachment #8979854 - Flags: review?(francois) → review+

Comment 14

6 months ago
mozreview-review
Comment on attachment 8979898 [details]
Bug 1456984 - Instrument inspection of shortcut combination used for toolbox opening with event telemetry

https://reviewboard.mozilla.org/r/246098/#review253512

Looks good!
Attachment #8979898 - Flags: review?(ystartsev) → review+

Comment 15

6 months ago
Pushed by mratcliffe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/10244430b101
Instrument inspection of shortcut combination used for toolbox opening with event telemetry r=yulia

Comment 16

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/10244430b101
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 62
status-firefox61: affected → wontfix

Updated

5 months ago
Product: Firefox → DevTools
Hey there - has enough Telemetry data been gathered to reach a conclusion about which keyboard shortcuts to keep and which to discard?
Flags: needinfo?(hkirschner)
(Reporter)

Comment 18

2 months ago
Following up in github as this issue isn't about discarding decisions.
Flags: needinfo?(hkirschner)
You need to log in before you can comment on or make changes to this bug.