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

          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