Loading...

XML

Word

Printable

Type: Bug
Resolution: Fixed
Priority: Major
Fix Version/s: 2.4.7, 2.5.3
Affects Version/s: 2.4.6, 2.5.2
Component/s: Questions
Labels:
- triaged

Affected Branches:
MOODLE_24_STABLE, MOODLE_25_STABLE
Fixed Branches:
MOODLE_24_STABLE, MOODLE_25_STABLE
Pull from Repository:
git://github.com/timhunt/moodle.git
Pull Main Branch:
~~MDL-42232~~
Pull Main Diff URL:
https://github.com/timhunt/moodle/compare/master...MDL-42232
Testing Instructions:
Hide

This testing needs to be done in all of Firefox, Chrome and IE.

Create a multiple choice question with choices

{A table}

{A numbered list}

{Anything else interesting you can think of}

Also set feedback for each choice containing similar things.

Preview the question.

Ensure the preview options at the bottom of the pop-up are set to show everything.

Ensure that the choices are lined up OK. (Before this fix, the contents of table cells was shifted left 25px in IE and Chrome.)

Select a choice and submit.

Check that the feedback is inside the yellow background.
Show
This testing needs to be done in all of Firefox, Chrome and IE. Create a multiple choice question with choices {A table} {A numbered list} {Anything else interesting you can think of} Also set feedback for each choice containing similar things. Preview the question. Ensure the preview options at the bottom of the pop-up are set to show everything. Ensure that the choices are lined up OK. (Before this fix, the contents of table cells was shifted left 25px in IE and Chrome.) Select a choice and submit. Check that the feedback is inside the yellow background.

The text-indent: -25px; added in ~~MDL-39420~~ breaks tables or lists inside multiple-choice choices in Chrome or IE. (Strangely, Firefox is not affected.)

- - Sort By Name
  - Sort By Date
  - Ascending
  - Descending
  - Thumbnails
  - List
  - Download All

42232.png
23 kB
15/Oct/13 5:15 PM

is a regression caused by

MDL-39420 Improve display of multiple-choice questions with multi-line choices

Closed

Assignee:: Tim Hunt

Reporter:: Tim Hunt

Peer reviewer:: Sam Marshall

Integrator:: Dan Poltawski

Tester:: Andrew Lyons

Participants:: Andrew Lyons, Dan Poltawski, Eloy Lafuente (stronk7), Sam Marshall, Tim Hunt

Votes:: 0 Vote for this issue

Watchers:: 2 Start watching this issue

Created:: 10/Oct/13 12:34 AM

Updated:: 18/Oct/13 5:56 PM

Resolved:: 18/Oct/13 5:56 PM

Details

Description

Attachments

Attachments

Issue Links

Activity

People

Dates

Clockify