Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide
      1. Select Bootstrap theme from Theme selector
      2. Add block Calendar if not already added.
      3. Click on the block calendar month which takes you to the calendar main page.
      4. Float the browser screen so that you can resize the window by pushing the right-hand side towards the left.
      5. TEST, when resizing the window, that the main calendar table does not extend beyond the centre column into the side blocks on the right.
      6. TEST again using the Simple theme.
      Show
      Select Bootstrap theme from Theme selector Add block Calendar if not already added. Click on the block calendar month which takes you to the calendar main page. Float the browser screen so that you can resize the window by pushing the right-hand side towards the left. TEST, when resizing the window, that the main calendar table does not extend beyond the centre column into the side blocks on the right. TEST again using the Simple theme.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
      MDL-38896_master

      Description

      When the screen is around 1024 pixels wide, the main calendar onon calendar/view.php?view=month&cal_d=1&cal_m=4&cal_y=2013&course=1 overflows on to the right blocks

        Gliffy Diagrams

          Activity

          Hide
          bawjaws David Scotson added a comment -

          I'm not seeing this until I get down to about 900px, but I think it might depend on the language used and how long their weekday names are.

          If I change the names to Su M T W Th F Sa then it shrinks down as expected (though I'm testing with an empty calender so that content might affect it as well).

          I think if it only happens at 900px and below then it's not a particularly big problem. If it's happening at 1024 pixels then something should probably be done about it, possibly inserting soft-breaks into or truncating the day name in some way.

          Show
          bawjaws David Scotson added a comment - I'm not seeing this until I get down to about 900px, but I think it might depend on the language used and how long their weekday names are. If I change the names to Su M T W Th F Sa then it shrinks down as expected (though I'm testing with an empty calender so that content might affect it as well). I think if it only happens at 900px and below then it's not a particularly big problem. If it's happening at 1024 pixels then something should probably be done about it, possibly inserting soft-breaks into or truncating the day name in some way.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Hi Mary,

          This gets a -1 from me presently sorry.

          Overriding another renderer method in the bootstrap theme. One of the tasks for after release is going to be removing the overridden renderers in the bootstrap theme and make fixes within core.
          In this particular case the renderer method is huge (not your fault at all) and the change being made is tiny.

          Having looked at the issue, and then at what others do I think we should just make these string changes in calendar/renderers.php anyway.

          I'll reopen this at the moment so that we can decide what to do.

          Many thanks
          Sam

          Show
          samhemelryk Sam Hemelryk added a comment - Hi Mary, This gets a -1 from me presently sorry. Overriding another renderer method in the bootstrap theme. One of the tasks for after release is going to be removing the overridden renderers in the bootstrap theme and make fixes within core. In this particular case the renderer method is huge (not your fault at all) and the change being made is tiny. Having looked at the issue, and then at what others do I think we should just make these string changes in calendar/renderers.php anyway. I'll reopen this at the moment so that we can decide what to do. Many thanks Sam
          Hide
          lazydaisy Mary Evans added a comment -

          I was thinking much the same thing after I commited this. Would be better all round if the strings where changed in CORE. Thanks for looking at this anyway.

          Show
          lazydaisy Mary Evans added a comment - I was thinking much the same thing after I commited this. Would be better all round if the strings where changed in CORE. Thanks for looking at this anyway.
          Hide
          cibot CiBoT added a comment -

          Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

          Show
          cibot CiBoT added a comment - Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Cool thanks Mary.

          Would you like me to make the changes or are you happy to?

          Many thanks
          Sam

          Show
          samhemelryk Sam Hemelryk added a comment - Cool thanks Mary. Would you like me to make the changes or are you happy to? Many thanks Sam
          Hide
          lazydaisy Mary Evans added a comment -

          It's OK I can do this. Thanks

          Show
          lazydaisy Mary Evans added a comment - It's OK I can do this. Thanks
          Hide
          lazydaisy Mary Evans added a comment -

          All fixed n ready to roll!

          Show
          lazydaisy Mary Evans added a comment - All fixed n ready to roll!
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          I'm not sure if this is going to fix anything really. If the main-calendar has events with medium strings (I just tried with "eat babanas @ superstore" in 2-3 days), then those events cause exactly the same problem to happen.

          So, abbreviating the day names always (that I was reticent about, no matter of the previous paragraph) doesn't sound to me like the best solution here.

          Hence my -0.5...ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - I'm not sure if this is going to fix anything really. If the main-calendar has events with medium strings (I just tried with "eat babanas @ superstore" in 2-3 days), then those events cause exactly the same problem to happen. So, abbreviating the day names always (that I was reticent about, no matter of the previous paragraph) doesn't sound to me like the best solution here. Hence my -0.5...ciao
          Hide
          lazydaisy Mary Evans added a comment - - edited

          But that's tables for you. It would probably be better changing the layout for the calendar to a grid so it could reshape if needed.

          Show
          lazydaisy Mary Evans added a comment - - edited But that's tables for you. It would probably be better changing the layout for the calendar to a grid so it could reshape if needed.
          Hide
          bawjaws David Scotson added a comment -

          You can replicate the current behaviour by stopping the container from shrinking, so something like

          @media (min-width: 768px) {

          #page-calender-view .container fluid

          {min-width: 1024px;}

          }

          It needs to be in a media tag to stop it interfering with display on mobile devices which do want to be smaller than that.

          Now when you narrow the page the 3 columns wont shrink past that limit and the right column will start to disappear.

          Show
          bawjaws David Scotson added a comment - You can replicate the current behaviour by stopping the container from shrinking, so something like @media (min-width: 768px) { #page-calender-view .container fluid {min-width: 1024px;} } It needs to be in a media tag to stop it interfering with display on mobile devices which do want to be smaller than that. Now when you narrow the page the 3 columns wont shrink past that limit and the right column will start to disappear.
          Hide
          poltawski Dan Poltawski added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          poltawski Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          lazydaisy Mary Evans added a comment - - edited

          Just redone this and added @media css rule.

          What I did not do is compile moodle.less => style/generated.css as not sure what the policy is going to be, nor if Eloys proposed changes have been applied or are going to be applied?

          Show
          lazydaisy Mary Evans added a comment - - edited Just redone this and added @media css rule. What I did not do is compile moodle.less => style/generated.css as not sure what the policy is going to be, nor if Eloys proposed changes have been applied or are going to be applied?
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Alrighty, this has been integrated now

          Show
          samhemelryk Sam Hemelryk added a comment - Alrighty, this has been integrated now
          Hide
          lazydaisy Mary Evans added a comment -

          Show
          lazydaisy Mary Evans added a comment -
          Hide
          ankit_frenz Ankit Agarwal added a comment -

          Works as described.
          Thanks

          Show
          ankit_frenz Ankit Agarwal added a comment - Works as described. Thanks
          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:
              5 Start watching this issue

              Dates

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