Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Accessibility
    • Labels:
    • Rank:
      33902

      Description

      The buttons on the forgot password page have identical IDs. Each button simply needs to have a unique ID.

        Issue Links

          Activity

          Hide
          Ankit Agarwal added a comment - - edited

          Hi Glenn,
          Can you review this please?
          Thanks

          Show
          Ankit Agarwal added a comment - - edited Hi Glenn, Can you review this please? Thanks
          Hide
          Glenn Ansley added a comment -

          Looks good. I don't have ability to submit for integration though. Just a heads up.

          Show
          Glenn Ansley added a comment - Looks good. I don't have ability to submit for integration though. Just a heads up.
          Hide
          Ankit Agarwal added a comment -

          Thanks Glenn for the review
          Submitting for integration.
          Thanks

          Show
          Ankit Agarwal added a comment - Thanks Glenn for the review Submitting for integration. Thanks
          Hide
          Eloy Lafuente (stronk7) added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          Ankit Agarwal added a comment -

          rebased!

          Show
          Ankit Agarwal added a comment - rebased!
          Hide
          Sam Hemelryk added a comment -

          Thanks Ankit this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks Ankit this has been integrated now
          Hide
          Rossiani Wijaya added a comment -

          This looks good.

          Test passed.

          Show
          Rossiani Wijaya added a comment - This looks good. Test passed.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Your changes are now upstream and will be included in the next minor released scheduled for March 13th (next Monday!).

          icao_reverse('arreis olik rebemevon afla letoh ognat');
          

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Your changes are now upstream and will be included in the next minor released scheduled for March 13th (next Monday!). icao_reverse('arreis olik rebemevon afla letoh ognat'); Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: