Search Icon in Mobile View is slightly detached #4952

New Issue

OVERLORD · 2026-02-05T09:29:01+03:00

OVERLORD commented

2026-02-05 09:29:01 +03:00

Originally created by @whoamiafterall on GitHub (Sep 17, 2024).

Describe the Bug

This happens only in mobile view of Search Results.

Steps to Reproduce

Go to Search
Search something
Click Inspect and mobile view or resize window to mobile
See the Problem occur

Expected Behaviour

The Icon should stay inside the Search Bar.

Screenshots or Additional Context

No response

Browser Details

Firefox 130.0 (64.bit)

Exact BookStack Version

v24.05.4

Originally created by @whoamiafterall on GitHub (Sep 17, 2024). ### Describe the Bug ![Bildschirmfoto vom 2024-09-17 19-17-44](https://github.com/user-attachments/assets/5ad3dc70-d355-4155-804b-104769ce465e) This happens only in mobile view of Search Results. ### Steps to Reproduce 1. Go to Search 2. Search something 3. Click Inspect and mobile view or resize window to mobile 4. See the Problem occur ### Expected Behaviour The Icon should stay inside the Search Bar. ### Screenshots or Additional Context _No response_ ### Browser Details Firefox 130.0 (64.bit) ### Exact BookStack Version v24.05.4

OVERLORD added the 🐛 Bug label 2026-02-05 09:29:01 +03:00

OVERLORD closed this issue

2026-02-05 09:29:03 +03:00

OVERLORD commented

2026-02-05 09:29:05 +03:00

@ninaa-e commented on GitHub (Sep 18, 2024):

We solved this issue for us by adding an tabindex="-1" in all.blade.php to the Button/Searchicon element.
The line looks like this in the End <button tabindex="-1" type="submit">@icon('search')</button>

The reason this works is, that if this is added, a css rule that the searchbar in the header uses will apply here.

@ninaa-e commented on GitHub (Sep 18, 2024): We solved this issue for us by adding an `tabindex="-1" ` in all.blade.php to the Button/Searchicon element. The line looks like this in the End `<button tabindex="-1" type="submit">@icon('search')</button>` The reason this works is, that if this is added, a css rule that the searchbar in the header uses will apply here.

OVERLORD commented

2026-02-05 09:29:07 +03:00

@ssddanbrown commented on GitHub (Sep 27, 2024):

Thanks for reporting @whoamiafterall, and thanks @ninaa-e for sharing the solution.
This has now been addressed in 42264f402d to be part of the next feature release.

@ssddanbrown commented on GitHub (Sep 27, 2024): Thanks for reporting @whoamiafterall, and thanks @ninaa-e for sharing the solution. This has now been addressed in 42264f402d2733fdf7e1260c6731455d85da12b1 to be part of the next feature release.

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#4952