Closed
Bug 593838
Opened 14 years ago
Closed 14 years ago
unsortedBookmarks.png ought to have equal amount of transparency at top/bottom
Categories
(SeaMonkey :: Themes, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.1b1
People
(Reporter: stefanh, Assigned: stefanh)
References
Details
Attachments
(1 file)
2.38 KB,
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
While working on bug 586026, I noticed that the unsortedBookmarks.png icon is not vertically centered - it has all its transparency at the top. I think it will look a lot better with equal amounts of transparency at top/bottom - especially in the file bookmark menulist.
Assignee | ||
Updated•14 years ago
|
Summary: UnsortedBookmarks.png ought to have equal amount of transparency at top/bottom → unsortedBookmarks.png ought to have equal amount of transparency at top/bottom
Assignee | ||
Comment 1•14 years ago
|
||
Hmm, I will file a separate bug for removing some data chunks from all our icons
Attachment #472426 -
Flags: superreview?(neil)
Attachment #472426 -
Flags: review?(kairo)
Comment 2•14 years ago
|
||
Comment on attachment 472426 [details] [diff] [review] Adjust icon I'll leave that to Neil - I thought it was reasonable to have the bucket the bookmarks land in sit at the bottom border, but if centering the image vertically looks better for you guys, let's do it.
Attachment #472426 -
Flags: review?(kairo) → review?(neil)
Assignee | ||
Comment 3•14 years ago
|
||
It looks a bit bad on mac, if you think it's fine on win/nix I can fork the icon - but I checked some other win/nix icons (that appear in the menulist) and they seem to have transparency at the bottom.
Updated•14 years ago
|
Attachment #472426 -
Flags: superreview?(neil)
Attachment #472426 -
Flags: superreview+
Attachment #472426 -
Flags: review?(neil)
Attachment #472426 -
Flags: review+
Assignee | ||
Comment 4•14 years ago
|
||
http://hg.mozilla.org/comm-central/rev/7fbe67b91630
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b1
You need to log in
before you can comment on or make changes to this bug.
Description
•