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

Main course formats need to have tables removed but keep AJAX working

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.8
    • Fix Version/s: 2.0
    • Component/s: Accessibility, JavaScript
    • Labels:
      None
    • Affected Branches:
      MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      The current main course formats currently use tables for layout

      course/format/topics
      course/format/weeks

      which the AJAX course editing functionality (eg Turn Editing on) relies upon (because of the DOM).

      These formats need to to be changed to use a DIV+CSS only structure exactly like

      course/format/weekscss

      And since weekscss will be deprecated it needs to be deleted and all courses using it can be changed to simply "weeks".

      The main issue though is that AJAX will no longer work because of the changed DOM.

      Someone needs to fix the AJAX so that it will work on these formats exactly as it does now.

        Attachments

        1. courseformats.tar
          43 kB
        2. csscourseformatpatch.txt
          7 kB
        3. cssformatpatch.txt
          5 kB
        4. cssformats.tar
          38 kB
        5. moodle2_tableless_course.png
          moodle2_tableless_course.png
          116 kB
        6. Musik WDH 2CB.doc
          26 kB

          Issue Links

            Activity

              People

              Assignee:
              moodle.com moodle.com
              Reporter:
              dougiamas Martin Dougiamas
              Tester:
              Nobody
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              7 Vote for this issue
              Watchers:
              14 Start watching this issue

                Dates

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