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

The new versions layout seems a bit ugly

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

      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

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              poltawski Dan Poltawski added a comment -

              Attaching screenshot of what I mean

              Show
              poltawski Dan Poltawski added a comment - Attaching screenshot of what I mean
              Hide
              mudrd8mz David Mudrák added a comment -

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

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

              Submitting a trivial CSS patch for integration.

              Show
              mudrd8mz David Mudrák added a comment - Submitting a trivial CSS patch for integration.
              Hide
              poltawski Dan Poltawski added a comment -

              Thanks David, i've integrated this now

              Show
              poltawski Dan Poltawski added a comment - Thanks David, i've integrated this now
              Hide
              stronk7 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
              stronk7 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:
                    Fix Release Date:
                    25/Jun/12