Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search and search filter UI tweaks #14381

Merged
merged 9 commits into from
Oct 16, 2024
Merged

Search and search filter UI tweaks #14381

merged 9 commits into from
Oct 16, 2024

Conversation

hawkeye217
Copy link
Collaborator

Proposed change

  • Fix bug preventing search type switches from changing
  • Add selected style to more filters button when more filters are in use
  • Fix reset button
  • Fix scrollbar on labels to be on the outer div
  • Remove modal from sheet so that scrolling can still happen on the search results with the filters sheet open
  • Add hover colors to thumbnail footer icons
  • Better match font size of more filters headings to other parts of the UI
  • Stop sheet from displaying console errors

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • The code has been formatted using Ruff (ruff format frigate)

Copy link

netlify bot commented Oct 16, 2024

Deploy Preview for frigate-docs canceled.

Name Link
🔨 Latest commit 8d83d28
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/670f426b5ea9ca000827e433

@NickM-27 NickM-27 merged commit eda52a3 into dev Oct 16, 2024
11 of 13 checks passed
@NickM-27 NickM-27 deleted the filter-ui-tweaks branch October 16, 2024 12:15
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.

2 participants