The default bug view has changed. See this FAQ.

Bugzilla login field should use "placeholder" HTML5 attribute instead of JavaScript



User Interface
4 years ago
4 years ago


(Reporter: glob, Assigned: glob)





(1 attachment)



4 years ago
+++ 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.

Comment 2

4 years ago
Created attachment 769578 [details] [diff] [review]
patch v1

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 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+

Comment 4

4 years ago
Committing to: bzr+ssh://
modified js/global.js
modified template/en/default/index.html.tmpl
modified template/en/default/account/auth/login-small.html.tmpl
Committed revision 8865.
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.