Details

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

      Description

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

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              ankit_frenz Ankit Agarwal added a comment - - edited

              Hi Glenn,
              Can you review this please?
              Thanks

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

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

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

              Thanks Glenn for the review
              Submitting for integration.
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - Thanks Glenn for the review Submitting for integration. Thanks
              Hide
              stronk7 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
              stronk7 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_frenz Ankit Agarwal added a comment -

              rebased!

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

              Thanks Ankit this has been integrated now

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

              This looks good.

              Test passed.

              Show
              rwijaya Rossiani Wijaya added a comment - This looks good. Test passed.
              Hide
              stronk7 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
              stronk7 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:
                    Fix Release Date:
                    12/Mar/12