Moodle
  1. Moodle
  2. MDL-23344

Login page does not focus on the login form inputs after the page loads

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1
    • Fix Version/s: 2.1.1, 2.2
    • Component/s: Themes
    • Labels:
    • Rank:
      6145

      Description

      When loading Moodle 2.0 and you are prompted with the login page, the form is not automatically focused, so one has to either tab to the username/password inputs or click into them with your mouse or other input device. I think having the focusing behavior by default is good and expected, because I believe this was also the default in 1.9 and previous. If it wasn't, then maybe it's a good time to make it the default as well.

        Issue Links

          Activity

          Hide
          Patrick Malley added a comment -

          Probably an improvement more than a bug, but something to consider for usability nonetheless. Any thoughts, Sam?

          Show
          Patrick Malley added a comment - Probably an improvement more than a bug, but something to consider for usability nonetheless. Any thoughts, Sam?
          Hide
          Petr Škoda added a comment -

          I did my little research - on single purpose login pages (such as our login/index.php) the username and password fields are focused automatically. If username is pre-filled password is selected, if not username is selected.

          I am going to implement something similar on top of MDL-28158

          Show
          Petr Škoda added a comment - I did my little research - on single purpose login pages (such as our login/index.php) the username and password fields are focused automatically. If username is pre-filled password is selected, if not username is selected. I am going to implement something similar on top of MDL-28158
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated thanks! (master and 21)

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated thanks! (master and 21)
          Hide
          Eloy Lafuente (stronk7) added a comment -

          (unsetting me as tester, sorry, pressed the wrong button)

          Show
          Eloy Lafuente (stronk7) added a comment - (unsetting me as tester, sorry, pressed the wrong button)
          Hide
          Rossiani Wijaya added a comment -

          Hi everyone,

          Just noticed this issue as I'm about to test it.

          Auto-focused was on purposely removed from Moodle 2.0 in order to support accessibility (MDL-20410). With auto focus enabled, screen reader tends to jump directly to the focus field information and skip all the information above that. This could create confusion for vision impaired user.

          Show
          Rossiani Wijaya added a comment - Hi everyone, Just noticed this issue as I'm about to test it. Auto-focused was on purposely removed from Moodle 2.0 in order to support accessibility ( MDL-20410 ). With auto focus enabled, screen reader tends to jump directly to the focus field information and skip all the information above that. This could create confusion for vision impaired user.
          Hide
          Sam Hemelryk added a comment -

          Hi Rosie, just moving this to testing in progress so that no one takes it off you good spotting by the way I remember when that occurred.

          Cheers
          Sam

          Show
          Sam Hemelryk added a comment - Hi Rosie, just moving this to testing in progress so that no one takes it off you good spotting by the way I remember when that occurred. Cheers Sam
          Hide
          Petr Škoda added a comment -

          All sites I verified do autofocus on login forms, why should we not do that? The missing focus is extremely annoying for most people.

          If this is a problem for screen reader software they should fix it instead.

          Show
          Petr Škoda added a comment - All sites I verified do autofocus on login forms, why should we not do that? The missing focus is extremely annoying for most people. If this is a problem for screen reader software they should fix it instead.
          Hide
          Petr Škoda added a comment - - edited

          We do not support permanent logins - this is another reason to make logging-in as easy as possible because you need to repeat it several times each day - if you store password in browser all you need is to press Enter (or type password and press enter).

          Show
          Petr Škoda added a comment - - edited We do not support permanent logins - this is another reason to make logging-in as easy as possible because you need to repeat it several times each day - if you store password in browser all you need is to press Enter (or type password and press enter).
          Hide
          Eloy Lafuente (stronk7) added a comment -

          My own +1 for this

          Show
          Eloy Lafuente (stronk7) added a comment - My own +1 for this
          Hide
          moodle.com added a comment -

          We can't keep putting this in and out.

          Show
          moodle.com added a comment - We can't keep putting this in and out.
          Hide
          Michael de Raadt added a comment -

          This needs to be reverted. -100

          Show
          Michael de Raadt added a comment - This needs to be reverted. -100
          Show
          moodle.com added a comment - http://tracker.moodle.org/browse/MDL-20410?focusedCommentId=86406&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-86406
          Hide
          David Mudrak added a comment - - edited

          Can we implement this as configurable per-site behaviour? Many schools that do not have any users with screen readers would probably turn it on. Particularly, at most of the typical Czech elementary and high schools there are no visually impaired users with screen readers. Such sites would definitely benefit of this enable. We can even have a special section in the admin tree called "Accessibility" that would contain this option (among others possibly). This way, only those who really need to disable the focusing would not block this nice feature from those who want it.

          p.s. To be honest, the described behaviour of screen readers really sounds as their bug but we probably can't do much of it.

          Show
          David Mudrak added a comment - - edited Can we implement this as configurable per-site behaviour? Many schools that do not have any users with screen readers would probably turn it on. Particularly, at most of the typical Czech elementary and high schools there are no visually impaired users with screen readers. Such sites would definitely benefit of this enable. We can even have a special section in the admin tree called "Accessibility" that would contain this option (among others possibly). This way, only those who really need to disable the focusing would not block this nice feature from those who want it. p.s. To be honest, the described behaviour of screen readers really sounds as their bug but we probably can't do much of it.
          Hide
          Petr Škoda added a comment - - edited

          Could we at least let administrators decide what is good for their users instead of letting one accessibility report decide what is good for the whole universe? (add new admin option)

          Show
          Petr Škoda added a comment - - edited Could we at least let administrators decide what is good for their users instead of letting one accessibility report decide what is good for the whole universe? (add new admin option)
          Hide
          Aparup Banerjee added a comment -

          just adding 2 cents after it was mentioned in the scrum: my +1 to accessibility options page. The defaults should cater to accessibility as priority though.

          Show
          Aparup Banerjee added a comment - just adding 2 cents after it was mentioned in the scrum: my +1 to accessibility options page. The defaults should cater to accessibility as priority though.
          Hide
          Sam Hemelryk added a comment -

          Raj brought up the idea of using a config option here at the office at the scrum meeting today.
          Would it be overkill to do that or do you think it would be justified?
          My -1 for this to go in however a setting may be the best way to keep the peace.
          My first impression was that it was overkill... but peace sounds pretty good.

          Show
          Sam Hemelryk added a comment - Raj brought up the idea of using a config option here at the office at the scrum meeting today. Would it be overkill to do that or do you think it would be justified? My -1 for this to go in however a setting may be the best way to keep the peace. My first impression was that it was overkill... but peace sounds pretty good.
          Hide
          Petr Škoda added a comment -

          Dear moodle.com user,

          "We can't keep putting this in and out." is not an argument that explains why every high profile site and all LMSs do the autofocus on single purpose login pages. In fact the removal was not discussed properly and should not get through in the first place. The linked issue also discusses general pages, but the login page is single purpose page which imho has different rules, I personally think that if screen reader or real user visits login page the first time it is always confusing no matter what is focused. On the subsequent visits try to imagine the screen reader starts to shout "username" or "password" - do we have any other page that does this? no, is it easy to understand that you are on the login page without all the other text being read aloud? I believe yes. We could also use longer labels on the login form to make sure the form is read like "Login username" or "Login password"...

          I am going to try JAWS myself now, but I would love to hear what real people with disabilities think, our discussion here seems to be pretty clueless.

          Ciao

          Show
          Petr Škoda added a comment - Dear moodle.com user, "We can't keep putting this in and out." is not an argument that explains why every high profile site and all LMSs do the autofocus on single purpose login pages. In fact the removal was not discussed properly and should not get through in the first place. The linked issue also discusses general pages, but the login page is single purpose page which imho has different rules, I personally think that if screen reader or real user visits login page the first time it is always confusing no matter what is focused. On the subsequent visits try to imagine the screen reader starts to shout "username" or "password" - do we have any other page that does this? no, is it easy to understand that you are on the login page without all the other text being read aloud? I believe yes. We could also use longer labels on the login form to make sure the form is read like "Login username" or "Login password"... I am going to try JAWS myself now, but I would love to hear what real people with disabilities think, our discussion here seems to be pretty clueless. Ciao
          Hide
          Petr Škoda added a comment -

          Anyway, feel free to revert it, I will try to find some more arguments for inclusion later.

          Show
          Petr Škoda added a comment - Anyway, feel free to revert it, I will try to find some more arguments for inclusion later.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          +1 for adding config option (or detect reader)

          +1 to take out autofocus from other places, lol, http://tracker.moodle.org/

          Peace, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - +1 for adding config option (or detect reader) +1 to take out autofocus from other places, lol, http://tracker.moodle.org/ Peace, ciao
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This has been reverted and reopened. Then re-triaged to stable. Plz, agree one solution and implement based on that.

          Show
          Eloy Lafuente (stronk7) added a comment - This has been reverted and reopened. Then re-triaged to stable. Plz, agree one solution and implement based on that.
          Hide
          Tim Hunt added a comment -

          Doesn't HTML5 have a new (non-JS) way to focus a particular form field on page-load? What do accessibility experts think of that? If you like, get sam m, or one of the other OU people, to ask our accessibility experts.

          Show
          Tim Hunt added a comment - Doesn't HTML5 have a new (non-JS) way to focus a particular form field on page-load? What do accessibility experts think of that? If you like, get sam m, or one of the other OU people, to ask our accessibility experts.
          Hide
          Petr Škoda added a comment - - edited

          Why would they include it in HTML 5 if it was forbidden to use it everywhere. I was reading several articles about autofocus, it seems the general agreement is that is it bad to include it in every form but there are some legitimate uses on pages where the only purpose is to enter something - I personally think our login page is exactly that reasonable exception.

          It would be great if we could announce that Moodle 2.2 moves from XHTML strict to HTML 5, then we would simply add the autofocus and blame the browser vendors. New browsers should give the option to turn off the HTML5 autofocus completely, the JS solution is nasty because it would be hard to detect the nasty autofocus tricks from browser code in some backwards compatible way. By the way the iOS Safari already disables setting of focus manually, so yes technically it might be possible inside browser too, but of course it does not have any keyboard navigation which makes is easier to do.

          Anyway I do not want to waste my time fighting here, I am going to create a patch with setting that defaults to no focus on the login page.

          Show
          Petr Škoda added a comment - - edited Why would they include it in HTML 5 if it was forbidden to use it everywhere. I was reading several articles about autofocus, it seems the general agreement is that is it bad to include it in every form but there are some legitimate uses on pages where the only purpose is to enter something - I personally think our login page is exactly that reasonable exception. It would be great if we could announce that Moodle 2.2 moves from XHTML strict to HTML 5, then we would simply add the autofocus and blame the browser vendors. New browsers should give the option to turn off the HTML5 autofocus completely, the JS solution is nasty because it would be hard to detect the nasty autofocus tricks from browser code in some backwards compatible way. By the way the iOS Safari already disables setting of focus manually, so yes technically it might be possible inside browser too, but of course it does not have any keyboard navigation which makes is easier to do. Anyway I do not want to waste my time fighting here, I am going to create a patch with setting that defaults to no focus on the login page.
          Hide
          Basil Gohar added a comment -

          Thanks for resolving this issue, Petr. I think this is the best way (don't harm accessibility by default, but allow usability to be improved with an option) to achieve the most benefit and cause the less harm.

          Show
          Basil Gohar added a comment - Thanks for resolving this issue, Petr. I think this is the best way (don't harm accessibility by default, but allow usability to be improved with an option) to achieve the most benefit and cause the less harm.
          Hide
          Sam Hemelryk added a comment -

          Thanks Petr - integrated now - second time round success

          Cheers
          Sam

          Show
          Sam Hemelryk added a comment - Thanks Petr - integrated now - second time round success Cheers Sam
          Hide
          David Mudrak added a comment -

          Tested on Firefox and Chrome. Works well, the autofocus obey they configuration setting. Thanks Petr!

          Show
          David Mudrak added a comment - Tested on Firefox and Chrome. Works well, the autofocus obey they configuration setting. Thanks Petr!

            People

            • Votes:
              2 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: