Moodle
  1. Moodle
  2. MDL-26953

theme nonzero: right column disappears in .side-post-only course

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.0.1, 2.0.2
    • Fix Version/s: 2.0.6, 2.1.3
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      1. Before selecting Nonzero in Theme Selector make sure ALL blocks are on the right-hand side (side-post).
      2. Select Nonzero theme in normal way taking particular note of the SCREENSHOT of what the theme should look like when side-pre is visible.
      3. Once selected CHECK page layout, especially "Side Post ONLY". Is it consistent throughout, including when side blocks are docked?
      4. Now enable EDITING and attempt to MOVE a block or two. You should be able to move a block from SIDE-POST (right) to SIDE-PRE (left).
      5. With blocks in position (left and right) disable editing, and test for consistency, when docking Side-Pre (left) and then Side-Post (right). Normally when all blocks from Side-Pre have been docked the page reverts to a side-post-only layout. This is true also when Side-Pre is still in place and Side-Post is docked.
      6. FURTHER TESTING IF TIME PERMITS: Start by going to Nonzero Custom Settings page (Site Administration > Appearance > Themes -> CLICK the link Nonzero to bring you to the custom settings page for this theme. Here you can change the width settings for both side-pre and side-post. Choose different settings. Save changes and test as per 1 to 5 above.

      Show
      1. Before selecting Nonzero in Theme Selector make sure ALL blocks are on the right-hand side (side-post). 2. Select Nonzero theme in normal way taking particular note of the SCREENSHOT of what the theme should look like when side-pre is visible. 3. Once selected CHECK page layout, especially "Side Post ONLY". Is it consistent throughout, including when side blocks are docked? 4. Now enable EDITING and attempt to MOVE a block or two. You should be able to move a block from SIDE-POST (right) to SIDE-PRE (left). 5. With blocks in position (left and right) disable editing, and test for consistency, when docking Side-Pre (left) and then Side-Post (right). Normally when all blocks from Side-Pre have been docked the page reverts to a side-post-only layout. This is true also when Side-Pre is still in place and Side-Post is docked. 6. FURTHER TESTING IF TIME PERMITS: Start by going to Nonzero Custom Settings page (Site Administration > Appearance > Themes -> CLICK the link Nonzero to bring you to the custom settings page for this theme. Here you can change the width settings for both side-pre and side-post. Choose different settings. Save changes and test as per 1 to 5 above.
    • Difficulty:
      Moderate
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull Master Branch:
    • Rank:
      16584

      Description

      If you have a .side-post-only course the right column disappears.

      A possible solution seems to be:

      /** Only side post **/
      .side-post-only #page-content #region-main-box {
      margin-left: 0px;
      }

      .side-post-only #page-content #region-post-box {
      margin-left: -[[setting:regionpostwidth]];
      }

      .side-post-only #page-content #region-main {
      margin-left: [[setting:regionpostwidth]];
      }

      .side-post-only #page-content #region-post {
      /*** removed *** left: [[setting:regionsumwidth]]; ***/
      left: [[setting:regionpostwidth]];
      width: [[setting:regionpostwidth]];
      }

      /*** new ***/.pagelayout-frontpage.side-post-only #page-content #region-main-box #region-post-box #region-post,
      .pagelayout-frontpage.has_dock.side-post-only #page-content #region-main-box #region-post-box #region-post,
      .editing.has_dock.side-post-only #page-content #region-main-box #region-post-box #region-post {
      left: [[setting:regionsumwidth]];
      width: [[setting:regionpostwidth]];
      }

      .has_dock.side-post-only .page-middle #region-main-box #region-post-box #region-main-wrap #region-main {
      margin-left: [[setting:regionprewidth]];
      }

      @Mary: I'm a little bit confused about the rendering problems. I hope we can find a fast solution!
      I have the feeling that some body classes (.has_dock, .editing, .side-post-only, .drag) are not working together in the right way!? Dietmar

        Issue Links

          Activity

          Hide
          John Stabinger added a comment -

          I think this works better:

          .side-post-only #page-content #region-post

          { left: [[setting:regionpostwidth]]; width: [[setting:regionpostwidth]]; }

          .side-post-only #page-content #region-pre

          { display: none; }

          There is a lot less to account for and seems to address the issue. If anyone else sees differently, let me know. Otherwise, I will make this change and commit it in the next few days.

          Show
          John Stabinger added a comment - I think this works better: .side-post-only #page-content #region-post { left: [[setting:regionpostwidth]]; width: [[setting:regionpostwidth]]; } .side-post-only #page-content #region-pre { display: none; } There is a lot less to account for and seems to address the issue. If anyone else sees differently, let me know. Otherwise, I will make this change and commit it in the next few days.
          Hide
          Dietmar Wagner added a comment -

          Hi John, thanks for your reply.

          Your solution works until you turn edting on in a .side-post-only course (please look at the attachement above).
          I think there are some unwanted interferences between the body classes .edting, .has-dock and .side-post-only!?
          That's why I proposed a more complicated solution.

          Regards!
          Dietmar

          (Same problem in sky-high)

          Show
          Dietmar Wagner added a comment - Hi John, thanks for your reply. Your solution works until you turn edting on in a .side-post-only course (please look at the attachement above). I think there are some unwanted interferences between the body classes .edting, .has-dock and .side-post-only!? That's why I proposed a more complicated solution. Regards! Dietmar (Same problem in sky-high)
          Hide
          Dietmar Wagner added a comment -

          Hi John,
          as I'm going on foreign travel in the next days I would like to leave some further thoughts.

          I think the gap between .region-main and .region-post in nonzero should not open before moving blocks.
          But we can live with your (and my) solution that the gap comes up if you turn editing on.

          On the other hand, there are some themes that have a strange behaviour in the same situation.
          If you have a .side-post-only course

          • and theme standard/anomaly/overlay and perhaps other themes you are "imprisoned" in .side-post-only layout as there is no such gap and it is not possible to move blocks to .region-pre,
          • and theme magazine there is no such gap but it is possible to move blocks to the left.

          Shouldn't all themes behave similar in the described situation?

          Regards,
          Dietmar

          Show
          Dietmar Wagner added a comment - Hi John, as I'm going on foreign travel in the next days I would like to leave some further thoughts. I think the gap between .region-main and .region-post in nonzero should not open before moving blocks. But we can live with your (and my) solution that the gap comes up if you turn editing on. On the other hand, there are some themes that have a strange behaviour in the same situation. If you have a .side-post-only course and theme standard/anomaly/overlay and perhaps other themes you are "imprisoned" in .side-post-only layout as there is no such gap and it is not possible to move blocks to .region-pre, and theme magazine there is no such gap but it is possible to move blocks to the left. Shouldn't all themes behave similar in the described situation? Regards, Dietmar
          Hide
          Dietmar Wagner added a comment -

          Lets start again.

          Nonzero, course view, side-post-only, right column disappears.

          We agree that changing

          .side-post-only #page-content #region-post

          {left: [[setting:regionsumwidth]];width: [[setting:regionpostwidth]];}

          to
          .side-post-only #page-content #region-post

          {left: [[setting:regionpostwidth]];width: [[setting:regionpostwidth]];}

          makes region-post visible in course view.

          [.side-post-only #page-content #region-pre

          { display: none; }

          has no effects!?]

          Everything seems to be fine so far.

          Course

          But if you turn editing on, the class side-post-only is no more related to the body and you get a gap at the place where region-pre should be.
          Why that?? This gap is only required when moving blocks.
          If you dock one of the blocks after that, the gap disappears and the body is again side-post-only. What for??

          Frontpage

          Go to the frontpage and undock all blocks, but let editing turned on.
          If you now move all the blocks to right side the class side-post-only stays related to the body, even if you turn editing off and on again, and you can't see any gap.
          What's the difference between a side-post-only pagelayout-course and a side-post-only pagelayout-frontpage??

          If you try to move a block to region-pre in this situation, you have no chance to do that.
          But if you dock one of the blocks, it is possible to move the other blocks to region-pre. Confusing??!!
          Undocking this block leaves all as it is???

          My Conclusion

          All that doesn't really make sense. We could correct this issues by some more css rules in nonzero/pagelayout.css.
          As even the core themes don't work correctly in side-post-only situations I propose to rework the body classes
          (pagelayout-course, pagelayout-frontpage, editing, side-post-only, has_dock and blocks-moving) as a whole.

          What's your opinion?

          Dietmar

          Show
          Dietmar Wagner added a comment - Lets start again. Nonzero, course view, side-post-only, right column disappears. We agree that changing .side-post-only #page-content #region-post {left: [[setting:regionsumwidth]];width: [[setting:regionpostwidth]];} to .side-post-only #page-content #region-post {left: [[setting:regionpostwidth]];width: [[setting:regionpostwidth]];} makes region-post visible in course view. [.side-post-only #page-content #region-pre { display: none; } has no effects!?] Everything seems to be fine so far. Course But if you turn editing on, the class side-post-only is no more related to the body and you get a gap at the place where region-pre should be. Why that?? This gap is only required when moving blocks. If you dock one of the blocks after that, the gap disappears and the body is again side-post-only. What for?? Frontpage Go to the frontpage and undock all blocks, but let editing turned on. If you now move all the blocks to right side the class side-post-only stays related to the body, even if you turn editing off and on again, and you can't see any gap. What's the difference between a side-post-only pagelayout-course and a side-post-only pagelayout-frontpage?? If you try to move a block to region-pre in this situation, you have no chance to do that. But if you dock one of the blocks, it is possible to move the other blocks to region-pre. Confusing??!! Undocking this block leaves all as it is??? My Conclusion All that doesn't really make sense. We could correct this issues by some more css rules in nonzero/pagelayout.css. As even the core themes don't work correctly in side-post-only situations I propose to rework the body classes (pagelayout-course, pagelayout-frontpage, editing, side-post-only, has_dock and blocks-moving) as a whole. What's your opinion? Dietmar
          Hide
          John Stabinger added a comment -

          I think that gap, when editing is turned on, is to give you a place to move blocks. Otherwise, you could not get out of that layout. This was probably the easiest way to do it (i.e., remove the side-post-only and side-pre-only classes when editing is turned on).

          I can live with the gap when editing is turned on, as similar behavior would be expected from all three-column themes. However I am open to something else if you can think of anything...

          Show
          John Stabinger added a comment - I think that gap, when editing is turned on, is to give you a place to move blocks. Otherwise, you could not get out of that layout. This was probably the easiest way to do it (i.e., remove the side-post-only and side-pre-only classes when editing is turned on). I can live with the gap when editing is turned on, as similar behavior would be expected from all three-column themes. However I am open to something else if you can think of anything...
          Hide
          Dietmar Wagner added a comment -

          You are right. I can live with this gap too. But it should also be visible on a side-post-only frontpage when editing is turned on, or? And the gap should not disappear, when you dock one of the blocks!

          Show
          Dietmar Wagner added a comment - You are right. I can live with this gap too. But it should also be visible on a side-post-only frontpage when editing is turned on, or? And the gap should not disappear, when you dock one of the blocks!
          Hide
          Dietmar Wagner added a comment -

          This should fix the main issues:

          /** Only side post **/

          .side-post-only #page-content #region-main-box

          { margin-left: 0px; }

          .side-post-only #page-content #region-post-box

          { margin-left: -[[setting:regionpostwidth]]; }

          .side-post-only #page-content #region-main

          { margin-left: [[setting:regionpostwidth]]; }

          .side-post-only #page-content #region-post

          { left: [[setting:regionpostwidth]]; width: [[setting:regionpostwidth]]; }

          .editing.side-post-only.has_dock #page-content #region-main-box #region-post-box #region-post

          { left: [[setting:regionsumwidth]]; width: [[setting:regionpostwidth]]; }

          .editing.side-post-only.has_dock #page-content #region-main-box #region-post-box #region-main

          {margin-right: [[setting:regionprewidth]]; }

          .pagelayout-frontpage.editing.side-post-only #page-content #region-main-box #region-post-box #region-main

          {margin-right: [[setting:regionprewidth]]; }

          .pagelayout-frontpage.editing.side-post-only #page-content #region-main-box #region-post

          { left: [[setting:regionsumwidth]]; width: [[setting:regionpostwidth]]; }

          .pagelayout-frontpage.editing.side-post-only #page-content #region-main-box #region-pre

          { left: 0px; }

          .has_dock.side-post-only .page-middle #region-main-box #region-post-box #region-main-wrap #region-main

          { margin-left: [[setting:regionprewidth]]; }

          (Perhaps we can have a little bit shorter.)

          One issue is left:
          Go to a side-post-only course, turn editing on, dock a block, undock it: region-post jumps to a position it never had before!Unable to render embedded object: File (?? Very strange) not found.! Any idea?

          Show
          Dietmar Wagner added a comment - This should fix the main issues: /** Only side post **/ .side-post-only #page-content #region-main-box { margin-left: 0px; } .side-post-only #page-content #region-post-box { margin-left: -[[setting:regionpostwidth]]; } .side-post-only #page-content #region-main { margin-left: [[setting:regionpostwidth]]; } .side-post-only #page-content #region-post { left: [[setting:regionpostwidth]]; width: [[setting:regionpostwidth]]; } .editing.side-post-only.has_dock #page-content #region-main-box #region-post-box #region-post { left: [[setting:regionsumwidth]]; width: [[setting:regionpostwidth]]; } .editing.side-post-only.has_dock #page-content #region-main-box #region-post-box #region-main {margin-right: [[setting:regionprewidth]]; } .pagelayout-frontpage.editing.side-post-only #page-content #region-main-box #region-post-box #region-main {margin-right: [[setting:regionprewidth]]; } .pagelayout-frontpage.editing.side-post-only #page-content #region-main-box #region-post { left: [[setting:regionsumwidth]]; width: [[setting:regionpostwidth]]; } .pagelayout-frontpage.editing.side-post-only #page-content #region-main-box #region-pre { left: 0px; } .has_dock.side-post-only .page-middle #region-main-box #region-post-box #region-main-wrap #region-main { margin-left: [[setting:regionprewidth]]; } (Perhaps we can have a little bit shorter.) One issue is left: Go to a side-post-only course, turn editing on, dock a block, undock it: region-post jumps to a position it never had before! Unable to render embedded object: File (?? Very strange) not found. ! Any idea?
          Hide
          John Stabinger added a comment -

          The frontpage issue, is related to a problem with the way the frontpage layout file has declared the body classes. It works just as expected it they are corrected to:

          $hasheading = ($PAGE->heading);
          $hasnavbar = (empty($PAGE->layout_options['nonavbar']) && $PAGE->has_navbar());
          $hasfooter = (empty($PAGE->layout_options['nofooter']));
          $hassidepre = $PAGE->blocks->region_has_content('side-pre', $OUTPUT);
          $hassidepost = $PAGE->blocks->region_has_content('side-post', $OUTPUT);
          $custommenu = $OUTPUT->custom_menu();
          $hascustommenu = (empty($PAGE->layout_options['nocustommenu']) && !empty($custommenu));

          $bodyclasses = array();
          if ($hassidepre && !$hassidepost)

          { $bodyclasses[] = 'side-pre-only'; }

          else if ($hassidepost && !$hassidepre)

          { $bodyclasses[] = 'side-post-only'; }

          else if (!$hassidepost && !$hassidepre)

          { $bodyclasses[] = 'content-only'; }

          if ($hascustommenu)

          { $bodyclasses[] = 'has_custom_menu'; }

          Of course, that still leaves the problem of docking when in editing mode. Docking, when in editing mode, creates the body class that turning on editing removes.

          I believe your code works for the most part, I am just worried about maintainability...

          Show
          John Stabinger added a comment - The frontpage issue, is related to a problem with the way the frontpage layout file has declared the body classes. It works just as expected it they are corrected to: $hasheading = ($PAGE->heading); $hasnavbar = (empty($PAGE->layout_options ['nonavbar'] ) && $PAGE->has_navbar()); $hasfooter = (empty($PAGE->layout_options ['nofooter'] )); $hassidepre = $PAGE->blocks->region_has_content('side-pre', $OUTPUT); $hassidepost = $PAGE->blocks->region_has_content('side-post', $OUTPUT); $custommenu = $OUTPUT->custom_menu(); $hascustommenu = (empty($PAGE->layout_options ['nocustommenu'] ) && !empty($custommenu)); $bodyclasses = array(); if ($hassidepre && !$hassidepost) { $bodyclasses[] = 'side-pre-only'; } else if ($hassidepost && !$hassidepre) { $bodyclasses[] = 'side-post-only'; } else if (!$hassidepost && !$hassidepre) { $bodyclasses[] = 'content-only'; } if ($hascustommenu) { $bodyclasses[] = 'has_custom_menu'; } Of course, that still leaves the problem of docking when in editing mode. Docking, when in editing mode, creates the body class that turning on editing removes. I believe your code works for the most part, I am just worried about maintainability...
          Hide
          Dietmar Wagner added a comment -

          Some important hints ...!

          Finally - in order to retain maintainability - I think you should do some minimal changes

          • at the top of the frontpage.php and general.php (body classes) and
          • to the pagelayout.css (.side-post-only #page-content #region-post { left: [[setting:regionpostwidth]]; width: [[setting:regionpostwidth]]; }

          hoping that the side-post-only issue happens extremely rarely.

          Thanks for caring about this issue!

          Show
          Dietmar Wagner added a comment - Some important hints ...! Finally - in order to retain maintainability - I think you should do some minimal changes at the top of the frontpage.php and general.php (body classes) and to the pagelayout.css (.side-post-only #page-content #region-post { left: [[setting:regionpostwidth]]; width: [[setting:regionpostwidth]]; } hoping that the side-post-only issue happens extremely rarely. Thanks for caring about this issue!
          Hide
          Dietmar Wagner added a comment -

          Welcome here, Mary! Let's start again.

          Here the TODO-LIST for nonzero (as far as I can see):

          1. fix the ;;-bug

          2. change
          .side-post-only #page-content #region-post

          {left: [[setting:regionsumwidth]];width: [[setting:regionpostwidth]];}

          to
          .side-post-only #page-content #region-post

          {left: [[setting:regionpostwidth]];width: [[setting:regionpostwidth]];}

          3. add
          .side-post-only #page-content #region-pre

          { width: 0; }

          4. add
          .blocks-moving.side-post-only #page-content #region-main-box

          { margin-left: -[[setting:regionpostwidth]]; }

          .blocks-moving.side-post-only #page-content #region-post-box

          { margin-left: -[[setting:regionprewidth]]; }

          .blocks-moving.side-post-only #page-content #region-main

          { margin-left: [[setting:regionsumwidth]]; }

          .blocks-moving.side-post-only #page-content #region-pre

          { width: [[setting:regionprewidth]]; left: [[setting:regionprewidth]]; }

          .blocks-moving.side-post-only #page-content #region-post

          { left: [[setting:regiondoublepresumwidth]]; width: [[setting:regionpostwidth]]; }

          5. alter first lines of general.php: copy the following lines form the frontpage.php and paste them into general.php:

          <?php

          $hasheading = ($PAGE->heading);
          $hasnavbar = (empty($PAGE->layout_options['nonavbar']) && $PAGE->has_navbar());
          $hasfooter = (empty($PAGE->layout_options['nofooter']));

          $hassidepre = $PAGE->blocks->region_has_content('side-pre', $OUTPUT);
          $hassidepost = $PAGE->blocks->region_has_content('side-post', $OUTPUT);

          $showsidepre = ($hassidepre && !$PAGE->blocks->region_completely_docked('side-pre', $OUTPUT));
          $showsidepost = ($hassidepost && !$PAGE->blocks->region_completely_docked('side-post', $OUTPUT));

          $custommenu = $OUTPUT->custom_menu();
          $hascustommenu = (empty($PAGE->layout_options['nocustommenu']) && !empty($custommenu));

          $bodyclasses = array();
          if ($showsidepre && !$showsidepost)

          { $bodyclasses[] = 'side-pre-only'; }

          else if ($showsidepost && !$showsidepre)

          { $bodyclasses[] = 'side-post-only'; }

          else if (!$showsidepost && !$showsidepre)

          { $bodyclasses[] = 'content-only'; }

          if ($hascustommenu)

          { $bodyclasses[] = 'has_custom_menu'; }

          6. test, test, test ... (It works for me!)

          Cheers
          Dietmar

          Show
          Dietmar Wagner added a comment - Welcome here, Mary! Let's start again. Here the TODO-LIST for nonzero (as far as I can see): 1. fix the ;;-bug 2. change .side-post-only #page-content #region-post {left: [[setting:regionsumwidth]];width: [[setting:regionpostwidth]];} to .side-post-only #page-content #region-post {left: [[setting:regionpostwidth]];width: [[setting:regionpostwidth]];} 3. add .side-post-only #page-content #region-pre { width: 0; } 4. add .blocks-moving.side-post-only #page-content #region-main-box { margin-left: -[[setting:regionpostwidth]]; } .blocks-moving.side-post-only #page-content #region-post-box { margin-left: -[[setting:regionprewidth]]; } .blocks-moving.side-post-only #page-content #region-main { margin-left: [[setting:regionsumwidth]]; } .blocks-moving.side-post-only #page-content #region-pre { width: [[setting:regionprewidth]]; left: [[setting:regionprewidth]]; } .blocks-moving.side-post-only #page-content #region-post { left: [[setting:regiondoublepresumwidth]]; width: [[setting:regionpostwidth]]; } 5. alter first lines of general.php: copy the following lines form the frontpage.php and paste them into general.php: <?php $hasheading = ($PAGE->heading); $hasnavbar = (empty($PAGE->layout_options ['nonavbar'] ) && $PAGE->has_navbar()); $hasfooter = (empty($PAGE->layout_options ['nofooter'] )); $hassidepre = $PAGE->blocks->region_has_content('side-pre', $OUTPUT); $hassidepost = $PAGE->blocks->region_has_content('side-post', $OUTPUT); $showsidepre = ($hassidepre && !$PAGE->blocks->region_completely_docked('side-pre', $OUTPUT)); $showsidepost = ($hassidepost && !$PAGE->blocks->region_completely_docked('side-post', $OUTPUT)); $custommenu = $OUTPUT->custom_menu(); $hascustommenu = (empty($PAGE->layout_options ['nocustommenu'] ) && !empty($custommenu)); $bodyclasses = array(); if ($showsidepre && !$showsidepost) { $bodyclasses[] = 'side-pre-only'; } else if ($showsidepost && !$showsidepre) { $bodyclasses[] = 'side-post-only'; } else if (!$showsidepost && !$showsidepre) { $bodyclasses[] = 'content-only'; } if ($hascustommenu) { $bodyclasses[] = 'has_custom_menu'; } 6. test, test, test ... (It works for me!) Cheers Dietmar
          Hide
          Mary Evans added a comment -

          It's looking good!

          These seem to match what I have in my version of Nonzero.

          Cheers
          Mary

          Show
          Mary Evans added a comment - It's looking good! These seem to match what I have in my version of Nonzero. Cheers Mary
          Hide
          Sam Hemelryk added a comment -

          Thanks guys - this has been integrated now. Spot on

          Show
          Sam Hemelryk added a comment - Thanks guys - this has been integrated now. Spot on
          Hide
          Mary Evans added a comment -

          Thanks Sam!

          Show
          Mary Evans added a comment - Thanks Sam!
          Hide
          Rossiani Wijaya added a comment -

          In attempt to test this, I moved all pre-side block to post-side then attempted a move for 1 of the block, I noticed the indication for possible new location for pre-side block is located on the top right of page-content section (see-attachment). It also effected IE9 and chrome. on the side note, I did purge all caches and browser cache.

          This issue also occurs in formFormat theme. I will create new issue to fix formformat.

          Show
          Rossiani Wijaya added a comment - In attempt to test this, I moved all pre-side block to post-side then attempted a move for 1 of the block, I noticed the indication for possible new location for pre-side block is located on the top right of page-content section (see-attachment). It also effected IE9 and chrome. on the side note, I did purge all caches and browser cache. This issue also occurs in formFormat theme. I will create new issue to fix formformat.
          Hide
          Mary Evans added a comment -

          I put a wrong value in for.blocks-moving.side-post-only #region-pre. For some reason I picked up the wrong code for that line!!! It should read as follows:

          .blocks-moving.side-post-only #page-content #region-main-box #region-post-box #region-pre {
              float: right;
              position: relative;
              width: [[setting:regionprewidth]];
              left: [[setting:regionprewidth]];
          }
          

          Is it too late to fix?

          Show
          Mary Evans added a comment - I put a wrong value in for.blocks-moving.side-post-only #region-pre. For some reason I picked up the wrong code for that line!!! It should read as follows: .blocks-moving.side-post-only #page-content #region-main-box #region-post-box #region-pre { float : right; position: relative; width: [[setting:regionprewidth]]; left: [[setting:regionprewidth]]; } Is it too late to fix?
          Hide
          Mary Evans added a comment - - edited

          @Sam

          I have just made amendments to MDL-26953, MDL-26953-M20 and MDL-26953-M21 to correct the error in the CSS I hope I have done the right thing by correcting this and not spoilt the chances of this getting through in this weeks PULL???

          CHANGED from THIS

          .blocks-moving.side-post-only #page-content #region-main-box #region-post-box #region-pre {
              float: right;
              position: relative;
              width: [[setting:leftregionwidthmargin]]; 
              left: [[setting:regionprewidth]];
          }
          

          to THIS

          .blocks-moving.side-post-only #page-content #region-main-box #region-post-box #region-pre {
              float: right;
              position: relative; 
              width: [[setting:regionprewidth]];
              left: [[setting:regionprewidth]];
          }
          

          Cheers
          Mary

          Show
          Mary Evans added a comment - - edited @Sam I have just made amendments to MDL-26953 , MDL-26953 -M20 and MDL-26953 -M21 to correct the error in the CSS I hope I have done the right thing by correcting this and not spoilt the chances of this getting through in this weeks PULL??? CHANGED from THIS .blocks-moving.side-post-only #page-content #region-main-box #region-post-box #region-pre { float : right; position: relative; width: [[setting:leftregionwidthmargin]]; left: [[setting:regionprewidth]]; } to THIS .blocks-moving.side-post-only #page-content #region-main-box #region-post-box #region-pre { float : right; position: relative; width: [[setting:regionprewidth]]; left: [[setting:regionprewidth]]; } Cheers Mary
          Hide
          Rossiani Wijaya added a comment -

          Failing the test to allow Sam to re-integrate.

          Show
          Rossiani Wijaya added a comment - Failing the test to allow Sam to re-integrate.
          Hide
          Mary Evans added a comment -

          Thanks Rossiani

          Show
          Mary Evans added a comment - Thanks Rossiani
          Hide
          Sam Hemelryk added a comment -

          Thanks for spotting Rosie and thanks for the fix Mary.
          The fix has been integrated and this is up for testing now.

          Cheers
          Sam

          Show
          Sam Hemelryk added a comment - Thanks for spotting Rosie and thanks for the fix Mary. The fix has been integrated and this is up for testing now. Cheers Sam
          Hide
          Rossiani Wijaya added a comment -

          Thanks Mary and Sam.

          This is now working great.

          Test passed.

          Show
          Rossiani Wijaya added a comment - Thanks Mary and Sam. This is now working great. Test passed.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Many thanks for all the hard work. This is now part of Moodle, your favorite LMS.

          Closing as fixed, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Many thanks for all the hard work. This is now part of Moodle, your favorite LMS. Closing as fixed, ciao

            People

            • Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: