Closed Bug 1008647 Opened 6 years ago Closed 6 years ago
Left padding of Label of BMB
_bookmarks Show All Top is wrong, separator isn't necessary
No description provided.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Summary: Left padding of Label of BMB_bookmarksShowAllTop is wrong on Windows7 Classic → Left padding of Label of BMB_bookmarksShowAllTop is wrong, separator isn't necessary
This is ugly, because I don't think this ever actually does have an icon, but it's the most contained and safely upliftable fix to this patch that I can see (as opposed to adding separate CSS rules on all the platforms) and brings the item in line with all the other non-checkbox items in that menu.
Attachment #8422346 - Flags: review?(mak77)
Gijs, please flag bugs that you start working on by needinfo'ing Marco or Jenn, and provide a point estimate (I've done that here)
Whiteboard: p=1 s=it-32c-31a-30b.2 [qa-]
(In reply to :Gavin Sharp (email email@example.com) from comment #2) > Gijs, please flag bugs that you start working on by needinfo'ing Marco or > Jenn, and provide a point estimate (I've done that here) Thanks. To clarify, this is basically a followup from bug 986866 and we need to uplift that (it's tracking 30), hence taking care of it.
Added to Iteration.
Comment on attachment 8422346 [details] [diff] [review] fix left padding and remove separator for top show all bookmarks item, https://hg.mozilla.org/integration/fx-team/rev/39e1fdf059c5
Attachment #8422346 - Flags: checkin+
Fixed on branches by the branch patch in bug 986866
You need to log in before you can comment on or make changes to this bug.