Closed Bug 980248 Opened 6 years ago Closed 6 years ago

DevTools Themes: Extra top border showing up on responsive-container splitters when width > 700px

Categories

(DevTools :: Inspector, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 30

People

(Reporter: pbro, Assigned: bgrins)

References

Details

(Whiteboard: [good first verify])

Attachments

(2 files)

This is a screenshot of a portion of the inspector:
https://dl.dropboxusercontent.com/u/714210/Screenshot%202014-03-06%2011.14.40.png

If you look at the top of the splitter element (between the markup-view and rule-view), you will see that there are a couple of pixels that shouldn't be there.

Here is another screenshot in which I've made the splitter blue and wider, so it's easier to see:
https://dl.dropboxusercontent.com/u/714210/Screenshot%202014-03-06%2011.14.23.png

This is really minor, but some people (like me) might notice it.
I've quickly tried to play with margin/padding/border/... on the splitter element but no css property I tried to reset got rid of the extra pixels.
Same happens in the debugger.
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Depends on: 973191
Summary: Minor GUI glitch in the inspector splitter → DevTools Themes: Extra top border showing up on responsive-container splitters when width > 700px
There was an additional top border being added in all cases when the splitter was inside a .devtools-responsive-container.  It should only be added when width < 700px.

Rather than just wrap the rule in a media query, I've reorganized it a bit to avoid having 3 magic numbers for responsive width.
Attachment #8386794 - Flags: review?(pbrosset)
Comment on attachment 8386794 [details] [diff] [review]
splitter-extra-border.patch

Review of attachment 8386794 [details] [diff] [review]:
-----------------------------------------------------------------

Tested, works fine.
Had a little trouble understanding comment 3 at first, but got it now.
Attachment #8386794 - Flags: review?(pbrosset) → review+
https://hg.mozilla.org/mozilla-central/rev/bb950168dbda
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 30
Whiteboard: [good first verify]
[bugday-20140409]

Hi, I'm getting a 404 error when I try to view the dropbox screenshots (in bug description and comment 1). Could I get a screenshot to check which part of the Inspector splitter to look at?

Is it a valid check if I compare the splitter between Aurora and Firefox 28?
Flags: needinfo?(pbrosset)
You should rather compare the splitter between version 29 (current release) and aurora (30), since this bug got fixed in 30. 
The bug doesn't exist in 28 since it came along with the huge devtools theme changes made in 29.

Having said that, I'm trying to replicate the bug in 29, but I'm not able to for some reasons.

Brian, do you remember how to replicate this issue so it can be verified?
Flags: needinfo?(pbrosset) → needinfo?(bgrinstead)
(In reply to Patrick Brosset [:pbrosset] [:patrick] from comment #8)
> You should rather compare the splitter between version 29 (current release)
> and aurora (30), since this bug got fixed in 30. 

Correction: version 29 is currently the beta release.

> The bug doesn't exist in 28 since it came along with the huge devtools theme
> changes made in 29.
> 
> Having said that, I'm trying to replicate the bug in 29, but I'm not able to
> for some reasons.
> 
> Brian, do you remember how to replicate this issue so it can be verified?

This was a minor issue in which the splitter bled over a pixel or two and I think maybe covered up a pixel on the selected sidebar tab in the inspector.  But I'm looking at 29 on Beta and cannot see the issue.  The patch is not applied in this environment by checking chrome://browser/skin/devtools/light-theme.css and chrome://browser/skin/devtools/common.css.

It *was* a really minor issue, so maybe I'm just not seeing it right now.  The screenshot that was in Comment 1 had an arrow pointing to the exact problem if there is some way to recover it.
Flags: needinfo?(bgrinstead)
Attached image same-splitters.png
I've looked further into this - comparing the splitter in 29 vs 30 and I can't see any difference.  The only thing I can think is that a separate bug that got uplifted to 29 has fixed or masked this problem, even though this patch is only on 30.
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.