667456 - [meta] [10.7] Update Firefox Theme for Mac OS X 10.7 Lion

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Dominic Spitaler]

Attachment: Mockup of OS X Theme Refresh for Lion

The release of OS X Lion is nearing and running Firefox on it works well, but it’s default theme hasn’t been updated yet to incorporate all the visual loveliness introduced in this OS update. 

So this bug is for tracking all the theme changes that are necessary to feel native again. Above you’ll find a mockup, showing what I think is necessary to be done.

Comment 1

[Markus Stange [:mstange]]

Awesome, thanks for doing this!

(I haven't done any work on this yet because I don't have access to a Lion preview and because I'm short on time currently anyways.)

Comment 2

[Nomis101]

(In reply to comment #0)
> Above you’ll find a mockup, showing what I think is necessary
> to be done.

You nice mockup also shows the fullscreen arrows in the right upper edge, which is necessary for a 10.7 look, but doesn't mention it. Is this Bug 639705 or does Bug 639705 only implement the support and the UI thing needs to be done in another bug?

Comment 3

[Markus Stange [:mstange]]

(In reply to comment #2)
Bug 639705 will also add the fullscreen arrows. They're provided automatically.

Comment 4

[Henri Sivonen (:hsivonen)]

Don't we want to keep the round Back button on Lion? It's a distinctive part of the UI branding that distinguishes Firefox from Safari at a glance.

Comment 5

[Dão Gottwald [:dao]]

The mockup shows small-icons mode.

Comment 6

[d]

The mockups are shown with the bookmarks-bar disabled. Is there any change wanted on it for Lion? Smoother/lighter folder icons? Or no changes at all, perhaps?

Comment 7

[Oz]

Looks amazing!

Comment 8

[Jackson Fox]

Looking good, but take a look at the split buttons in Lion. The divider no longer spans the height of the button, but instead stops several pixels short on either end (screenshot: http://cl.ly/0J1c3P3Z1U0N200i1V2r).

Comment 9

[Markus Stange [:mstange]]

Jackson: That's how we implemented split buttons on Lion. The only exception that has a connecting separator is the keyhole-style back/forward button.

Comment 10

[Florian Bender]

Bug 690910 and Bug 766106 (and maybe Bug 688678) are probably obsolete with Australis, can you please WONTFIX them?