Closed
Bug 132184
Opened 24 years ago
Closed 24 years ago
Make Quick Search bar in AB look like Quick Search Bar in Mail
Categories
(SeaMonkey :: MailNews: Address Book & Contacts, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.0
People
(Reporter: jglick, Assigned: tuukka.tolvanen)
Details
Attachments
(2 files, 1 obsolete file)
|
40.34 KB,
image/gif
|
Details | |
|
1.25 KB,
patch
|
tuukka.tolvanen
:
review+
sspitzer
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
Example. Make AB border look like Mail Quick Search Bar border
Minor UI nit.
Add border around AB quick search bar to match Mail Quick Search bar.
See bug 106739.
Screenshot below.
Ben, would you be willing to fix this since you fixed bug 106739? Thanks.
|
||
Comment 2•24 years ago
|
||
Hi, I'll dig up my changes to bug 106739 and see if they work for the address
book. Unfortunately, my fix to bug 106739 ended up showing only some of the
time (bug 107571 ..never learned why), and I think bug 106739 was fixed for good
when a lot of the rest of the 3-pane was re-written. Maybe someone should ask
Seth about it.
|
Assignee | |
Comment 3•24 years ago
|
||
copy-paste patch */messenger/mailWindow1.css -> */messenger/addressbook.css
|
|
Assignee | |
Comment 4•24 years ago
|
||
sorry 'bout bustin' in, just happened to wander by when this was posted
Status: NEW → ASSIGNED
OS: Windows 2000 → All
|
|
Assignee | |
Updated•24 years ago
|
Status: NEW → ASSIGNED
Comment on attachment 75108 [details] [diff] [review]
patch
r=andreww providing you put the "::::: Search Bar ::::: " comment in the
classic addressbook.css as well for consistency.
Attachment #75108 -
Flags: review+
|
|
Assignee | |
Comment 7•24 years ago
|
||
with the search bar comment in classic too
Attachment #75108 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 8•24 years ago
|
||
Comment on attachment 75124 [details] [diff] [review]
patch v2
carry over r=
Attachment #75124 -
Flags: review+
|
|
||
Comment 9•24 years ago
|
||
I'll try out the patch, verify that it works, and drive it in.
|
|
||
Comment 10•24 years ago
|
||
Comment on attachment 75124 [details] [diff] [review]
patch v2
sr=sspitzer
Attachment #75124 -
Flags: superreview+
|
|
||
Comment 11•24 years ago
|
||
I'll get some driver approval, and land this.
thanks for the patch, Tuukka.
|
|
||
Updated•24 years ago
|
Target Milestone: --- → mozilla1.0
|
||
Updated•24 years ago
|
Attachment #75124 -
Flags: approval+
|
|
||
Comment 12•24 years ago
|
||
Comment on attachment 75124 [details] [diff] [review]
patch v2
a=asa (on behalf of drivers) for checkin to the 1.0 trunk
|
|
||
Comment 13•24 years ago
|
||
fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
|
||
Comment 14•24 years ago
|
||
Trunk build 2002-04-01: WinMe, Linux RH 7.1, Mac 10.1.3
Verified Fixed, Classic and Modern.
Status: RESOLVED → VERIFIED
|
||
Updated•21 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•