Closed
Bug 468658
Opened 16 years ago
Closed 15 years ago
show "no match" text when there are no autocomplete results
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(Not tracked)
VERIFIED
FIXED
fennec1.0b2
People
(Reporter: Gavin, Assigned: Gavin)
References
Details
Attachments
(1 file)
6.55 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
Per bug 460100 comment 4. > Yeah, let's make sure we put in a "No matches" message if there are no matches. > Horizontally entered and on row two or three, vertically.
Assignee | ||
Comment 1•15 years ago
|
||
I'll file a bug for the getElementsByTagName failure I was seeing (though it might be a dupe). Also fixes bug 486656.
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #379603 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 2•15 years ago
|
||
-had to add the label="" for the label to take space layout-wise (hope it doesn't affect perf!) -used attribute/dtd rather than a stringbundle to avoid the overhead (DTD was already being loaded) -got ui-r from madhava in-person on the "no results" styling
Comment 3•15 years ago
|
||
Comment on attachment 379603 [details] [diff] [review] patch italic? :)
Attachment #379603 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 4•15 years ago
|
||
Beltzner suggested that, actually, to better differentiate it from a normal result, and madhava and I are not italic-haters. https://hg.mozilla.org/mobile-browser/rev/632e7d859cbc
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•15 years ago
|
Target Milestone: --- → B2
Comment 5•15 years ago
|
||
verified FIXED On builds: Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.3a1pre) Gecko/20090818 Fennec/1.0a3pre and Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20090821 Fennec/1.0b3pre and Mozilla/5.0 (Macintosh; U; Intel Mac OSX 10.5; en-US; rv:1.9.2a2pre) Gecko/20090808 Fennec/1.0b3pre
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•