knossos icon indicating copy to clipboard operation
knossos copied to clipboard

Search: Advert position swaps place when filter menu is opened

Open MMK21Hub opened this issue 2 years ago • 3 comments

Describe the bug

On mobile viewports, filters can be used by pressing the Filters... button to "open" the card containing the filters.

When this card is closed, the advert (tagged with the class GBBNWLJVGRHFLYVGSZKSSKNTHFYXHMBD) appears below the search controls. However, when the filters are open, the advert is shown above the search controls.

This upsets the expected order of UI elements, and also stops the focus order matching the visual order of elements.

Steps to reproduce

  1. Use a viewport less than 1024px wide
  2. Go to https://modrinth.com/mods → :heavy_check_mark: The advert is above the search controls
  3. Click the Filters... button → :x: The advert is now below the search controls

Expected behavior

The advert would stay above the search controls (the top-to-bottom order would be filters card, advert, search controls, pagination, search results)

Screenshots

Expected behaviour (when filters aren't shown)

Cropped screenshot showing the advert placed above the search filters

Unexpected behaviour (when filters are shown)

Cropped screenshot showing the advert placed below the search filters. The "Filters..." button is highlighted green to indicate its active state

MMK21Hub avatar Jun 15 '23 12:06 MMK21Hub

Discovered while testing #1165

MMK21Hub avatar Jun 15 '23 12:06 MMK21Hub

Does this still happen for you, I can't reproduce

Minenash avatar Oct 05 '23 06:10 Minenash

I'm still able to reproduce this on the production site, as well as staging.modrinth.com (commit fcfa508). The visual inconsistency and the incorrect tab order are both still present. A screenshot of https://modrinth.com/mods, with a blue box containing a sponsored link to BisectHosting, displayed below the card that contains the search controls

MMK21Hub avatar Oct 07 '23 16:10 MMK21Hub