Details

    • Testing Instructions:
      Hide

      1. Create lesson activity in a course
      2. add content page
      3. as admin, pre-view content page and view source for the content page title and make sure it uses <h3> tag.
      4. as student, attempt the lesson and check the content page title is using <h3> tag.

      Show
      1. Create lesson activity in a course 2. add content page 3. as admin, pre-view content page and view source for the content page title and make sure it uses <h3> tag. 4. as student, attempt the lesson and check the content page title is using <h3> tag.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:

      Description

      The title of the individual content page should be an h3, so it is less than the lesson title which is an h2.

        Gliffy Diagrams

          Activity

          Hide
          Rossiani Wijaya added a comment -

          Hi Greg,

          Could you provide the site url or page name for this?

          Show
          Rossiani Wijaya added a comment - Hi Greg, Could you provide the site url or page name for this?
          Hide
          Rossiani Wijaya added a comment -

          Nevermind, I found the issue.

          Show
          Rossiani Wijaya added a comment - Nevermind, I found the issue.
          Hide
          Andrew Davis added a comment -

          Hi. Consider adding a comment for these to explain to people in the future why there is a hard coded number 3 and why it is 3 (and not 2 or 4). Either that or use a constant. Up to you whether you want to do either of those things or just put it in as is.

          Show
          Andrew Davis added a comment - Hi. Consider adding a comment for these to explain to people in the future why there is a hard coded number 3 and why it is 3 (and not 2 or 4). Either that or use a constant. Up to you whether you want to do either of those things or just put it in as is.
          Hide
          Rossiani Wijaya added a comment -

          Hi Andrew,

          Thank you for reviewing.

          I added comment to the code and updated the patch.

          Submitting this for integration review.

          Show
          Rossiani Wijaya added a comment - Hi Andrew, Thank you for reviewing. I added comment to the code and updated the patch. Submitting this for integration review.
          Hide
          Sam Hemelryk added a comment -

          Thanks Rosie, this has been integrated now.

          Show
          Sam Hemelryk added a comment - Thanks Rosie, this has been integrated now.
          Hide
          Ankit Agarwal added a comment -

          This is working as described!
          Passing
          Thanks

          Show
          Ankit Agarwal added a comment - This is working as described! Passing Thanks
          Hide
          Eloy Lafuente (stronk7) added a comment -

          It is late here and I'm very tired but I didn't want to go to sleep before expressing my admiration for your amazing collaboration. Thanks!

          Closing as fixed, heading to zzzZZZzzz, niao

          Show
          Eloy Lafuente (stronk7) added a comment - It is late here and I'm very tired but I didn't want to go to sleep before expressing my admiration for your amazing collaboration. Thanks! Closing as fixed, heading to zzzZZZzzz, niao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: