Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-35973

Moodle redirect to new page has no page title (specifically forums)

    Details

    • Story Points (Obsolete):
      5
    • Sprint:
      FRONTEND Sprint 8

      Description

      This is issues was specifically found in the forums create discussion and reply to a post, but the behavior of redirection happens all over Moodle and is a library function.

      Issue
      Sufficient cues - Once the message is submitted, the confirmation message for posted discussion thread is presented in a new page that has a redirection to another page - this behavior should be reflected into the page title, so screen reader users understand what is going on.

      Timing Adjustable - When posting a reply to discussion, a page with a message "Posted successfully - Continue" is presented. But enough time is not given to user to click on Continue button. The user is redirected to a different page. While using JAWS, this page is not read at all.

      Standard Level
      WCAG 2 3.3.2 (A) http://www.w3.org/TR/UNDERSTANDING-WCAG20/minimize-error-cues.html
      WCAG 2 2.2.1 (A) http://www.w3.org/TR/UNDERSTANDING-WCAG20/time-limits-required-behaviors.html

      Impact
      Serious
      Critical

      Example Link
      http://accessibility.mroomstest.net/mod/forum/post.php?reply=1

      Test Steps

      1. Login as user0
      2. Navigate to the Algebra 101 course
      3. Click on a forum
      4. Click the create a discussion button
      5. Add the necessary information
      6. Click post
      7. View the redirection page.

        Gliffy Diagrams

          Activity

          Hide
          salvetore Michael de Raadt added a comment -

          Although the redirect() function is part of a library, the pages that call it can be part of any plugin or component, so this change may affect a lot of pages.

          The timing issue is not one I think we can overcome easily. We could arbitrarily extend redirect times, but I suspect the length of time required by different reader software used by different users could be quite subjective. Aiming at descriptive page titles that indicate the of the central message of the page that they are about to be redirect may help.

          Show
          salvetore Michael de Raadt added a comment - Although the redirect() function is part of a library, the pages that call it can be part of any plugin or component, so this change may affect a lot of pages. The timing issue is not one I think we can overcome easily. We could arbitrarily extend redirect times, but I suspect the length of time required by different reader software used by different users could be quite subjective. Aiming at descriptive page titles that indicate the of the central message of the page that they are about to be redirect may help.
          Hide
          jrh18 Jason Hardin added a comment -

          Yeah this might be easiest to handle in the page title. Making a page title that actually is long like the confirmation text. Something like You are being redirected to <next page title> in x seconds. That way if the page redirects quickly it is more likely for the page title to be at least read.

          Show
          jrh18 Jason Hardin added a comment - Yeah this might be easiest to handle in the page title. Making a page title that actually is long like the confirmation text. Something like You are being redirected to <next page title> in x seconds. That way if the page redirects quickly it is more likely for the page title to be at least read.
          Hide
          salvetore Michael de Raadt added a comment -

          I'm shifting this issue into a new Epic issue, where we are collecting together accessibility issues. 218225232

          Show
          salvetore Michael de Raadt added a comment - I'm shifting this issue into a new Epic issue, where we are collecting together accessibility issues. 218225232
          Hide
          andyjdavis Andrew Davis added a comment -

          I am putting a potential fix up for peer review. This changes causes the page title on the redirection page to be set to 'This page should automatically redirect. If nothing is happening please use the continue link below.'

          Show
          andyjdavis Andrew Davis added a comment - I am putting a potential fix up for peer review. This changes causes the page title on the redirection page to be set to 'This page should automatically redirect. If nothing is happening please use the continue link below.'
          Hide
          andyjdavis Andrew Davis added a comment -

          I have also taken the liberty of changing the delay before redirection when a new discussion is posted from 4 to 6 seconds. This isn't technically required but the redirection was so fast that the text was basically unreadable which seems likely to unsettle people.

          Show
          andyjdavis Andrew Davis added a comment - I have also taken the liberty of changing the delay before redirection when a new discussion is posted from 4 to 6 seconds. This isn't technically required but the redirection was so fast that the text was basically unreadable which seems likely to unsettle people.
          Hide
          skodak Petr Skoda added a comment -

          looks ok, +1, please submit for integration when ready

          Show
          skodak Petr Skoda added a comment - looks ok, +1, please submit for integration when ready
          Hide
          andyjdavis Andrew Davis added a comment -

          Added 2.5 and 2.6 versions. Submitting for integration.

          Show
          andyjdavis Andrew Davis added a comment - Added 2.5 and 2.6 versions. Submitting for integration.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Andrew - this has been integrated now.

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Andrew - this has been integrated now.
          Hide
          phalacee Jason Fowler added a comment -

          Works fine Andrew, thank you.

          Show
          phalacee Jason Fowler added a comment - Works fine Andrew, thank you.
          Hide
          damyon Damyon Wiese added a comment -

          David built a framework for behat
          At first just to test this and that
          10000+ steps written
          Sounds like we're all smitten
          And David should be smiling at that

          Thanks for reporting, patching, and testing this issue. It has been released upstream along with 64 others today.

          Show
          damyon Damyon Wiese added a comment - David built a framework for behat At first just to test this and that 10000+ steps written Sounds like we're all smitten And David should be smiling at that Thanks for reporting, patching, and testing this issue. It has been released upstream along with 64 others today.

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                13/Jan/14

                Agile