Feeling Lucky
?
[#7607] - Admin autofocus login
- Closed
- 2 Aug 2015
- Medium
- Build: staging
- # 7607
- Diff
- brianteeman:admin-login
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This PR removes the inline javascript from the ISIS and Hathor templates which adds focus to the username field on the admin login screen and just uses autofocus=true instead
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Administration JavaScript Templates (admin) |
mbabker
- comment
- 1 Aug 2015
Our templates have a polyfill for backporting stuff. So as long as it's covered then no big deal. And if the browser ignores it completely and the page still works, I'd say no big deal either.
Fedik
- comment
- 1 Aug 2015
works good 
dgt41
- comment
- 1 Aug 2015
works for me as well
jwaisner
- comment
- 2 Aug 2015
PR works as intended.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7607.
| Status | Pending | ⇒ | Ready to Commit |
zero-24
- comment
- 2 Aug 2015
RTC
| Labels |
Added:
?
|
||
Bakual
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-08-02 14:37:28 |
| Closed_By | ⇒ | Bakual |
| Milestone |
Added: |
||
| Labels |
Removed:
?
|
||
I believe you can just add the word
autofocusto the input tags. No need forautofocus="true"according to w3schools. I had to addautofocus="true"in #7604 because it was being added to an XML file and not the input tag directly.One thought about doing this HTML5 attribute is the fact that it is not supported in all browsers, specifically anything lower than IE10. Does that matter?