Moodle
  1. Moodle
  2. MDL-36490

Standard theme border-radius on .generalbox

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.5, 2.3.2
    • Fix Version/s: 2.4.1
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      Test Instructions

      1. Navigate to Site Administration > Development > Purge all caches and Purge all caches
      2. View the div#notice element on the Purge all caches page, itself a div.generalbox, and note that bottom left and bottom right corners now have the same border-radius as top right and top left corners.
        1. Alternatively, navigate to Site Administration > Notifications; there are usually a few div.generalbox elements there upon which you can verify the change.
        2. Alternatively, view or create a Web page resource and then view the resource. The content is presented inside a div.generalbox. All corners should have the same border-radius.
      Show
      Test Instructions Navigate to Site Administration > Development > Purge all caches and Purge all caches View the div#notice element on the Purge all caches page, itself a div.generalbox, and note that bottom left and bottom right corners now have the same border-radius as top right and top left corners. Alternatively, navigate to Site Administration > Notifications; there are usually a few div.generalbox elements there upon which you can verify the change. Alternatively, view or create a Web page resource and then view the resource. The content is presented inside a div.generalbox. All corners should have the same border-radius.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull 2.4 Branch:
      MDL-36490_M24
    • Pull Master Branch:
      MDL-36490_master
    • Rank:
      45946

      Description

      The standard theme puts a border-radius on .generalbox objects which has a different radius on the bottom corners than the top corners. In some contexts this looks fine, and it makes sense why this was done. In some cases, especially when there is more than one .generalbox in the main content area, it looks kind of strange.

        Activity

        Hide
        Amy Groshek added a comment -

        As-is screenshot

        Show
        Amy Groshek added a comment - As-is screenshot
        Hide
        Amy Groshek added a comment -

        Screenshot with border-radius universal 3px

        Show
        Amy Groshek added a comment - Screenshot with border-radius universal 3px
        Hide
        Amy Groshek added a comment - - edited

        I did not add the vendor prefixes. They aren't necessary for recent versions of Webkit or Moz. IE9 also uses plain old border-radius.

        Show
        Amy Groshek added a comment - - edited I did not add the vendor prefixes. They aren't necessary for recent versions of Webkit or Moz. IE9 also uses plain old border-radius.
        Hide
        Mary Evans added a comment -

        Thanks Amy, I'll see what I can do.

        Show
        Mary Evans added a comment - Thanks Amy, I'll see what I can do.
        Hide
        Amy Groshek added a comment -

        Hi Mary! Thank you!

        Show
        Amy Groshek added a comment - Hi Mary! Thank you!
        Hide
        Mary Evans added a comment - - edited

        Hi Amy,
        If you have a few minutes to spare can you add some basic Test Instructions as I'm not too sure how you got that page with all the different headings which does show off the odd looking boxes!

        Thanks

        Show
        Mary Evans added a comment - - edited Hi Amy, If you have a few minutes to spare can you add some basic Test Instructions as I'm not too sure how you got that page with all the different headings which does show off the odd looking boxes! Thanks
        Hide
        Mary Evans added a comment -

        Re-based and new M24 branch added.

        Show
        Mary Evans added a comment - Re-based and new M24 branch added.
        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
        Rajesh Taneja added a comment -

        Thanks for fixing this Mary,

        All borders are same on div.generalbox box.

        Show
        Rajesh Taneja added a comment - Thanks for fixing this Mary, All borders are same on div.generalbox box.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Changes are now upstream, thanks for your collaboration!

        If you are going to have any celebration next days, enjoy with your gang, if not, too!

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Changes are now upstream, thanks for your collaboration! If you are going to have any celebration next days, enjoy with your gang, if not, too! Ciao
        Hide
        Amy Groshek added a comment -

        Thanks @Eloy! You have a great holidays too! I just pulled the new updates and I'm so excited to see this change. Looks great.

        Show
        Amy Groshek added a comment - Thanks @Eloy! You have a great holidays too! I just pulled the new updates and I'm so excited to see this change. Looks great.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: