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

Base & Standard themes custommenu vertical arrow overlaps menu item

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2, 2.3, 2.4
    • Fix Version/s: 2.2.5, 2.3.2
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide
      1. Select Standard theme via theme selector
      2. Make sure there is a custommenu set that has sub/sub-sub-menu items in Site Administration > Appearance > Themes > Theme settings (if necessary copy and paste the 'Example' given in that page).
      3. Next TEST that the vertical-arrow, visible only when hovering over a sub-menu item that already has a horizontal-arrow) is not overlapping the text of the menu item.
      Show
      Select Standard theme via theme selector Make sure there is a custommenu set that has sub/sub-sub-menu items in Site Administration > Appearance > Themes > Theme settings (if necessary copy and paste the 'Example' given in that page). Next TEST that the vertical-arrow, visible only when hovering over a sub-menu item that already has a horizontal-arrow) is not overlapping the text of the menu item.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull Master Branch:
      MDL-34957_master

      Description

      The vertical arrow, in BASE theme, overlaps the text menu item. This behaviour also affects Standard theme.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            lazydaisy Mary Evans added a comment -

            Attached please find small screen-shot of problem...

            Show
            lazydaisy Mary Evans added a comment - Attached please find small screen-shot of problem...
            Hide
            poltawski Dan Poltawski added a comment -

            Thanks Mary, i've integrated this to 22, 23 and master now.

            Show
            poltawski Dan Poltawski added a comment - Thanks Mary, i've integrated this to 22, 23 and master now.
            Hide
            rwijaya Rossiani Wijaya added a comment -

            This works great.

            Test passed.

            Show
            rwijaya Rossiani Wijaya added a comment - This works great. Test passed.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            YEAR!*

            CAF*, TOT!*

            • Your effort amazingly resulted. (unbelievable :-P)
            • Closing as fixed.
            • Tons of thanks.
            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - YEAR!* CAF*, TOT!* Your effort amazingly resulted. (unbelievable :-P) Closing as fixed. Tons of thanks.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Sep/12