-
Notifications
You must be signed in to change notification settings - Fork 44
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
✨ Add search toolbar and "all questions" tab to questionnaire view page, clean up layout #1313
✨ Add search toolbar and "all questions" tab to questionnaire view page, clean up layout #1313
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #1313 +/- ##
=======================================
Coverage 43.04% 43.04%
=======================================
Files 145 145
Lines 4330 4330
Branches 999 999
=======================================
Hits 1864 1864
Misses 2454 2454
Partials 12 12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
8e5fa98
to
7e3e829
Compare
Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>
Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>
Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>
Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>
Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>
Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>
d4c3914
to
8d54cd3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm - looks great!
…ge, clean up layout (konveyor#1313) Resolves konveyor#1293. We determined we don't need the filter dropdown (at least for now), just the search bar. * Cleans up some layout issues on the page. * Adds an "All questions" tab above the section tabs. When viewing this tab, the questions table has a "Section" column so you can still tell the sections apart when they are viewed together. * Adds the total number of questions under each tab title. When the search bar is used, a badge appears next to this to show how many questions match the search. * Adds an empty state to the table if there are no questions matching your search. <img width="1500" alt="Screenshot 2023-08-25 at 2 24 18 PM" src="https://github.com/konveyor/tackle2-ui/assets/811963/c655ba4e-4a18-493a-87ad-ff1e3c926da3"> ![U4kHZuOvzh](https://github.com/konveyor/tackle2-ui/assets/811963/3698dc9e-5841-44e4-9fee-40752b4def86) <img width="1501" alt="Screenshot 2023-08-25 at 2 31 19 PM" src="https://github.com/konveyor/tackle2-ui/assets/811963/27e547c8-76dc-4398-9a2f-84cd9da64e8c"> --------- Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>
Resolves #1293. We determined we don't need the filter dropdown (at least for now), just the search bar.