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

The "Add..." drop down menu of the "Add a block" pseudo-block is not well placed in formal_white.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.6, 2.4.3
    • Fix Version/s: 2.3.8, 2.4.5, 2.5.1
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      1. Turn editing on.
      Look at the "Add a block" pseudo-block.
      Look at the attachments to verify the patch really works.

      2. Change the formal_white setting [[blockcolumnwidth]] and redo from start.
      Look at the attachments to verify the patch really works.

      Show
      1. Turn editing on. Look at the "Add a block" pseudo-block. Look at the attachments to verify the patch really works. 2. Change the formal_white setting [ [blockcolumnwidth] ] and redo from start. Look at the attachments to verify the patch really works.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-39198_master

      Description

      The width of the "Add..." drop down menu of the "Add a block" pseudo-block is bigger than the block width with some value for the FW setting blockcolumnwidth. Further more, the drop down menu browses in the block content as a ship in the ocean.

      Attached are two set of screen shots of the problem and of the expected solution with two different [[blockcolumnwidth]].

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Again, something like this would be nice in the Bootstrap theme.

            Show
            salvetore Michael de Raadt added a comment - Again, something like this would be nice in the Bootstrap theme.
            Hide
            poltawski Dan Poltawski added a comment -

            Hi Daniele,

            This appears to be conflicting on master. Plese could you resolve the conflicts and resubmit? Thanks!

            Show
            poltawski Dan Poltawski added a comment - Hi Daniele, This appears to be conflicting on master. Plese could you resolve the conflicts and resubmit? Thanks!
            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
            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
            daniss Daniele Cordella added a comment -

            Submitting for peer review as the patch has been modified to remove conflicts found by Dan.

            Show
            daniss Daniele Cordella added a comment - Submitting for peer review as the patch has been modified to remove conflicts found by Dan.
            Hide
            poltawski Dan Poltawski added a comment -

            Integrated to master, 25, 24 and 23. Thanks Daniele

            Show
            poltawski Dan Poltawski added a comment - Integrated to master, 25, 24 and 23. Thanks Daniele
            Hide
            markn Mark Nelson added a comment -

            Works as expected, passing.

            Show
            markn Mark Nelson added a comment - Works as expected, passing.
            Hide
            markn Mark Nelson added a comment -

            Hi Daniele, thanks for contributing to Moodle.

            Show
            markn Mark Nelson added a comment - Hi Daniele, thanks for contributing to Moodle.
            Hide
            daniss Daniele Cordella added a comment -

            It is a pleasure!
            Thank you all.

            Show
            daniss Daniele Cordella added a comment - It is a pleasure! Thank you all.
            Hide
            damyon Damyon Wiese added a comment -

            Thanks for your contribution! This issue has been reviewed, integrated, tested and now released to everyone.

            Closing as Fixed!

            Show
            damyon Damyon Wiese added a comment - Thanks for your contribution! This issue has been reviewed, integrated, tested and now released to everyone. Closing as Fixed!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  8/Jul/13