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

Search menu item is 4 pixels too tall

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: 3.1
    • Component/s: Global search, Themes
    • Labels:
    • Testing Instructions:
      Hide
      1. Enable global search (you should be able to just enable it, without configuring it).
      2. Confirm that the navbar is 40px tall (41 including the bottom border).
      3. Confirm that the search hide/reveal works correctly.

      This should be done in various browsers, and in a RTL language.

      Note: If MDL-54559 has not been integrated, the bar will be 42px tall because of a similar problem. You can add the patch from that and re-grunt to confirm the behavior.

      Show
      Enable global search (you should be able to just enable it, without configuring it). Confirm that the navbar is 40px tall (41 including the bottom border). Confirm that the search hide/reveal works correctly. This should be done in various browsers, and in a RTL language. Note: If MDL-54559 has not been integrated, the bar will be 42px tall because of a similar problem. You can add the patch from that and re-grunt to confirm the behavior.
    • Affected Branches:
      MOODLE_31_STABLE
    • Fixed Branches:
      MOODLE_31_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-54560_master

      Description

      The search menu item, due to padding and such, is 4 pixels too tall, and pushes the bottom of the bar down.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                emerrill Eric Merrill
                Reporter:
                emerrill Eric Merrill
                Peer reviewer:
                David Monllaó
                Integrator:
                Andrew Nicols
                Tester:
                Jun Pataleta
                Participants:
                Component watchers:
                Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Bas Brands, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  23/May/16