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

Multichoice question type: correct answer display formatted badly

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2, 2.3.1, 2.4
    • Fix Version/s: 2.2.5, 2.3.2
    • Component/s: Questions
    • Labels:
    • Testing Instructions:
      Hide

      1. Create a multichoice question, with a single right answer.
      2. Preview it, and make sure the option Right answer is set to shown.
      3. Click Submit all and finish. Verify that the right answer is shown like "The correct answer is: No." all one one line, not split over several lines.

      4. Rinse and repeat with a question that allows multiple right answers.

      Show
      1. Create a multichoice question, with a single right answer. 2. Preview it, and make sure the option Right answer is set to shown. 3. Click Submit all and finish. Verify that the right answer is shown like "The correct answer is: No." all one one line, not split over several lines. 4. Rinse and repeat with a question that allows multiple right answers.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      The problem is TinyMCE wrapping each subquestion in <p> tags. We need to use CSS to supress that, like with multichoice specific feedback.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              amanda.doughty Amanda Doughty added a comment -

              This is the same issue as MDL-28979, so I cloned it, but I cannot edit the fix version out :/

              Show
              amanda.doughty Amanda Doughty added a comment - This is the same issue as MDL-28979 , so I cloned it, but I cannot edit the fix version out :/
              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
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated (22, 23 & master), thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (22, 23 & master), thanks!
              Hide
              dmonllao David Monllaó added a comment -

              Tested both in master and 22, it passes. The .r1.highlight overwrites of the last edited question also works as expected

              Show
              dmonllao David Monllaó added a comment - Tested both in master and 22, it passes. The .r1.highlight overwrites of the last edited question also works as expected
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Many thanks for the hard work.

              These changes have been spread upstream and are already available in the git and cvs repositories.

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work. These changes have been spread upstream and are already available in the git and cvs repositories. Ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    10/Sep/12