RESOLVED FIXED Bug 155557
Web Inspector: Quick Open dialog has fuzzy icons at 1x
https://bugs.webkit.org/show_bug.cgi?id=155557
Summary Web Inspector: Quick Open dialog has fuzzy icons at 1x
Joseph Pecoraro
Reported 2016-03-16 13:41:33 PDT
* SUMMARY Quick Open dialog has fuzzy icons at 1x. File icons are 16x16 being displayed at 24x24 size. * STEPS TO REPRODUCE 1. Open Inspector 2. Show Quick Open Dialog (Cmd-Shift-O) 3. Type 't' to get results => fuzzy icons
Attachments
[Image] Dialog with 16x16 icons (253.80 KB, image/png)
2016-03-16 16:59 PDT, Matt Baker
no flags
[Patch] Proposed Fix (100.03 KB, patch)
2016-03-23 11:29 PDT, Matt Baker
no flags
[Image] 32px 1x/2x icon set (36.22 KB, image/png)
2016-03-23 11:30 PDT, Matt Baker
no flags
Radar WebKit Bug Importer
Comment 1 2016-03-16 13:42:19 PDT
Timothy Hatcher
Comment 2 2016-03-16 14:40:41 PDT
We can deviate from Xcode here and use 16x16 icons. I don't want to add larger icons just for this.
Matt Baker
Comment 3 2016-03-16 16:14:54 PDT
(In reply to comment #2) > We can deviate from Xcode here and use 16x16 icons. I don't want to add > larger icons just for this. Sounds good.
Matt Baker
Comment 4 2016-03-16 16:59:50 PDT
Created attachment 274238 [details] [Image] Dialog with 16x16 icons Large tree items present better hit targets, but it looks odd with small icons.
Timothy Hatcher
Comment 5 2016-03-16 20:58:01 PDT
Hmm. Yeah, that does look worse. Though I suspect most users will uses arrow keys and enter to pick an item, not the mouse. Options: 1) Modifiy the UI to work with small icons, so smaller rows and input field to match. 2) Make 32x32 or 24x24 resource icons.
Matt Baker
Comment 6 2016-03-23 11:29:41 PDT
Created attachment 274761 [details] [Patch] Proposed Fix
Matt Baker
Comment 7 2016-03-23 11:30:35 PDT
Created attachment 274762 [details] [Image] 32px 1x/2x icon set
Joseph Pecoraro
Comment 8 2016-03-23 12:16:23 PDT
Comment on attachment 274761 [details] [Patch] Proposed Fix We should file a bug on GTK notifying them they may want to adjust their icons.
Matt Baker
Comment 9 2016-03-23 12:42:37 PDT
(In reply to comment #8) > Comment on attachment 274761 [details] > [Patch] Proposed Fix > > We should file a bug on GTK notifying them they may want to adjust their > icons. https://bugs.webkit.org/show_bug.cgi?id=155802
WebKit Commit Bot
Comment 10 2016-03-23 13:08:48 PDT
Comment on attachment 274761 [details] [Patch] Proposed Fix Clearing flags on attachment: 274761 Committed r198590: <http://trac.webkit.org/changeset/198590>
WebKit Commit Bot
Comment 11 2016-03-23 13:08:52 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.