Moodle
  1. Moodle
  2. MDL-32339

The new versions layout seems a bit ugly

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Administration, Themes
    • Labels:
    • Testing Instructions:
      Hide

      I really believe that this trivial CSS change does not require proper testing. Anyway, if you wanted to test it:

      1. Prepare an environment in which there is Moodle core update available
      2. Resize the browser window so that the release information needs more lines to display
      3. TEST: Make sure that the second line does not touch the highlighted area at the first line.

      Show
      I really believe that this trivial CSS change does not require proper testing. Anyway, if you wanted to test it: 1. Prepare an environment in which there is Moodle core update available 2. Resize the browser window so that the release information needs more lines to display 3. TEST: Make sure that the second line does not touch the highlighted area at the first line.
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32339-css-updates
    • Rank:
      39162

      Description

      Seems like you have to have a very wide screen for it to display in one row, and if you do not then the colours bleed between rows.

      Attaching screenshot

        Issue Links

          Activity

          Hide
          Dan Poltawski added a comment -

          Attaching screenshot of what I mean

          Show
          Dan Poltawski added a comment - Attaching screenshot of what I mean
          Hide
          David Mudrak added a comment -

          Attaching the after-patch.png to illustrate how the same screen looks in my Firefox after the fix.

          Show
          David Mudrak added a comment - Attaching the after-patch.png to illustrate how the same screen looks in my Firefox after the fix.
          Hide
          David Mudrak added a comment -

          Submitting a trivial CSS patch for integration.

          Show
          David Mudrak added a comment - Submitting a trivial CSS patch for integration.
          Hide
          Dan Poltawski added a comment -

          Thanks David, i've integrated this now

          Show
          Dan Poltawski added a comment - Thanks David, i've integrated this now
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

          Thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: