Skip to content

fix(advanced-search): improve filter layout (#353)#355

Open
paul43210 wants to merge 3 commits intoowncloud:mainfrom
paul43210:fix-advanced-search-layout
Open

fix(advanced-search): improve filter layout (#353)#355
paul43210 wants to merge 3 commits intoowncloud:mainfrom
paul43210:fix-advanced-search-layout

Conversation

@paul43210
Copy link
Contributor

Summary

  • Dropdowns (Type, Media Type, Tags) now display side-by-side in a flex row that wraps on narrow screens
  • Range fields (Size, Modified, ISO, Aperture, Focal Length) use vertical stacking (min above max with "to" separator) instead of horizontal layout
  • Range pairs arranged in a responsive grid (2 columns when space allows)
  • Text inputs (Name, Content, Caption, Object Detection) remain full-width between the two groups

Closes #353

Test plan

  • Verify dropdowns display side-by-side in Standard Filters section
  • Verify dropdowns display side-by-side in Photo / EXIF Filters section
  • Verify range fields show min above max (vertical stacking)
  • Verify layout wraps properly on narrow screens
  • Verify all filter inputs still function correctly

🤖 Generated with Claude Code

Dropdowns (Type, Media Type, Tags) now display side-by-side in a flex
row. Range fields (Size, Modified, ISO, Aperture, Focal Length) use
vertical stacking (min above max) in a responsive grid. Text inputs
remain full-width between the two groups.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mmattel
Copy link
Contributor

mmattel commented Mar 19, 2026

Good start, needs improvement 😅

These things catch my eyes:

  1. There is no space between the range blocks making this hard to read / differenciate.
  2. Range fields are very wide, too wide imho. They have been shorter before.
  3. The input boxes in range fields are not properly aligned /having the same height
  4. I am thinking what if the to devider is not centered but left aligned, same as the range header text.
  5. Selector boxes should be closer together. Their distance should be as wide as the opened selector box is
  6. I like how Name and Content is rendered. Each having an own line and full width
image image image

- Reduce gap between dropdown selectors (0.75rem → 0.5rem)
- Cap dropdown max-width at 220px
- Narrow range fields (max 220px) with more spacing between them (1rem)
- Align input heights (2.125rem) with box-sizing
- Left-align "to" separator to match label text

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mmattel
Copy link
Contributor

mmattel commented Mar 22, 2026

Great fix, this looks awsome 👍
This is really just fine tuning, what about if there is a very little more vertical space between the drop down selectors (when closed ) and the range items? This would make the selector more distinct to the range item(s). e.g. Type and Name are quite close together. The vertical space for the range items looks good.
image

… and range items

Adds margin below dropdown selector rows to visually distinguish them
from the range filter items below, per Martin's review feedback.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@paul43210
Copy link
Contributor Author

Good catch — added a bit more vertical spacing (margin-bottom) below the dropdown selector rows to better distinguish them from the range items below. Pushed in 24515a7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve advanced search layout

2 participants