Bug 91757 - Web Inspector: Status bar icons are misaligned by 1px to the right
Summary: Web Inspector: Status bar icons are misaligned by 1px to the right
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: dubroy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-19 10:03 PDT by dubroy
Modified: 2012-07-20 06:08 PDT (History)
10 users (show)

See Also:


Attachments
Patch (29.33 KB, patch)
2012-07-19 10:12 PDT, dubroy
no flags Details | Formatted Diff | Diff
Screenshot of old status bar, and result after applying the patch (12.72 KB, image/png)
2012-07-19 10:15 PDT, dubroy
no flags Details
Patch (1.42 KB, patch)
2012-07-19 10:39 PDT, Pavel Feldman
no flags Details | Formatted Diff | Diff
Patch (2.00 KB, patch)
2012-07-19 10:49 PDT, Pavel Feldman
no flags Details | Formatted Diff | Diff
Glyphs with revised window docking icons (3.05 KB, image/png)
2012-07-19 12:33 PDT, dubroy
no flags Details
Patch (13.52 KB, patch)
2012-07-19 13:39 PDT, Pavel Feldman
no flags Details | Formatted Diff | Diff
[IMAGE] Screenshot with patch applied. (8.55 KB, image/png)
2012-07-19 13:57 PDT, Pavel Feldman
no flags Details
Revised glyphs with long-press indicator fixed (3.05 KB, image/png)
2012-07-19 14:13 PDT, dubroy
no flags Details
Patch (14.03 KB, patch)
2012-07-20 01:46 PDT, Pavel Feldman
vsevik: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description dubroy 2012-07-19 10:03:27 PDT
Icons in the statusbar are shifted 1px to the right because they sprite assumes a 32px grid, but the button backgrounds are only 30px (with a 1px border).

Also, the icons for dock-to-right, undock, etc. are an odd number of pixels wide, so they can't be perfectly centered.
Comment 1 dubroy 2012-07-19 10:12:53 PDT
Created attachment 153290 [details]
Patch
Comment 2 dubroy 2012-07-19 10:15:54 PDT
Created attachment 153293 [details]
Screenshot of old status bar, and result after applying the patch
Comment 3 Pavel Feldman 2012-07-19 10:39:09 PDT
Created attachment 153298 [details]
Patch
Comment 4 Pavel Feldman 2012-07-19 10:46:08 PDT
Historically, the glyphs were 32x24. It is just that migration from background images with painted border to the CSS borders shifted things. Shifting glyhs 1px should fix that.
Comment 5 Pavel Feldman 2012-07-19 10:49:46 PDT
Created attachment 153305 [details]
Patch
Comment 6 dubroy 2012-07-19 11:52:22 PDT
Your patch will also reposition the .scripts-navigator-show-hide-button, etc. I'm not sure that we want to do that.
Comment 7 dubroy 2012-07-19 12:33:33 PDT
Created attachment 153321 [details]
Glyphs with revised window docking icons

Can you include the attached image in your patch? It reduces the width of the window docking icons by 1px, so that they have an even width and can be horizontally centered.
Comment 8 Pavel Feldman 2012-07-19 13:39:07 PDT
Created attachment 153337 [details]
Patch
Comment 9 Pavel Feldman 2012-07-19 13:56:12 PDT
(In reply to comment #7)
> Created an attachment (id=153321) [details]
> Glyphs with revised window docking icons
> 
> Can you include the attached image in your patch? It reduces the width of the window docking icons by 1px, so that they have an even width and can be horizontally centered.

It looks good, but the long-press-indicator in the dock icon seems to be 1px off (touches the border). See the screenshot.
Comment 10 Pavel Feldman 2012-07-19 13:57:08 PDT
Created attachment 153340 [details]
[IMAGE] Screenshot with patch applied.
Comment 11 dubroy 2012-07-19 14:13:25 PDT
Created attachment 153343 [details]
Revised glyphs with long-press indicator fixed

Whoops, I had assumed that was moving over 1px as well, since it did in my fix. New image attached.
Comment 12 Pavel Feldman 2012-07-20 01:46:56 PDT
Created attachment 153449 [details]
Patch
Comment 13 Pavel Feldman 2012-07-20 06:08:18 PDT
Committed r123207: <http://trac.webkit.org/changeset/123207>