Closed Bug 1077471 Opened 7 years ago Closed 7 years ago

Icons for new performance tool

Categories

(DevTools :: Performance Tools (Profiler/Timeline), defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 37

People

(Reporter: jsantell, Assigned: ntim)

References

Details

Attachments

(3 files)

From https://moqups.com/paulrouget/be6zm6hH/p:ad2d763eb

* toggling memory on/off
* toggling between detail views (call tree vs. waterfall vs. flamechart)
* toggling between overview views (timeline vs. frames)
* toggling between memory views (flamechart vs call tree)
Paul: We should start thinking about these icons now, as we're getting these things implemented. Maybe ping Darren?
Flags: needinfo?(paul)
Pinging Darrin and Tim on this
Flags: needinfo?(ntim007)
Flags: needinfo?(dhenein)
Flags: needinfo?(dhenein) → needinfo?(shorlander)
While waiting for UX, I think we should do something like that:

http://people.mozilla.org/~prouget/perfIcons.html (buttons are clickable)
Flags: needinfo?(paul)
Attached file icons.zip
Live demo : http://ntim.altervista.org/FirefoxUX/PerfTool/
Flags: needinfo?(ntim007)
Attached image performance-icons.svg
Now as icon sprite.
Attached patch perf-icons.patchSplinter Review
A patch I made to test the icons.
Attachment #8530553 - Flags: review?(jsantell)
Comment on attachment 8530553 [details] [diff] [review]
perf-icons.patch

Review of attachment 8530553 [details] [diff] [review]:
-----------------------------------------------------------------

Tim these look great! Let's get these landed so we can start using them in the other patches.

Thanks!
Attachment #8530553 - Flags: review?(jsantell) → review+
Assignee: nobody → ntim007
Status: NEW → ASSIGNED
Flags: needinfo?(shorlander)
Thanks !
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/68c805b2b5e3
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 37
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.