* 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.
Created attachment 308497 [details] Patch
Comment on attachment 308497 [details] Patch These look fantastic! r=me
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.
Created attachment 308498 [details] screenshot of before and after this fix
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
Thank you for your help with maintaining the web inspector icons! We had been worried about what to do when Andres stopped maintaining them.
Comment on attachment 308497 [details] Patch Clearing flags on attachment: 308497 Committed r215920: <http://trac.webkit.org/changeset/215920>
All reviewed patches have been landed. Closing bug.