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

Form 'required' icon is hidden

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.8, 1.9
    • Fix Version/s: 1.8, 1.9
    • Labels:
      None
    • Affected Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      Hi jamie, think this is one for you - hopefully i'll have a patch for you to review!
      (And, sorry i've left it so late to report this - I was aware of changes to this, and there are now problems...)

      There is a image of a red asterisk in the <label> against required form fields, but it is hidden with:
      " .req

      { display:none }

      "

      Issue 1 is that display:none means "expunge me from the DOM tree" and for some browser/ screen-reader combinations this means don't speak me. The work around is illustrated by the Standard theme class 'accesshide':
      " .accesshide

      { position:absolute; top: -1000px }

      "

      So, that's one way to fix it BUT issue 2 is that the only visible cue is dark red text in a <label> for a required field - if you're colour blind this may appear black, if you override with high-contrast settings...etc. - basic accessibility says do not convey information with colour alone.

      So I propose we show the red asterisk icon, perhaps add a title and help cursor, and keep the red text if you wish.
      Cheers Nick.

        Attachments

          Activity

            People

            • Assignee:
              nfreear Nick Freear
              Reporter:
              nfreear Nick Freear
              Tester:
              Nobody
              Participants:
              Component watchers:
              Adrian Greeve, Mihail Geshoski, Peter Dias, Jake Dallimore, Jun Pataleta, Ryan Wyllie
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                31/Mar/07