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

Accessibility ergonomy filtering #11634

Merged

Conversation

littlesvensson
Copy link
Contributor

Fixes #11621

Description

Used bootstrap classes and some JS logic for filtering. The fields are now aligned so its easier to orientate and keep responsivity at the same time. Clear button is disabled in case no filter is set in the fields.

Before:
Screenshot 2025-01-23 at 11 29 22

After:
Screenshot 2025-01-23 at 11 23 12

smaller screen
Screenshot 2025-01-23 at 11 23 38

smaller screen with at least one filter option
Screenshot 2025-01-23 at 11 23 50

@littlesvensson littlesvensson marked this pull request as ready for review January 24, 2025 10:25
Copy link
Contributor

@Maffooch Maffooch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow this is such a tremendous improvement!

Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SO VERY Approved

@Maffooch Maffooch merged commit 0e91248 into DefectDojo:bugfix Jan 29, 2025
72 of 73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants