Closed Bug 1452890 Opened 6 years ago Closed 6 years ago

Move right-click selection from "enterSelection" to its own bucket

Categories

(Firefox :: Address Bar, defect, P1)

55 Branch
defect

Tracking

()

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

(Reporter: mak, Assigned: mak)

References

Details

(Whiteboard: [fxsearch])

Attachments

(1 file)

Comment on attachment 8967734 [details]
Bug 1452890 - Move address bar right-click selection from 'enterSelection' to its own histogram bucket.

https://reviewboard.mozilla.org/r/236452/#review242354

Wow, good catch...  Do you think "right-click" would be a better label for this userSelectionBehavior?  To me, "click" means a left-click, and that's a little confusing here.  Or maybe even "enter-after-right-click", since the enter key is still involved -- the selection is not triggered by a right-click alone.
Attachment #8967734 - Flags: review?(adw) → review+
(In reply to Drew Willcoxon :adw from comment #3)
> Wow, good catch...  Do you think "right-click" would be a better label for
> this userSelectionBehavior?  To me, "click" means a left-click, and that's a
> little confusing here.  Or maybe even "enter-after-right-click", since the
> enter key is still involved -- the selection is not triggered by a
> right-click alone.

Imo, it doesn't matter yet, what we care is whether the user selected with the mouse or the keyboard, and this case covers a negligible (<0.5%) percentage.
In the future, if we introduce a contextual menu, we may want to measure its usage, but for now it's a bit pointless.
Agree about negligibility, just think the label could be clearer.  I leave it up to you
ok, fair enough, it's not particularly complicate, it will probably be a very tiny bucket.
Summary: Move right-click selection from "enterSelection" to the "click" bucket → Move right-click selection from "enterSelection" to its own bucket
I was about to use rightClickEnterSelection, but histogram labels have a 20 chars limit, so I guess I'll just use rightClickEnter
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/eb41147ad8b7
Move address bar right-click selection from 'enterSelection' to its own histogram bucket. r=adw
https://hg.mozilla.org/mozilla-central/rev/eb41147ad8b7
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Hey Marco, sorry for the fly-by: please note that adding new labels to categorical histograms isn't supported on the pipeline side: see https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/collection/histograms.html#categorical
Flags: needinfo?(mak77)
Hi, well we actually already added categories here before this change...

I think the documentation is unclear, at
https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/collection/histograms.html#changing-a-histogram
it states:
"The one exception is categorical histograms which can only be changed by adding labels, and only until it reaches 50 labels."
which documentation is right?

Fwiw, I'd have no problem at renaming this histogram and letting the old one die...
Flags: needinfo?(mak77) → needinfo?(alessio.placitelli)
Turns out I was wrong :) This can be done up to 50 labels without any problòem.
Flags: needinfo?(alessio.placitelli)
You need to log in before you can comment on or make changes to this bug.