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

DOCKED Navigation block sub-level menu items are hidden in afterburner theme

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1, 2.2
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      1. Select Afterburner theme
      2. Dock Navigation block
      3. Test that the docked panel expands as the sub-level menu items are clicked...the more levels the wider the panel.
    • Workaround:
      Hide

      ADD the following CSS mark-up to the end of theme/afterburner/style/afterburner_dock.css

      #dockeditempanel .dockeditempanel_bd .block_navigation .block_tree li {overflow:visible;}

      Show
      ADD the following CSS mark-up to the end of theme/afterburner/style/afterburner_dock.css #dockeditempanel .dockeditempanel_bd .block_navigation .block_tree li {overflow:visible;}
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      When a Navigation block is docked and hovered over for access, the right side is cut off.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              samhemelryk Sam Hemelryk added a comment -

              Hi Mary,

              Can you please add more information to this issue.

              1. Git repo to pull from
              2. Testing instructions
              3. What branches should this change be applied to? I am happy to cherry-pick it to the required branches during integration providing it cherry-picks cleanly.

              Cheers
              Sam

              Show
              samhemelryk Sam Hemelryk added a comment - Hi Mary, Can you please add more information to this issue. Git repo to pull from Testing instructions What branches should this change be applied to? I am happy to cherry-pick it to the required branches during integration providing it cherry-picks cleanly. Cheers Sam
              Hide
              lazydaisy Mary Evans added a comment -

              Sam I do apologise for missing of vital information in this issue!

              Show
              lazydaisy Mary Evans added a comment - Sam I do apologise for missing of vital information in this issue!
              Hide
              lazydaisy Mary Evans added a comment -

              I've cherry-picked them and added the PULL branches here.
              Cheers
              Mary

              Show
              lazydaisy Mary Evans added a comment - I've cherry-picked them and added the PULL branches here. Cheers Mary
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Awesome thanks for clarifying that Mary - this has been integrated now!

              Show
              samhemelryk Sam Hemelryk added a comment - Awesome thanks for clarifying that Mary - this has been integrated now!
              Hide
              gerry Gerard Caulfield added a comment -

              Tested on master, 22 and 21 and all work fantastic, test passed and awesome fix.

              Show
              gerry Gerard Caulfield added a comment - Tested on master, 22 and 21 and all work fantastic, test passed and awesome fix.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Your nice code represents only 1/46 of the issues that have been sent upstream this week, so thanks, but not many.

              Nah, joking, many thanks! Closing this a fixed, ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Your nice code represents only 1/46 of the issues that have been sent upstream this week, so thanks, but not many. Nah, joking, many thanks! Closing this a fixed, ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    12/Mar/12