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

Link to Issue:

Place the search bar on the search results view below the navbar (#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 ## Link to Issue: [Place the search bar on the search results view below the navbar (#97)](https://gitea.jcloud-services.ddns.net/Seekra/frontend/issues/97)
johannes.vos added this to the responsive-design-ready milestone 2026-06-01 17:11:31 +02:00
johannes.vos added the
type
bug
area
search-results
prio
critical
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 MVP Frontend 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 MVP Frontend 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 MVP Frontend 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

Pull request closed

Sign in to join this conversation.