[Mac] Need new css for clearListButton

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
Themes
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: stefanh, Assigned: stefanh)

Tracking

Trunk
seamonkey2.1a1
x86
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

2.81 KB, patch
Karsten Düsterloh
: review+
neil@parkwaycc.co.uk
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

7 years ago
I forgot the button - same principle as the pageInfo css (all icons have been removed) in bug 525698.
(Assignee)

Comment 1

7 years ago
Created attachment 419260 [details] [diff] [review]
New css

I added Markus to the file since I rip most stuff from him.
Attachment #419260 - Flags: review?(mnyromyr)

Updated

7 years ago
Attachment #419260 - Flags: review?(mnyromyr) → review+
(Assignee)

Updated

7 years ago
Attachment #419260 - Flags: superreview?(neil)

Comment 2

7 years ago
Comment on attachment 419260 [details] [diff] [review]
New css

>+  background: -moz-linear-gradient(top, #FFF, #ADADAD) repeat-x;
Three nits, two of which you got right later on:
1. repeat-x is unnecessary (makes no sense for a gradient)
2. should be background-image (background is a shortcut for multiple styles)
3. I'd prefer six-digit colour codes i.e. #FFFFFF throughout.

>+  background: #B5B5B5;
Nit: background-color

sr=me with these fixed.
Attachment #419260 - Flags: superreview?(neil) → superreview+

Comment 3

7 years ago
(In reply to comment #2)
>(From update of attachment 419260 [details] [diff] [review])
>>+  background: -moz-linear-gradient(top, #FFF, #ADADAD) repeat-x;
>Three nits, two of which you got right later on:
>1. repeat-x is unnecessary (makes no sense for a gradient)
>2. should be background-image (background is a shortcut for multiple styles)
>3. I'd prefer six-digit colour codes i.e. #FFFFFF throughout.
> 
>>+  background: #B5B5B5;
>Nit: background-color
OK, so the second is supposed to override the first, and just to be difficult, one uses a solid colour and one uses a gradient. In that case, how about:
background-color: #B5B5B5;
background-image: -moz-linear-gradient(top, #FFFFFF, #ADADAD);
...
background-image: none;
(Assignee)

Comment 4

7 years ago
Landed with suggested changes:
http://hg.mozilla.org/comm-central/rev/a61f577ca2e4

Then I discovered tha I missed one nit, so I landed another changeset:
http://hg.mozilla.org/comm-central/rev/2c8fc8ea5c2f
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.