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

Remove wrongly placed region-pre ids in aside tags

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Themes
    • Labels:

      Description

      Like Base theme, both Bootstrap the Simple themes MUST be able to use drag and drop.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              damyon Damyon Wiese added a comment -

              Hmm - oops. I didn't see this in the queue when integrating MDL-38898 and I fixed it there.

              I'll close this as a duplicate (my fault sorry).

              Show
              damyon Damyon Wiese added a comment - Hmm - oops. I didn't see this in the queue when integrating MDL-38898 and I fixed it there. I'll close this as a duplicate (my fault sorry).
              Hide
              lazydaisy Mary Evans added a comment -

              Damyon, looks like you left 2 region-pre ids in the aside tags in the fix you added when you closed this issue without using the original patch when Integration MDL-38898.

              I've just pushed the fix to clean up the general.php.

              Cheers

              Show
              lazydaisy Mary Evans added a comment - Damyon, looks like you left 2 region-pre ids in the aside tags in the fix you added when you closed this issue without using the original patch when Integration MDL-38898 . I've just pushed the fix to clean up the general.php. Cheers
              Hide
              lazydaisy Mary Evans added a comment -

              Reopening this to fix regression

              Show
              lazydaisy Mary Evans added a comment - Reopening this to fix regression
              Hide
              poltawski Dan Poltawski added a comment -

              Hi Mary,

              I've looked at this but can't find a way to reproduce the problem? From my reading of the code it seems this only happens when there are no blocks at all? Am I misreading? How can I reproduce this problem?

              Show
              poltawski Dan Poltawski added a comment - Hi Mary, I've looked at this but can't find a way to reproduce the problem? From my reading of the code it seems this only happens when there are no blocks at all? Am I misreading? How can I reproduce this problem?
              Hide
              lazydaisy Mary Evans added a comment -

              There isn't a problem as such it is just that Damyon did not remove the id="region-pre" from the aside tag. Look at Bootstrap theme layout/general.php to compare.

              It makes me mad to think this would have been a smooth transition had Damyon not messed up the integration. Read MDL-38898 discussion for the story.

              Show
              lazydaisy Mary Evans added a comment - There isn't a problem as such it is just that Damyon did not remove the id="region-pre" from the aside tag. Look at Bootstrap theme layout/general.php to compare. It makes me mad to think this would have been a smooth transition had Damyon not messed up the integration. Read MDL-38898 discussion for the story.
              Hide
              lazydaisy Mary Evans added a comment -
              Show
              lazydaisy Mary Evans added a comment - You can see the commit for bootstrap here... https://github.com/lazydaisy/moodle/commit/c9ff4d0e0fa679ed7cfddf14456fc91c64131301
              Hide
              damyon Damyon Wiese added a comment -

              Thanks Mary, sorry for the original mess up.

              This has been integrated to master now. I tested it in integration, so I'll pass the test for this.

              Show
              damyon Damyon Wiese added a comment - Thanks Mary, sorry for the original mess up. This has been integrated to master now. I tested it in integration, so I'll pass the test for this.
              Hide
              damyon Damyon Wiese added a comment -

              Works as described.

              Thanks Mary!

              Show
              damyon Damyon Wiese added a comment - Works as described. Thanks Mary!
              Hide
              lazydaisy Mary Evans added a comment -

              Great stuff! Thanks you're a star!

              Show
              lazydaisy Mary Evans added a comment - Great stuff! Thanks you're a star!
              Hide
              poltawski Dan Poltawski added a comment -

              Thanks! You're changes are now spread to the world through this git and our source control repositories.

              No time to rest though, we've got days to make 2.5 the best yet!

              ciao

              Show
              poltawski Dan Poltawski added a comment - Thanks! You're changes are now spread to the world through this git and our source control repositories. No time to rest though, we've got days to make 2.5 the best yet! ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    14/May/13