674229 - Repeat bubbles in Web Console are too close to the Source Link

Status: RESOLVED FIXED

Description

[Rob Campbell [:rc] (:robcee)]

The repeat bubbles in the web console are running into the source link area in the web console. See attached screenshot.

Comment 1

[Rob Campbell [:rc] (:robcee)]

added pack="end" to the repeated nodes to ensure adequate spacing is retained.

Comment 2

[Dão Gottwald [::dao]]

width:1em on the label is causing this problem, isn't it? Why is this applied in the first place?

Comment 3

[David Dahl :ddahl]

(In reply to comment #2)
> width:1em on the label is causing this problem, isn't it? Why is this
> applied in the first place?

In the CSS, I removed the width from the node and from the 'webconsole-location' node, and got this result:
http://img840.imageshack.us/img840/4610/selection028g.png

Comment 4

[Dão Gottwald [::dao]]

Comment on attachment 548662 [details] [diff] [review]
Packed

So if I understand it correctly, comment 3 confirms that width:1em isn't needed and removing it fixes this bug.

Comment 5

[Rob Campbell [:rc] (:robcee)]

the spacing looks a little tight around that source link, but at least it's not bleeding into it.

No idea why there's a 1em width on this, I don't remember.

Comment 6

[Rob Campbell [:rc] (:robcee)]

removed the pack attribute and excised the widths from both the source link and repeat bubbles. Result fits at least, but the bubbles are right aligned with the left edge of the source.

Comment 7

[Rob Campbell [:rc] (:robcee)]

Comment 8

[Dão Gottwald [::dao]]

Comment on attachment 548834 [details] [diff] [review]
unpacked

(In reply to comment #6)
> Result fits at least, but the bubbles are right aligned
> with the left edge of the source.

That's because you removed width:10em from .webconsole-location, which seems wrong... It's probably going to prevent the location from being cropped too.

Comment 9

[David Dahl :ddahl]

(In reply to comment #8)
> That's because you removed width:10em from .webconsole-location, which seems
> wrong... It's probably going to prevent the location from being cropped too.

What would be the ideal solution here? just remove the 1em width on the repeat bubble nodes? I was tinkering with it in DOMInspector and I thought the overlap was still there with just the 'width:1em' removed.

Comment 10

[Dão Gottwald [::dao]]

(In reply to comment #9)
> I was tinkering with it in DOMInspector and I thought
> the overlap was still there with just the 'width:1em' removed.

That's not what I see.

Also, do you know what change caused this problem? This seems to be a recent regression.

Comment 11

[Rob Campbell [:rc] (:robcee)]

(In reply to comment #10)
> (In reply to comment #9)
> > I was tinkering with it in DOMInspector and I thought
> > the overlap was still there with just the 'width:1em' removed.
> 
> That's not what I see.
> 
> Also, do you know what change caused this problem? This seems to be a recent
> regression.

yeah, it's a regression from bug 662807. align="start" seems to have compressed the layout of these nodes.

I'll re-add the width to the .webconsole-location and resubmit with a screenshot.

Comment 12

[Rob Campbell [:rc] (:robcee)]

this looks much better.

Comment 13

[Rob Campbell [:rc] (:robcee)]

With the width of the source link readded, this looks much better. Thanks!

Comment 14

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 549078 [details] [diff] [review]
[in-fx-team] unpacked, unwidthed

moving review to mihai to get this checked in.

Comment 15

[Mihai Sucan [:msucan]]

Comment on attachment 549078 [details] [diff] [review]
[in-fx-team] unpacked, unwidthed

Good to go!

Comment 16

[Dão Gottwald [::dao]]

(In reply to comment #14)
> Comment on attachment 549078 [details] [diff] [review] [diff] [details] [review]
> unpacked, unwidthed
> 
> moving review to mihai to get this checked in.

FYI, I'm the owner of toolkit/themes/.

Comment 17

[Rob Campbell [:rc] (:robcee)]

(In reply to comment #16)
> (In reply to comment #14)
> > Comment on attachment 549078 [details] [diff] [review] [diff] [details] [review] [diff] [details] [review]
> > unpacked, unwidthed
> > 
> > moving review to mihai to get this checked in.
> 
> FYI, I'm the owner of toolkit/themes/.

I know. I just wanted signoff from a devtools peer too.

Thanks for the reviews, all!

Comment 18

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 549078 [details] [diff] [review]
[in-fx-team] unpacked, unwidthed

http://hg.mozilla.org/integration/fx-team/rev/e2e7b2275ade

Comment 19

[Tim Taubert [:ttaubert] (inactive)]

http://hg.mozilla.org/mozilla-central/rev/e2e7b2275ade