Closed
Bug 888806
Opened 11 years ago
Closed 11 years ago
Bugzilla login field should use "placeholder" HTML5 attribute instead of JavaScript
Categories
(bugzilla.mozilla.org :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: glob, Assigned: glob)
Details
Attachments
(1 file)
7.79 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #888731 +++ The Bugzilla login form at the bottom of the main bugzilla page seems to use the init_mini_login_form script to pre-file the input form to display "login" (on bugzilla trunk) as gray text in the background. This is bad as this breaks Firefox autocomplete of saved passwords, at least in recent nightly builds (see Bug 886720 Comment 16). Firefox devs are not sure why this worked fine so far. Bugzilla could use the "placeholder" HTML5 attribute there to display the gray text as placeholder in the text field. Then FF autocomplete should work fine. in the upstream bug lpsolit stated this isn't desirable as bugzilla uses html 4.01 not html 5. however for bmo i think it's appropriate to fix the problem by using the placeholder attribute.
i also fixed the quicksearch placeholder mechanism (which, oddly enough, doesn't use the same timing-sensitive timeouts as the login form does). this also (accidentally!) fixes bug 790644.
Attachment #769578 -
Flags: review?(dkl)
Comment 3•11 years ago
|
||
Comment on attachment 769578 [details] [diff] [review] patch v1 Review of attachment 769578 [details] [diff] [review]: ----------------------------------------------------------------- Spot on as usual. Look good and works as expected. r=dkl
Attachment #769578 -
Flags: review?(dkl) → review+
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.2/ modified js/global.js modified template/en/default/index.html.tmpl modified template/en/default/account/auth/login-small.html.tmpl Committed revision 8865.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•