Closed Bug 1115374 Opened 9 years ago Closed 9 years ago

DevEdition theme - Inconsistent borders on toolbars

Categories

(Firefox :: Theme, defect)

All
Windows 10
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 38

People

(Reporter: ntim, Assigned: bgrins)

References

Details

(Whiteboard: [devedition-polish][testday-20150901])

Attachments

(1 file, 1 obsolete file)

See attachment 8536605 [details], the navbar doesn't borders on the sides, while the bookmarks toolbar does have some. This makes the light theme look unpolished.
Whiteboard: [devedition-polish]
It appears in the screenshot that you have the patch from Bug 1093820 applied.  Do you see this problem even without that?
(In reply to Brian Grinstead [:bgrins] from comment #1)
> It appears in the screenshot that you have the patch from Bug 1093820
> applied.  Do you see this problem even without that?

Yes. I see that without the patch.
Attached patch toolbar-border-windows.patch (obsolete) — Splinter Review
Tim, does it look better with this patch applied?
Attachment #8549144 - Flags: feedback?(ntim007)
Comment on attachment 8549144 [details] [diff] [review]
toolbar-border-windows.patch

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

Fixes the issue for me.
Attachment #8549144 - Flags: feedback?(ntim007) → feedback+
Comment on attachment 8549144 [details] [diff] [review]
toolbar-border-windows.patch

Checked on OSX and Ubuntu and this Windows fix doesn't seem to cause any changes, so requesting review
Attachment #8549144 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8549144 [details] [diff] [review]
toolbar-border-windows.patch

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

This was annoying me the other day on win8. Thanks!
Attachment #8549144 - Flags: review?(gijskruitbosch+bugs) → review+
Adds commit message
Assignee: nobody → bgrinstead
Attachment #8549144 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8549190 - Flags: review+
Hi, seems this doesn't apply cleanly:

patching file browser/themes/shared/devedition.inc.css
Hunk #2 FAILED at 221
1 out of 2 hunks FAILED -- saving rejects to file browser/themes/shared/devedition.inc.css.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working dir
errors during apply, please fix and refresh toolbar-border-windows.patch

could you take a look, thanks!
Flags: needinfo?(bgrinstead)
Keywords: checkin-needed
Rebased and pushed to fx-team:

https://hg.mozilla.org/integration/fx-team/rev/2cc184d6dc0f
Flags: needinfo?(bgrinstead)
Whiteboard: [devedition-polish] → [fixed-in-fx-team][devedition-polish]
https://hg.mozilla.org/mozilla-central/rev/2cc184d6dc0f
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team][devedition-polish] → [devedition-polish]
Target Milestone: --- → Firefox 38
This has been verified fixed for a QA test day.
Status: RESOLVED → VERIFIED
Whiteboard: [devedition-polish] → [devedition-polish][testday-20150901]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: