Moodle
  1. Moodle
  2. MDL-13983

in a html quick if a radio button is disabled you get '( )' or '(x)' rather than just disabling the button

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.9, 2.0.2
    • Fix Version/s: 2.0.3
    • Component/s: Accessibility
    • Labels:
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Rank:
      4200

      Description

      as per http://tracker.moodle.org/browse/MDL-8627 but with radio buttons

      patch copied/modified from lib/form/advcheckbox.php should be applied to lib/form/radio.php

        Issue Links

          Activity

          Hide
          Sam Marshall added a comment -

          Petr asked for this to be assigned to him so he can check it

          Show
          Sam Marshall added a comment - Petr asked for this to be assigned to him so he can check it
          Hide
          Petr Škoda added a comment -

          going to ask MD tomorrow, my +0.9 for this

          Show
          Petr Škoda added a comment - going to ask MD tomorrow, my +0.9 for this
          Hide
          Eloy Lafuente (stronk7) added a comment -

          +0.1 here too (the idea, haven't reviewed the patch)

          Show
          Eloy Lafuente (stronk7) added a comment - +0.1 here too (the idea, haven't reviewed the patch)
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Pinging Mr. MD....

          Show
          Eloy Lafuente (stronk7) added a comment - Pinging Mr. MD....
          Hide
          Tim Hunt added a comment -

          This fix still seems to be good, and is now available as https://github.com/timhunt/moodle/commit/81994df4af82a24444aaac5afd4be3bd14684e78. Shall I do a pull request?

          Show
          Tim Hunt added a comment - This fix still seems to be good, and is now available as https://github.com/timhunt/moodle/commit/81994df4af82a24444aaac5afd4be3bd14684e78 . Shall I do a pull request?
          Hide
          Tim Hunt added a comment -

          Note that the corresponding change was made for checkboxes years ago, so I think it makes a lot of sense to do this for radio buttons to. This is code we have been using at the OU for ages.

          For testing, you need to find a form with radio buttons that get frozen, and there does not seem to be one in the standard Moodle code (I searched for ->freeze). I suspect this only came up in an ou-specific plugin.

          I you really want to test, create a test script with a form with a frozen radio button.

          I think this is a worthwhile accessibility fix, so the pull request should be accepted, even though this bit of the API is not currently used.

          Show
          Tim Hunt added a comment - Note that the corresponding change was made for checkboxes years ago, so I think it makes a lot of sense to do this for radio buttons to. This is code we have been using at the OU for ages. For testing, you need to find a form with radio buttons that get frozen, and there does not seem to be one in the standard Moodle code (I searched for ->freeze). I suspect this only came up in an ou-specific plugin. I you really want to test, create a test script with a form with a frozen radio button. I think this is a worthwhile accessibility fix, so the pull request should be accepted, even though this bit of the API is not currently used.
          Hide
          Helen Foster added a comment -

          Thanks again Tim.

          Show
          Helen Foster added a comment - Thanks again Tim.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: