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

Two inconsistent layout files should be consistent, use id=page-content rather than content

    XMLWordPrintable

Details

    • MOODLE_26_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE
    • MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • MDL-47915_master
    • Hide
      1. Select Base theme.
      2. Go to a course page and add a SCORM package, such as the "Basic Run-Time Calls" golf sample from SCORM.com.
      3. Set the SCORM to display in a new window
      4. Open the SCORM and advance at least one page.
      5. Click the Exit button within the SCORM
      6. When prompted, choose Ok to save your progress

      Ensure that no errors are thrown. Specifically, this error should be fixed:
      Uncaught TypeError: Cannot read property 'appendChild' of null
      Display course structure on entry page Help with Display course structure on entry page

      I apologize, I'm not sure which user-visible functionality is affected. I'll be out of the office for a while, so I probably won't track it down soon, but clearly a) there should not be errors thrown and b) consistent naming between themes and layouts is a good thing.

      Show
      Select Base theme. Go to a course page and add a SCORM package, such as the "Basic Run-Time Calls" golf sample from SCORM.com. Set the SCORM to display in a new window Open the SCORM and advance at least one page. Click the Exit button within the SCORM When prompted, choose Ok to save your progress Ensure that no errors are thrown. Specifically, this error should be fixed: Uncaught TypeError: Cannot read property 'appendChild' of null Display course structure on entry page Help with Display course structure on entry page I apologize, I'm not sure which user-visible functionality is affected. I'll be out of the office for a while, so I probably won't track it down soon, but clearly a) there should not be errors thrown and b) consistent naming between themes and layouts is a good thing.

    Description

      All of the layout files have a main content div, which is always called id="page-content", except in two of the embedded.php layouts. This creates a problem for mod_scorm, whose module.js has long used getElementById('page-content').

      base/layout/embedded.php: <div id="content"
      base/layout/frontpage.php: <div id="page-content">
      base/layout/general.php: <div id="page-content">
      base/layout/report.php: <div id="page-content"
      bootstrapbase/layout/columns1.php: <div id="page-content"
      bootstrapbase/layout/columns2.php: <div id="page-content"
      bootstrapbase/layout/columns3.php: <div id="page-content"
      bootstrapbase/layout/embedded.php: <div id="page-content"
      bootstrapbase/layout/maintenance.php: <div id="page-content"
      bootstrapbase/layout/popup.php: <div id="page-content"
      bootstrapbase/layout/secure.php: <div id="page-content"
      canvas/layout/embedded.php: <div id="content"
      canvas/layout/frontpage.php: <div id="page-content">
      canvas/layout/general.php: <div id="page-content">
      canvas/layout/report.php: <div id="page-content">
      clean/layout/columns1.php: <div id="page-content"
      clean/layout/columns2.php: <div id="page-content"
      clean/layout/columns3.php: <div id="page-content"
      clean/layout/embedded.php: <div id="page-content"
      clean/layout/maintenance.php: <div id="page-content"
      clean/layout/secure.php: <div id="page-content"

      Attachments

        Activity

          People

            lazydaisy Mary Evans
            raymor Ray Morris
            Marina Glancy Marina Glancy
            Dan Poltawski Dan Poltawski
            Simey Lameze Simey Lameze
            Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Huong Nguyen, Bas Brands, Mathew May
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              12/Jan/15