Closed Bug 405892 Opened 15 years ago Closed 15 years ago

Finish polish of the search field

Categories

(Toolkit :: Downloads API, enhancement, P4)

enhancement

Tracking

()

VERIFIED FIXED
mozilla1.9.1a2

People

(Reporter: madhava, Assigned: dao)

References

Details

(Keywords: polish)

Attachments

(1 file, 1 obsolete file)

[actually nominating for wanted]

As shown in this mockup (attached to bug 397655) :
https://bugzilla.mozilla.org/attachment.cgi?id=288692

We should add the magnifying glass icon (appropriate per platform) that we use in all search fields in Firefox.
Flags: blocking-firefox3?
We need a global searchbar style for every search widget, and to just use that where appropriate.
Flags: wanted-firefox3+
Flags: blocking-firefox3?
Flags: blocking-firefox3-
Priority: -- → P4
Target Milestone: --- → Firefox 3 M11
Would it be bad to just reuse chrome://browser/skin/Search.png and related styles?

Is there an issue of download manager being in toolkit?

madhava: Is the endcap supposed to be round on os x (basically copy what the search engine bar looks like)?

If so, Dão, the search box can't be just a plain textbox with an image tacked on at the end? (Even if not, what about rtl issues?)
(In reply to comment #2)
> Would it be bad to just reuse chrome://browser/skin/Search.png and related
> styles?
>
> Is there an issue of download manager being in toolkit?

Yes. What we can do is move the image to /toolkit and update the references in /browser.

> If so, Dão, the search box can't be just a plain textbox with an image tacked
> on at the end? (Even if not, what about rtl issues?)

We should just use the upcoming search textbox widget.
Depends on: 388811
Yeah, it ought to match OS X style on mac, so we should use the search textbox widget.
So OS X has a search icon, rounded search box, text "Search...", and it's right aligned in the window.

Should other platforms get some/all of this treatment?

Assuming we won't be able to use the search textbox widget..
On OSX round=search, yes, and it should get that view. I'm sad we won't be able to use the search textbox widget, though. A boo hoo hoo.
Are we not getting the textbox widget anymore?
Attached patch patch (obsolete) — Splinter Review
this builds upon the patch in bug 388811
Assignee: nobody → dao
Status: NEW → ASSIGNED
Do we need to remove any styles for this?  Do our unit tests still pass?
pinstripe should be cleaned up. The tests should pass since the search box comes from a timed textbox, although I haven't yet looked at your tests in detail.
Attachment #308928 - Attachment is obsolete: true
Shawn is probably referring to this testcase in particular:

http://mxr.mozilla.org/seamonkey/source/toolkit/mozapps/downloads/tests/browser/browser_bug_416303.js

It makes sure hitting esc in the input clears the search and moves focus to the list. Hitting esc from the list closes the window.
Yes, I knew about these tests theoretically. The second test script that interacts with the search field is browser_multiword_search.js. Both will pass if bug 388811 gets fixed as expected.
Comment on attachment 308938 [details] [diff] [review]
patch, including pinstripe cleanup

r=sdwilsh, but you need to cleanup winstripe and gnomestripe as well.
Attachment #308938 - Flags: review+
winstripe and gnomestripe don't seem to provide a special styling for the search field. Am I missing something?
Keywords: polish
Target Milestone: Firefox 3 beta3 → Firefox 3 beta5
Comment on attachment 308938 [details] [diff] [review]
patch, including pinstripe cleanup

hehe, I thought they did, but upon further investigation I see I was wrong.  r=sdwilsh as the patch stands.
Attachment #308938 - Flags: approval1.9?
Comment on attachment 308938 [details] [diff] [review]
patch, including pinstripe cleanup

a1.9+=damons
Attachment #308938 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
please don't check this in before bug 388811.
Keywords: checkin-needed
No longer blocks: 397655
Target Milestone: Firefox 3 beta5 → ---
Attachment #308938 - Flags: approval1.9+
Product: Firefox → Toolkit
http://hg.mozilla.org/index.cgi/mozilla-central/rev/4cd7a909b6f7
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a2
Mmh why the magnifying icon is still on the left side? Each search field shows this icon on the right side. Is this intended or shall I file a new bug about that issue?
(In reply to comment #20)
> Mmh why the magnifying icon is still on the left side?

That is taken over from pinstripe's styling for the search fields in the bookmarks and history sidebar.
(In reply to comment #21)
> That is taken over from pinstripe's styling for the search fields in the
> bookmarks and history sidebar.

I filed bug 449465.
Blocks: 449045
Dao, when hitting ESC within the web search field the content doesn't get cleared like it happens for all the other search textbox fields. Is this instance different? If its not intended we should reopen the bug and fix the remaining issue.
Which web search field?
"Tools | Web Search" or simply the one near the location bar e.g the Google search.
So which bug would you want to reopen? This one is about the download manager.

FYI: The searchbar has its own binding. I don't think we want to use the new search textbox widget from toolkit there, since you could only clear the textbox but not the results.
Thanks for the enlightenment.

Verified this replacement with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1a2pre) Gecko/20080808120213 Minefield/3.1a2pre ID:20080808120213

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1a2pre) Gecko/20080808120213 Minefield/3.1a2pre ID:20080808120213 
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.