Closed
Bug 711206
Opened 10 years ago
Closed 9 years ago
The text "Enter Search or Address" should appear in the awesomebar on the start page AND the awesomescreen from the start page
Categories
(Firefox for Android Graveyard :: General, defect, P4)
Tracking
(firefox11 verified, firefox12 verified)
VERIFIED
FIXED
Firefox 12
People
(Reporter: mtrombley, Assigned: Margaret)
Details
(Whiteboard: [MTD] [user-testing] [polish])
Attachments
(1 file)
2.30 KB,
patch
|
mfinkle
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Web page or screen you were on when you saw the issue: About:home and awesomescreen Steps to reproduce: 1. Start at about:home. The awesomebar shows the string "Nightly Start" 2. Click in the awesomebar 3. The awesomescreen shows the string "about:home" 4. Both these strings do not give the user the idea that tapping in the awesomebar is the way to start a search. What you expected: Have the text "Enter Search or Address" in the field when a user is on the start page AND when they've tapped through to the awesomescreen from the start page. Crash report ID (if applicable):
Reporter | ||
Updated•10 years ago
|
Whiteboard: [MTD] → [MTD] [user-testing]
Updated•10 years ago
|
OS: Mac OS X → Android
Hardware: x86 → ARM
Updated•9 years ago
|
Assignee: nobody → madhava
Priority: -- → P4
Comment 1•9 years ago
|
||
Yes - agreed. Let's do this!
Comment 2•9 years ago
|
||
Oh - weird - it's showing the string "Enter Search or Address" for a bit when the page loads, and then it gets replaced with "Nightly Start." Is this because we usually show the URL until we get the title, and we've faked in that string for the URL?
Assignee: madhava → mark.finkle
Updated•9 years ago
|
Whiteboard: [MTD] [user-testing] → [MTD] [user-testing] [polish]
Assignee | ||
Comment 3•9 years ago
|
||
This patch special-cases about:home to keep the urlbar empty, which addresses Mary's concerns from comment 0.
Assignee: mark.finkle → margaret.leibovic
Attachment #589316 -
Flags: review?(mark.finkle)
Comment 4•9 years ago
|
||
I kinda disagree with #3 in comment 0. All webpages should have a URL. In this case, the text should be selected and ready to be replaced with whatever the user types. #1 is fine and we actually did that in XUL Fennec using the <title> of the about:home page itself - it was set to "Enter Search or Address" I think we should do the same thing here, reducing the amount of hardcoding special behavior for "about:home" - we have enough already.
Assignee | ||
Comment 5•9 years ago
|
||
Comment on attachment 589316 [details] [diff] [review] patch I'll write a new patch to address mfinkle's comments.
Attachment #589316 -
Attachment is obsolete: true
Attachment #589316 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 6•9 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #4) > #1 is fine and we actually did that in XUL Fennec using the <title> of the > about:home page itself - it was set to "Enter Search or Address" > > I think we should do the same thing here, reducing the amount of hardcoding > special behavior for "about:home" - we have enough already. Actually, this won't prevent the jankiness where we briefly show the "about:home" URL before we get the title of the page (we call setTitle when we first get the url, then we call it again when/if we get a title). Also, it will change from gray placeholder text to black title text. And also, we still keep about:home as a history entry, and if we change the title to "Enter Search or Address" that will show up in the awesomebar results, which seems bad. I agree with your point about all pages having a URL, and I know hard-coding behavior for about:home sucks, but it might be worth it in this place to stop the UI from jumping around. (So just the BrowserToolbar part of the patch.)
Comment 7•9 years ago
|
||
Comment on attachment 589316 [details] [diff] [review] patch OK, based on the situation, using the BrowserToolbar.java part of this patch seems like a good compromise. r+ with the AwesomeBar.java part removed.
Attachment #589316 -
Attachment is obsolete: false
Attachment #589316 -
Flags: review+
Assignee | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1b9c27fad1b9
tracking-fennec: --- → ?
Comment 9•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1b9c27fad1b9
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Assignee | ||
Comment 10•9 years ago
|
||
Comment on attachment 589316 [details] [diff] [review] patch [Approval Request Comment] Mobile-only. Fixes janky UI on startup.
Attachment #589316 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•9 years ago
|
status-firefox11:
--- → affected
status-firefox12:
--- → fixed
Comment 11•9 years ago
|
||
Comment on attachment 589316 [details] [diff] [review] patch [Triage Comment] Mobile only - approved for Aurora.
Attachment #589316 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 12•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/6fa1b61cf405
Comment 13•9 years ago
|
||
Verified on M-C Samsung Galaxy SII (Android 2.3.4) & Nexus S (Android 4.0.3) 20120119042014
Comment 14•9 years ago
|
||
Verified on M-A Galaxy Nexus (Android 4.0.3) 20120126053754 http://hg.mozilla.org/releases/mozilla-aurora/rev/9d5ede8cb11a
Updated•4 months ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•