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

Search menu item is 4 pixels too tall

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.1
    • 3.1
    • Global search, Themes
    • MOODLE_31_STABLE
    • MOODLE_31_STABLE
    • MDL-54560_master
    • 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.

    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

          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-54559 Usermenu margin expands navbar

          • Minor - Minor loss of function where workaround is possible
          • Closed
          is a regression caused by

          Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. MDL-31989 Global Search 2

          • Major - Major loss of function, incorrect output
          • Closed

          Activity

            People

              emerrill Eric Merrill
              emerrill Eric Merrill
              David Monllaó David Monllaó
              Andrew Lyons Andrew Lyons
              Jun Pataleta Jun Pataleta
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona), Huong Nguyen, Bas Brands, Mathew May
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                23/May/16