Moodle
  1. Moodle
  2. MDL-32869

Ugly filter plugin table in formal_white needs rows resize

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.3
    • Component/s: Themes
    • Labels:
      None
    • Testing Instructions:
      Hide

      Go to Settings -> Site administration -> Plugins -> Filters -> Manage filters in formal_white.
      you are supposed to see a table very tall rows as in the second attached picture.
      Apply the patch.
      The table should have acceptable tall rows now.

      Show
      Go to Settings -> Site administration -> Plugins -> Filters -> Manage filters in formal_white. you are supposed to see a table very tall rows as in the second attached picture. Apply the patch. The table should have acceptable tall rows now.
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32869_master
    • Rank:
      39936

      Description

      Take a tranquilizer.
      Go to Settings -> Site administration -> Plugins -> Filters -> Manage filters in formal_white.
      I strongly feel that table's rows need resizing.

        Activity

        Hide
        Andrea Bicciolo added a comment -

        The update to theme formal_white looks fine to me.

        Show
        Andrea Bicciolo added a comment - The update to theme formal_white looks fine to me.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

        TIA and ciao

        Show
        Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
        Hide
        Dan Poltawski added a comment -

        Thanks, i've integrated this now.

        Show
        Dan Poltawski added a comment - Thanks, i've integrated this now.
        Hide
        Adrian Greeve added a comment -

        I followed the instructions and looked at the issue before and after the patch. I also tried it out in a few browsers to see if there was any difference there. Everything looks good.
        Thanks.

        Show
        Adrian Greeve added a comment - I followed the instructions and looked at the issue before and after the patch. I also tried it out in a few browsers to see if there was any difference there. Everything looks good. Thanks.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        U P S T R E A M I Z E D !

        Many thanks for the hard work, closing this as fixed.

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - U P S T R E A M I Z E D ! Many thanks for the hard work, closing this as fixed. Ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: