Moodle
  1. Moodle
  2. MDL-36197

PATCH Fusion theme, "Your progress" overlaps with checkbox if description is shown

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.2
    • Fix Version/s: 2.2.6, 2.3.3
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide
      1. select fusion theme
      2. go to a course page where course format is either topics or weeks and has completion tracking enabled
      3. TEST that Your progress help link is clear of the completion tracking check-box, for the first item, when there is no topic or weekly outline title visible.
      Show
      select fusion theme go to a course page where course format is either topics or weeks and has completion tracking enabled TEST that Your progress help link is clear of the completion tracking check-box, for the first item, when there is no topic or weekly outline title visible.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull Master Branch:
      MDL-36197_master

      Description

      The Your Progress span can overlap the checkbox as shown in the image if the first item has a description, making it two lines.
      Attached is a patch which sets clear: both on the topics block so that the floated text "Your Progress" doesn't intrude into it's space.

        Gliffy Diagrams

          Activity

          Hide
          Mary Evans added a comment -

          Thanks for reporting this Rob, and for attaching a patch too.
          Many thanks
          Mary

          Show
          Mary Evans added a comment - Thanks for reporting this Rob, and for attaching a patch too. Many thanks Mary
          Hide
          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
          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
          Mary Evans added a comment -

          Rebased

          Show
          Mary Evans added a comment - Rebased
          Hide
          Sam Hemelryk added a comment -

          Thanks Mary, this has been integrated now.

          Show
          Sam Hemelryk added a comment - Thanks Mary, this has been integrated now.
          Hide
          Sam Hemelryk added a comment -

          Tested during integration review

          Show
          Sam Hemelryk added a comment - Tested during integration review
          Hide
          Mary Evans added a comment -

          Hi Sam,
          Is this too late to add a fix for Base theme too as I came across the same thing quite by chance the other day?

          Show
          Mary Evans added a comment - Hi Sam, Is this too late to add a fix for Base theme too as I came across the same thing quite by chance the other day?
          Hide
          Sam Hemelryk added a comment -

          Hi Mary, too late on this issue sorry. Best you open a new issue for it

          Many thanks
          Sam

          Show
          Sam Hemelryk added a comment - Hi Mary, too late on this issue sorry. Best you open a new issue for it Many thanks Sam
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Amazed. Inspired. Grateful. That’s how your generosity makes me feel.

          (not really)

          Closing, thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Amazed. Inspired. Grateful. That’s how your generosity makes me feel. (not really) Closing, thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: