Closed
Bug 627324
Opened 14 years ago
Closed 14 years ago
Bottom toolbox border interrupts shadow on the sides
Categories
(Firefox :: Theme, defect)
Tracking
()
RESOLVED
FIXED
Firefox 4.0b11
People
(Reporter: dao, Assigned: dao)
References
Details
(Keywords: polish)
Attachments
(1 file, 1 obsolete file)
|
2.90 KB,
patch
|
fryn
:
review+
Dolske
:
approval2.0+
|
Details | Diff | Splinter Review |
|
Assignee | |
Comment 1•14 years ago
|
||
|
||
Comment 2•14 years ago
|
||
Comment on attachment 506106 [details] [diff] [review]
patch
Code and appearance look good.
> + background: ThreeDShadow;
Nit: I think this should be |background-color: ThreeDShadow;|.
r+ with that addressed.
Attachment #506106 -
Flags: review?(fryn) → review+
|
|
Assignee | |
Comment 3•14 years ago
|
||
Attachment #506106 -
Attachment is obsolete: true
Attachment #506117 -
Flags: approval2.0?
|
|
||
Updated•14 years ago
|
Attachment #506117 -
Flags: review+
|
||
Updated•14 years ago
|
Attachment #506117 -
Flags: approval2.0? → approval2.0+
|
|
||
Comment 4•14 years ago
|
||
Pushed.
http://hg.mozilla.org/mozilla-central/rev/cd10b3f59da4
Dão, is bug 626351 fixed by this too?
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b11
|
|
Assignee | |
Comment 5•14 years ago
|
||
(In reply to comment #4)
> Pushed.
>
> http://hg.mozilla.org/mozilla-central/rev/cd10b3f59da4
>
> Dão, is bug 626351 fixed by this too?
I think so.
You need to log in
before you can comment on or make changes to this bug.
Description
•