Moodle
  1. Moodle
  2. MDL-36544

Remove special border-radius on course/site description block, standard theme

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.6, 2.3.3, 2.4
    • Fix Version/s: 2.4
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      Test Instructions

      1. Navigate to Site Administration > Development > Purge all caches and Purge all caches
      2. Turn editing on
      3. Add the Site/course description block to front page or course
      4. Edit the block and add a little content to it (if you're on the front page this will send you to edit the front page description in the front page settings)
      5. Turn editing off
      6. Site/course description block should have the same border-radius as the rest of the side blocks
      Show
      Test Instructions Navigate to Site Administration > Development > Purge all caches and Purge all caches Turn editing on Add the Site/course description block to front page or course Edit the block and add a little content to it (if you're on the front page this will send you to edit the front page description in the front page settings) Turn editing off Site/course description block should have the same border-radius as the rest of the side blocks
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
      git@github.com:amygroshek/moodle.git
    • Pull Master Branch:
      wip-MDL-36544-m24
    • Rank:
      46015

      Description

      The course/site description block inhabits the side block areas just like other blocks, but it is given a 10px border-radius in the standard theme (other blocks have no border-radius). On sites where this block is used I am often asked to override the 10px border-radius to make this block look like the others. Generally speaking, a site will look more professional if border-radius is consistent across all elements of a given type (in this case, blocks).

      I realize that this block is special, and the developers want to accentuate it, but it seems like we would be able to leave it to theme developers to decide how they want to accentuate it, and otherwise leave border-radius the same as other blocks. (There is another aspect of this block's appearance that makes it special: this block has no header. So it still looks distinct without the special border-radius.)

        Activity

        Hide
        Mary Evans added a comment - - edited

        It looks fine, so I have set this for Integration Review, but it will also need to be Cherry Picked into Master.

        Show
        Mary Evans added a comment - - edited It looks fine, so I have set this for Integration Review, but it will also need to be Cherry Picked into Master.
        Hide
        Amy Groshek added a comment -

        Thanks Mary. Cherry-pick from 1 repo to another is beyond me, but I have created a new branch with the change for master.

        Show
        Amy Groshek added a comment - Thanks Mary. Cherry-pick from 1 repo to another is beyond me, but I have created a new branch with the change for master.
        Hide
        Dan Poltawski added a comment -

        Thanks Mary/Amy.

        I've integrated this to 2.4 only as its technically an improvement and also its not nice to change the appearance mid way through the stable branch. (though very minor)

        Show
        Dan Poltawski added a comment - Thanks Mary/Amy. I've integrated this to 2.4 only as its technically an improvement and also its not nice to change the appearance mid way through the stable branch. (though very minor)
        Hide
        Amy Groshek added a comment -

        Super! Thank you Mary and Dan!

        Show
        Amy Groshek added a comment - Super! Thank you Mary and Dan!
        Hide
        Petr Škoda added a comment -

        Confirming, the block radius is gone both in course and on the frontpage. Thanks.

        Show
        Petr Škoda added a comment - Confirming, the block radius is gone both in course and on the frontpage. Thanks.
        Hide
        Dan Poltawski added a comment -

        Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week!

        ciao
        Dan

        Show
        Dan Poltawski added a comment - Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week! ciao Dan

          People

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

            Dates

            • Created:
              Updated:
              Resolved: