Closed Bug 335822 Opened 14 years ago Closed 14 years ago
Deleting all search engines leaves useless separator in search box dropdown, search bar is useless
When all engines are removed using the search manager, the search bar becomes dead UI, and the dropdown contains a useless seperator. Perhaps "Manage Search Engines" should become "Add Engines" in that case, with the extra seperator removed.
The Search button should also become disabled in that case.
Hmm, kinda edge-to-cornerish, but I'll bite for now.
Flags: blocking-firefox2? → blocking-firefox2+
Assignee: nobody → michael.wu
Status: ASSIGNED → NEW
This eliminates the separator, disables the textbox, makes the search button open manage search engines dialog, and puts the message "No Search Engines Installed" in the textbox when there are no search engines. I don't replace manage engines with add engines because when the button to restore default search engines is added to the manage engines dialog, the user will need to add an engine in order to restore default search engines.
Instead of trying to make the search bar useful when there are no engines installed, I think we should just make it impossible to remove all the engines. I can't think of a good reason why someone would want to do that, they can just remove the search bar if they don't want to use it. That's a simpler fix, too: just disable the "remove" button in the engine manager when there's only one engine left.
Yeah, I'll go with that instead!
Comment on attachment 225792 [details] [diff] [review] Make search box act better without any search engines I'm with gavin and mconnor :)
Attachment #225792 - Flags: ui-review?(beltzner) → ui-review-
Whatever. As long as I get this bug off my list. :p
Attachment #225883 - Flags: review?(gavin.sharp) → review+
Attachment #225883 - Flags: approval-branch-1.8.1?(mconnor)
Attachment #225883 - Flags: approval-branch-1.8.1?(mconnor) → approval-branch-1.8.1+
Summary: Deleting all search engines leaves useless seperator in search box dropdown, search bar is useless → Deleting all search engines leaves useless separator in search box dropdown, search bar is useless
Patch checked in on branch and trunk.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.