Moodle
  1. Moodle
  2. MDL-32299

Help icons for the review options part of the quiz settings form

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.3
    • Component/s: Quiz
    • Labels:
      None
    • Rank:
      39096

      Description

      This part of the form is not particularly easy to understand, and so would benefit from help icons.

        Activity

        Hide
        Tim Hunt added a comment -

        Here is a patch for peer-review.

        Show
        Tim Hunt added a comment - Here is a patch for peer-review.
        Hide
        Tim Hunt added a comment -

        Here is a screen-grab of the relevant section of the form after my changes.

        Show
        Tim Hunt added a comment - Here is a screen-grab of the relevant section of the form after my changes.
        Hide
        Tim Hunt added a comment -

        You can see the corresponding help strings in the patch.

        Show
        Tim Hunt added a comment - You can see the corresponding help strings in the patch.
        Hide
        Tim Hunt added a comment -

        Well, I think these changes are good, so I am submitting for integration without peer review. If anyone wants to object to that, please comment here.

        Peer review still welcome, if anyone has time to do that before next Monday.

        Show
        Tim Hunt added a comment - Well, I think these changes are good, so I am submitting for integration without peer review. If anyone wants to object to that, please comment here. Peer review still welcome, if anyone has time to do that before next Monday.
        Hide
        Aparup Banerjee added a comment -

        The main moodle.git repository has just been updated (yesterday) 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
        Aparup Banerjee added a comment - The main moodle.git repository has just been updated (yesterday) 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
        Dan Poltawski added a comment -

        Typo: $string['rightanswer_help'] = 'an automtically gererated summary

        Show
        Dan Poltawski added a comment - Typo: $string ['rightanswer_help'] = 'an automtically gererated summary
        Hide
        Dan Poltawski added a comment -

        I've integrated this, I added a commit on top to fix those typos.

        Show
        Dan Poltawski added a comment - I've integrated this, I added a commit on top to fix those typos.
        Hide
        Tim Hunt added a comment -

        Thanks for fixing the typos Dan.

        Show
        Tim Hunt added a comment - Thanks for fixing the typos Dan.
        Hide
        Adrian Greeve added a comment -

        I reviewed the text in all of the help icons. All of the comments make sense and give a better understanding of each option.
        Test passed.
        Thanks.

        Show
        Adrian Greeve added a comment - I reviewed the text in all of the help icons. All of the comments make sense and give a better understanding of each option. Test passed. Thanks.
        Hide
        Dan Poltawski added a comment -

        Jolly good show!

        Your changes have made it into the Moodle release - its time to celebrate! I suggest a hot cup of English tea (with milk, no sugar) or a hoppy English ale.

        Tally-ho!

        Show
        Dan Poltawski added a comment - Jolly good show! Your changes have made it into the Moodle release - its time to celebrate! I suggest a hot cup of English tea (with milk, no sugar) or a hoppy English ale. Tally-ho!

          People

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

            Dates

            • Created:
              Updated:
              Resolved: