Closed Bug 1077471 Opened 11 years ago Closed 11 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
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
Status: ASSIGNED → RESOLVED
Closed: 11 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.

Attachment

General

Created:
Updated:
Size: