Closed
Bug 129946
Opened 22 years ago
Closed 20 years ago
clicking Clear button should disable the Fewer button
Categories
(SeaMonkey :: MailNews: Address Book & Contacts, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: bugzilla, Assigned: standard8)
Details
Attachments
(1 file)
980 bytes,
patch
|
dmosedale
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
Beside clicking the Clear button is sooooo slow if you pressed the More button 10 times, the the Fewer button should become disabled when you click the clear button. 20020309
Updated•20 years ago
|
Product: Browser → Seamonkey
Assignee | ||
Comment 2•20 years ago
|
||
This is a one line patch that disables the less button after the clear is complete. Tried refactoring the code that clears the search list down, but that didn't improve speed. The only way would be to improve the listbox xul element might be to allow a remove all rows function. Do we want to be able to do that? If so I'd suggest doing it in a seperate bug.
Assignee | ||
Updated•20 years ago
|
Attachment #167369 -
Flags: review?(dmose)
Comment 3•20 years ago
|
||
Comment on attachment 167369 [details] [diff] [review] Simple Patch to disable less button after clear r=dmose
Attachment #167369 -
Flags: review?(dmose) → review+
Assignee | ||
Updated•20 years ago
|
Attachment #167369 -
Flags: superreview?(bienvenu)
Updated•20 years ago
|
Attachment #167369 -
Flags: superreview?(bienvenu) → superreview+
Assignee | ||
Comment 4•20 years ago
|
||
Fix checked in to trunk, marking fixed. 2004-12-01 14:26 timeless%mozdev.org mozilla/ mailnews/ base/ search/ resources/ content/ searchTermOverlay.js 1.38 3/0
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment 5•19 years ago
|
||
My finding is that "Clear" removes all additional rows, leaving only the 1st default, original row intact with its "+" button. The "-" is disabled. Your 1-line patch was removed in bug 195224. Anyway, it's good behavior, verified FIXED with build 2005-07-10-05, Windows XP SeaMonkey trunk.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•