Closed Bug 1553305 Opened 5 years ago Closed 3 years ago

Profiler popup icon should be more accessible when turned on

Categories

(DevTools :: Performance Tools (Profiler/Timeline), enhancement, P2)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gregtatum, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: access)

Right now it just changes color. There is a small regression from features from the web extension, where the label changed from "Gecko Profiler (on)" and "Gecko Profiler (off)". This label should be updated in devtools/startup/ProfilerMenuButton.jsm by directly modifying the attribute. The problem is that the .jsm file doesn't directly have access to localization dependencies. I'm not sure what's the best course to localize that label, hence this bug, which can investigate it.

$("#profiler-button").setAttribute("tooltiptext", "The profiler is running")

Depends on: 1307227
Priority: -- → P2

Thanks for the bug! The title can indeed be important for screen reader users.
I'd like to add that we should also change the shape of the icon (for example with a shadow or something ?) when it's on, this is for folks that can actually see but don't see colors right.

Idea:

  • Stopped should have the hand pointing up,
  • Running could have the hand elsewhere, and maybe some "speed lines" to infer movement, or filling the space between "noon" and the hand position.

I'm guessing animating it would be too distracting and resource-wasting.

Blocks: 1566920
Keywords: access
Blocks: 1572786
Component: Gecko Profiler → Performance Tools (Profiler/Timeline)
Product: Core → DevTools
Version: 62 Branch → Trunk
No longer blocks: 1572786

The icon itself now changes, and the tooltip (localized) also changes, so I believe this is now fixed.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.