Closed Bug 977592 Opened 8 years ago Closed 8 years ago

HTTP auth dialog puts focus in password field by default

Categories

(Firefox for Android Graveyard :: General, defect)

30 Branch
All
Android
defect
Not set
normal

Tracking

(firefox30 affected, fennec30+)

VERIFIED FIXED
Firefox 30
Tracking Status
firefox30 --- affected
fennec 30+ ---

People

(Reporter: kats, Assigned: wesj)

References

Details

(Keywords: regression, reproducible)

Attachments

(1 file)

As of not too long ago (sometime in the last month, I think) I've noticed that when I go to pages which force a HTTP auth dialog, the focus is in the password field rather than the username field by default. At least on my Galaxy Q running 2.3.

STR:
1. Go to a page with HTTP auth (e.g. https://staktrace.com/apps/bugmash/dashboard.php)
2. Wait for the HTTP auth dialog to pop up

Expected: focus is in the username field
Actual: focus is in the password field
I see this too on my Nexus 4 (4.4.2).

Last good revision: 660b62608951 (2014-02-20)
First bad revision: 7010ab83a06e (2014-02-21)
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=660b62608951&tochange=7010ab83a06e


a10bb6a9da93	Mark Capella — Bug 927882 - Standardize TextSelection caret positioning logic for soft keyboards, r=jchen, wesj, f=margaret
tracking-fennec: --- → ?
Flags: needinfo?(markcapella)
Keywords: reproducible
Assignee: nobody → wjohnston
tracking-fennec: ? → 30+
547342061dc3	Wes Johnston — Bug 931900 - Make null value parameter handling match Desktop. r=margaret
Blocks: 931900
Flags: needinfo?(markcapella)
I can repro the situation as stated, but I can't think of any way my patch(s) would affect positioning of the blinking IME caret's field position, (vs positioning of the orange TextSelection UI caret).

(Fyi the issue doesn't seem to exist on desktop).

For good measure, I backed out my changes and re-tested and the issue still exists.
Yes, why I cancelled your need-info.
Attached patch PatchSplinter Review
I just flipped this when we recently changed things.
Attachment #8383905 - Flags: review?(margaret.leibovic)
Attachment #8383905 - Flags: review?(margaret.leibovic) → review+
https://hg.mozilla.org/mozilla-central/rev/9f335dcd5db1
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.