Closed Bug 747140 Opened 13 years ago Closed 11 years ago

Apply the Australis toolbar button design to the zoom-controls on Windows

Categories

(Firefox :: Theme, defect)

14 Branch
x86_64
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 924933

People

(Reporter: jaws, Assigned: jaws)

References

Details

(Whiteboard: [Australis:M?][Australis:P4])

Attachments

(1 file)

Attached patch Patch for bug (deleted) — Splinter Review
Bug 734373 will implement the Australis styling for the toolbar buttons on Windows. This bug is a follow-up for applying the styling to the zoom-controls. I will wait for requesting review until bug 734373 is fixed.
(In reply to Jared Wein [:jaws] from comment #0) > Created attachment 616721 [details] [diff] [review] > Patch for bug > > Bug 734373 will implement the Australis styling for the toolbar buttons on > Windows. This should be cross platform
Summary: Apply the Australis toolbar button design to the zoom-controls → Apply the Australis toolbar button design to the zoom-controls on Windows
That may have been resolved in bug 870897.
(In reply to Guillaume C. [:ge3k0s] from comment #2) > That may have been resolved in bug 870897. Apparently not so it still needs to be done.
Whiteboard: [Australis:M?]
Year old Australis bug? Is this still relevant?
Flags: needinfo?(jaws)
Yeah, it's still relevant. The zoom-controls should appear joined, with a splitter between them. This would be similar to how we style the notification buttons for the Social API. Hovering one of the zoom-control buttons should put an outline around the other button too, indicating that there is a relationship between the two.
Flags: needinfo?(jaws)
Ah, I guess this is still relevant for when the zoom control is placed in the navbar. Seems like a somewhat less common customization, so P4.
Flags: needinfo?(jaws)
Whiteboard: [Australis:M?] → [Australis:M?][Australis:P4]
Flags: needinfo?(jaws)
Assignee: soapyhamhocks → nobody
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Duping forward since the other bug has a further along patch that will fix this.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: