Bug 155077 – [GTK] Web Inspector: Add new GTK+ icons for different Script Timeline Views (Events and Call Trees)

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

* SUMMARY
Add new GTK+ icons for different Script Timeline Views (Events and Call Trees).

- Events.svg
- CallTrees.svg

Mac icons were based off of ResultLine.svg, so that is what we currently fallback to on GTK.

[Fujii Hironori]

Created attachment 308497 [details]
Patch

[Joseph Pecoraro]

Comment on attachment 308497 [details]
Patch

These look fantastic! r=me

[Joseph Pecoraro]

Comment on attachment 308497 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=308497&action=review

> Source/WebInspectorUI/UserInterface/Images/gtk/CallTrees.svg:7
> + <rect width="8.1401" height="1.9388" x="2.2201" y="2.9519" fill="#8f918c" rx="0.5" ry=".5"/>
> + <rect width="7.5399" height="1.9388" x="5.0149" y="7.0324" fill="#8f918c" rx=".5" ry=".5"/>
> + <rect width="5.5891" height="1.9388" x="8.2412" y="11.113" fill="#8f918c" rx="0.5" ry=".5"/>

Weird that sometimes there is "0.5" and other times there are just ".5". You may want to make them consistent.

[Fujii Hironori]

Created attachment 308498 [details]
screenshot of before and after this fix

[Fujii Hironori]

Thank you for the review.

(In reply to Joseph Pecoraro from comment #3)
> Weird that sometimes there is "0.5" and other times there are just ".5". You
> may want to make them consistent.

Andres's script optimizing.sh uses scour and svgo.
There two problems:

* scour 0.3 produces this inconsistent values
* Andres specifies cleanupNumericValues:false to svgo

To solve this, 
1) Use the latest scour
2) Or, specify cleanupNumericValues:true

[Michael Catanzaro]

Thank you for your help with maintaining the web inspector icons! We had been worried about what to do when Andres stopped maintaining them.

[WebKit Commit Bot]

Comment on attachment 308497 [details]
Patch

Clearing flags on attachment: 308497

Committed r215920: <http://trac.webkit.org/changeset/215920>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.