Fix(search): pleace search bar below navbar #98

Closed
johannes.vos wants to merge 1 commits from bugfix/search-bar-below-navbar into main
Member

Search bar below navbar (Bugfix)

I realize that the commit isn't safe; I'll fix the problem right away.

Goal

Move the Search bar below the navbar. -> when under 1000px.

Changes

Navbar.vue

  1. Added @media (max-width: 1000px) to hide search bar from navbar on small screens

SearchResultsView.vue

  1. Imported Searchbar component and added state management for search query
  2. Added .search-bar-container below navbar, hidden by default
  3. Added @media (max-width: 1000px) to show search bar container on small screens only
  4. Styling includes proper spacing, centering, and border separator

Related issues

Closes: #97

# Search bar below navbar (Bugfix) **I realize that the commit isn't safe; I'll fix the problem right away.** ## Goal Move the Search bar below the navbar. -> when under 1000px. ## Changes ### `Navbar.vue` 1. Added @media (max-width: 1000px) to hide search bar from navbar on small screens ### `SearchResultsView.vue` 1. Imported Searchbar component and added state management for search query 2. Added .search-bar-container below navbar, hidden by default 3. Added @media (max-width: 1000px) to show search bar container on small screens only 4. Styling includes proper spacing, centering, and border separator ## Related issues Closes: #97
johannes.vos added this to the responsive-design-ready milestone 2026-06-01 17:11:31 +02:00
johannes.vos added the Kind/BugArea/Search Results
Priority
Critical
1
labels 2026-06-01 17:11:31 +02:00
johannes.vos self-assigned this 2026-06-01 17:11:31 +02:00
johannes.vos added this to the Project Board project 2026-06-01 17:11:31 +02:00
johannes.vos added 1 commit 2026-06-01 17:11:31 +02:00
johannes.vos moved this to Review in Project Board on 2026-06-01 17:14:37 +02:00
jakob.scheid reviewed 2026-06-01 17:34:31 +02:00
@@ -93,2 +94,4 @@
width: 70%;
}
@media (max-width: 1000px) {
Owner

That is not a major device width break.

That is not a major device width break.
Author
Member

It‘s the critical width where it look bad

It‘s the critical width where it look bad
Owner

Please use 67.5rem (the width where tablets begin)

Please use `67.5rem` (the width where tablets begin)
jakob.scheid moved this to Done in Project Board on 2026-06-01 18:03:49 +02:00
jakob.scheid closed this pull request 2026-06-01 18:03:54 +02:00
johannes.vos reopened this pull request 2026-06-01 18:16:23 +02:00
jakob.scheid reviewed 2026-06-01 18:31:43 +02:00
@@ -37,0 +60,4 @@
border-bottom: 1px solid var(--border);
}
@media (max-width: 1000px) {
Owner

Please use 67.5rem (the width where tablets begin)

Please use `67.5rem` (the width where tablets begin)
johannes.vos closed this pull request 2026-06-01 18:44:24 +02:00
jakob.scheid changed title from Search bar below navbar to Fix: pleace search bar below navbar 2026-07-20 12:14:34 +02:00
jakob.scheid changed title from Fix: pleace search bar below navbar to Fix(search): pleace search bar below navbar 2026-07-20 12:27:24 +02:00
jakob.scheid modified the milestone from responsive-design-ready to 0.1.0 2026-07-22 11:36:56 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Seekra/frontend#98