Closed Bug 686778 Opened 13 years ago Closed 13 years ago

Web console: Move close button to the left on Mac

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 9

People

(Reporter: jaws, Assigned: jaws)

References

Details

(Whiteboard: [fixed-in-fx-team])

Attachments

(2 files, 1 obsolete file)

Attached patch Patch for bug (obsolete) — Splinter Review
Bug 684561 moved the close button to the right on Windows but accidentally moved the close button to the right on Mac as well. Also some toolbarbutton margins that were increased on winstripe are also needed on gnomestripe.
Attachment #560252 - Flags: review?(dcamp)
Assignee: nobody → jwein
Status: NEW → ASSIGNED
Attachment #560252 - Flags: review?(dcamp)
Attachment #560252 - Flags: review?(dao)
Attachment #560252 - Flags: review+
Attached image pre-patch screenshot
Here's what the buttons look like for me on Linux without this patch. This looks ok to me, I don't think more space around them is needed.
Removed the style changes for gnomestripe since it wasn't too terrible looking on gnomestripe pre-patch.

HUDService.jsm was r+ from dcamp whereas webconsole.css review was forwarded to dao. Therefore, I'll carry forward r+ from dcamp.
Attachment #560252 - Attachment is obsolete: true
Attachment #560252 - Flags: review?(dao)
Attachment #560825 - Flags: review+
Summary: Web console: Move close button to the left on Mac and carry over style changes from Windows to Linux → Web console: Move close button to the left on Mac
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/4866693ce45c
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: