Bug 100525 - Web Inspector: The bubble for repeated errors is misplaced.
Summary: Web Inspector: The bubble for repeated errors is misplaced.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Trivial
Assignee: Mike West
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-26 08:06 PDT by Mike West
Modified: 2012-10-29 01:48 PDT (History)
9 users (show)

See Also:


Attachments
Uglay. (128.65 KB, image/png)
2012-10-26 08:06 PDT, Mike West
no flags Details
Patch (2.63 KB, patch)
2012-10-26 10:56 PDT, Mike West
no flags Details | Formatted Diff | Diff
After. (110.81 KB, image/png)
2012-10-26 10:57 PDT, Mike West
no flags Details
Three digits. (85.10 KB, image/png)
2012-10-26 14:15 PDT, Mike West
no flags Details
Patch (2.56 KB, patch)
2012-10-26 14:21 PDT, Mike West
no flags Details | Formatted Diff | Diff
After the new patch. (15.08 KB, image/png)
2012-10-26 14:22 PDT, Mike West
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mike West 2012-10-26 08:06:58 PDT
Created attachment 170928 [details]
Uglay.

The bubble for repeated errors is misplaced, pushing the error content down a line. It's ugly. Let's fix that.
Comment 1 Mike West 2012-10-26 10:56:58 PDT
Created attachment 170959 [details]
Patch
Comment 2 Mike West 2012-10-26 10:57:34 PDT
Created attachment 170960 [details]
After.
Comment 3 Alexander Pavlov (apavlov) 2012-10-26 14:02:31 PDT
(In reply to comment #2)
> Created an attachment (id=170960) [details]
> After.

In the screenshot, the right-arrow next to (42) is not horizontally aligned with that next to "This is error message" above (looks shifted rightwards 1 or 2 px). Is this misalignment intentionally allowed due to the potentially unbounded bubble width?
Comment 4 Mike West 2012-10-26 14:15:02 PDT
Created attachment 171003 [details]
Three digits.
Comment 5 Mike West 2012-10-26 14:17:59 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > Created an attachment (id=170960) [details] [details]
> > After.
> 
> In the screenshot, the right-arrow next to (42) is not horizontally aligned with that next to "This is error message" above (looks shifted rightwards 1 or 2 px). Is this misalignment intentionally allowed due to the potentially unbounded bubble width?

Hrm. Yes, that was a compromise I was ok with making, but now I'm looking at it again. I think we can do better.
Comment 6 Mike West 2012-10-26 14:21:34 PDT
Created attachment 171006 [details]
Patch
Comment 7 Mike West 2012-10-26 14:22:13 PDT
Created attachment 171007 [details]
After the new patch.
Comment 8 Mike West 2012-10-26 14:22:40 PDT
(In reply to comment #5)
> (In reply to comment #3)
> > (In reply to comment #2)
> > > Created an attachment (id=170960) [details] [details] [details]
> > > After.
> > 
> > In the screenshot, the right-arrow next to (42) is not horizontally aligned with that next to "This is error message" above (looks shifted rightwards 1 or 2 px). Is this misalignment intentionally allowed due to the potentially unbounded bubble width?
> 
> Hrm. Yes, that was a compromise I was ok with making, but now I'm looking at it again. I think we can do better.

How about the current patch? It looks like https://bugs.webkit.org/attachment.cgi?id=171007
Comment 9 Pavel Feldman 2012-10-29 01:26:44 PDT
Comment on attachment 171006 [details]
Patch

Is flex box necessary to resolve this? We try not to use it due to its constant behavior changes.
Comment 10 Mike West 2012-10-29 01:41:44 PDT
(In reply to comment #9)
> (From update of attachment 171006 [details])
> Is flex box necessary to resolve this? We try not to use it due to its constant behavior changes.

The bubble is inline-block, pushing the 'ol' (block) down to the next line. I tried two other solutions before landing on flexbox:

1. Make the 'ol' inline-block. This has the unfortunate side-effect of shrink-wrapping the list to fit it's content. I tried fiddling with the width via -webkit-calc, but that's even flakier than flexbox.

2. Float the bubble. This works. But I hate float.

I chatted a bit with Alexander about this; he's ok with landing it as-is. If either of us come up with a brilliant solution to #1 above, I'm more than happy to spin up another patch.

That said, flexbox is the future! If it's broken, we should fix it, rather than avoiding it. :)
Comment 11 WebKit Review Bot 2012-10-29 01:48:35 PDT
Comment on attachment 171006 [details]
Patch

Clearing flags on attachment: 171006

Committed r132772: <http://trac.webkit.org/changeset/132772>
Comment 12 WebKit Review Bot 2012-10-29 01:48:39 PDT
All reviewed patches have been landed.  Closing bug.