Closed
Bug 689051
Opened 13 years ago
Closed 13 years ago
drop the drop shadow on the back button for Windows 7
Categories
(Firefox :: Theme, enhancement)
Tracking
()
RESOLVED
FIXED
Firefox 9
People
(Reporter: asa, Assigned: dao)
References
Details
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
shorlander
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #603790 +++
We removed the drop shadows from most toolbar buttons in bug 603790. There's still a slight shadow effect created by having the bottom borders of the buttons be darker than the top borders. This is nice and looks great.
The one button that didn't get this same treatment is the large circular back button. That button still has a slight drop shadow.
That button is unique in our toolbar in that it's large and round and sort of sits "on top" of the forward button. It might make some "physical" sense for the back button to feel raised up off of the toolbar but I don't think it makes for an ideal visual sense.
I propose that we fully drop the drop shadow on the back button and rely on the top and bottom border color differences to give it all the raised quality it needs.
I've attached a screenshot of the current button and the minor modification to it I've made of removing the drop shadow. You can see the two side-by-side at regular size and magnified to see the differences. It's subtle, but I think it's a win.
Comment 1•13 years ago
|
||
If you ask me, they should be completely flat. But aside that, go for it!
Assignee | ||
Comment 2•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Updated•13 years ago
|
Attachment #562364 -
Flags: review?(shorlander) → review+
Assignee | ||
Comment 3•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Comment 4•13 years ago
|
||
Dão backed this out because it had a typo that broke the world.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: Firefox 9 → ---
Comment 5•13 years ago
|
||
(In reply to Joe Drew (:JOEDREW!) from comment #4)
> Dão backed this out because it had a typo that broke the world.
That was bug 687754 wasn't it?
Comment 6•13 years ago
|
||
Yes it was.
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Comment 7•13 years ago
|
||
Whoops! Sorry.
You need to log in
before you can comment on or make changes to this bug.
Description
•