Bug 137248

Summary: Web Inspector: [GTK] Missing icons and enhancing the proportion and visibility of some icons
Product: WebKit Reporter: Andres Gomez Garcia <agomez>
Component: WebKitGTKAssignee: Andres Gomez Garcia <agomez>
Status: RESOLVED FIXED    
Severity: Normal CC: agomez, cgarcia, commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Updated patch
none
WebInspector icons before the patch
none
WebInspector icons after the patch none

Description Andres Gomez Garcia 2014-09-30 02:59:38 PDT
After fixing bug 136632, there are still some icons missing.

In addition, some of the new ones are to light to be understood, like many of the DOM elements.

Finally, the proportion of some icons, specially the ones in the navigator bar, is not correct. Some are smaller/bigger than others, unlike in the Apple version. This could be enhanced.
Comment 1 Andres Gomez Garcia 2014-09-30 03:28:21 PDT
Created attachment 238920 [details]
Patch
Comment 2 Andres Gomez Garcia 2014-10-01 02:46:33 PDT
Created attachment 239013 [details]
Updated patch

Removed the PDF icon. I will open a new bug for replacing it with a SVG version.
Comment 3 Andres Gomez Garcia 2014-10-01 02:47:10 PDT
Created attachment 239014 [details]
WebInspector icons before the patch
Comment 4 Andres Gomez Garcia 2014-10-01 02:47:35 PDT
Created attachment 239015 [details]
WebInspector icons after the patch
Comment 5 Andres Gomez Garcia 2014-10-01 03:05:57 PDT
Opened bug 137296 for the remaining PDF icon.
Comment 6 Carlos Garcia Campos 2014-10-02 03:19:54 PDT
Comment on attachment 239013 [details]
Updated patch

This requires a clean build in the bots
Comment 7 WebKit Commit Bot 2014-10-02 03:57:36 PDT
Comment on attachment 239013 [details]
Updated patch

Clearing flags on attachment: 239013

Committed r174207: <http://trac.webkit.org/changeset/174207>
Comment 8 WebKit Commit Bot 2014-10-02 03:57:39 PDT
All reviewed patches have been landed.  Closing bug.