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

SCORM player does not fulfill the XHTML 1.0 Strict specifications

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.3
    • Fix Version/s: 2.0
    • Component/s: Accessibility, SCORM
    • Labels:
      None

      Description

      This can also be seen in the demo page: http://demo.moodle.org/mod/scorm/player.php?a=4&scoid=25

      The main problems are:

      1) The form used here is
      <form name="tochead" method="post" action="player.php?id=237" target="_top">
      <input name="prev" type="button" value="Previous" onClick="document.location.href=' http://demo.moodle.org/mod/scorm/player.php?id=237&amp;currentorg=&amp;mode=normal&amp;scoid=24 '"/>
      </form>
      The form tag does not allow the parameters "name" and "target" in XHTML 1.0 Strict. Instead we should use "id" and no targe attribute.
      The input tag does not allow "onClick". The task of changing to url should be done in the "action" attribute of the form.

      2) All spacers image require an "alt" attribute.
      <img src="http://demo.moodle.org/mod/scorm/pix/spacer.gif" />
      should be
      <img src="http://demo.moodle.org/mod/scorm/pix/spacer.gif" alt="" />
      For more info why the alt attribute should be empty, visit: http://diveintoaccessibility.org/day_21_ignoring_spacer_images.html
      Moreover, this kind of web design should be avoided and use CSS instead.

      3) The SCORM player uses iframes which are not supported by XHTML 1.0 Strict. We should use the object tag instead as discussed in: MDL-17702

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                danmarsden Dan Marsden
                Reporter:
                david.horat David Horat
                Participants:
                Component watchers:
                Adrian Greeve, Mihail Geshoski, Peter Dias, Damyon Wiese, Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Mihail Geshoski, Peter Dias
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  24/Nov/10