-
Notifications
You must be signed in to change notification settings - Fork 92
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
Scroll area for filters and search results #3808
base: master
Are you sure you want to change the base?
Conversation
fiskus
commented
Nov 30, 2023
•
edited
Loading
edited
- Changelog entry (skip if change is not significant to end users, e.g. docs only)
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #3808 +/- ##
==========================================
- Coverage 35.58% 35.55% -0.04%
==========================================
Files 709 709
Lines 31110 31142 +32
Branches 4651 4667 +16
==========================================
Hits 11071 11071
- Misses 18892 18924 +32
Partials 1147 1147
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
a8499f6
to
7f2dc31
Compare
the ux feels too complicated and clumsy. i believe we can make it simpler by doing away with the custom top/bottom indicators and scrollable containers |
What do you mean by scrollable containers? Containers for keywords filter? It's no longer scrollable, it has "Show more" button, and show the whole list without scroll Top/bottom indicators are additional buttons, just to show that something is hidden at top or at bottom. You can use scroll instead of buttons. The similar UX is in vertical tabs component https://v4.mui.com/components/tabs/#vertical-tabs |
nope
that's good
i don't think they are necessary and i believe they just clutter the UI. i meant smth like this, without too much custom logic and controls, just some css positioning stuff: |
553a514
to
cd41d4c
Compare
Rewrote layout using sticky app header |
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.
this feels solid. some cosmetic issues:
- the background of these inputs should be white i believe, probably some z-index issue
![Screenshot 2023-12-07 at 12 20 43](https://private-user-images.githubusercontent.com/122294/288743176-edcba71e-954f-43fb-b140-c840f7e8b26f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQzMTc2LWVkY2JhNzFlLTk1NGYtNDNmYi1iMTQwLWM4NDBmN2U4YjI2Zi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT02OTQ0Y2YxZjBmZjE0ZDIzNzhlMjczZWU5OWFmY2M5NDQyM2RkMWI2MTEyMTUyNzRlYzRlMmIzNGQ2MTYyMTI2JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.yB2KZgNUuTA1Iq1Sox-hG9W_ATVv--bbfyRFTcnMogs)
- filters section could use some margin at the bottom
![Screenshot 2023-12-07 at 12 16 50](https://private-user-images.githubusercontent.com/122294/288743791-7f7df84f-ede7-4b25-b933-70a9abd94e22.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQzNzkxLTdmN2RmODRmLWVkZTctNGIyNS1iOTMzLTcwYTlhYmQ5NGUyMi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1iYmYzMGMwNGQ1ZjE5YTliMzVhYmUzMzI5ZmZmNDU0NGEyNjZjZmYyZDUxZWMwMmNjOWE1ZWJlNTRjMzgxNTYyJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.c6RGd781B1XmnBMapNk2wF4yT5FBFbzSbH56iEvllOg)
- some broken layout in the enum control
![Screenshot 2023-12-07 at 12 24 08](https://private-user-images.githubusercontent.com/122294/288743965-63228628-fafb-4d3f-9ab8-b17d6eaa63f3.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQzOTY1LTYzMjI4NjI4LWZhZmItNGQzZi05YWI4LWIxN2Q2ZWFhNjNmMy5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT02YjhhNjEyNTcwMzY4MzI3NmNiZWI2ZDMzOTdjZTU2MDIwNTAwMmFkZGY5ODhkYmEyNTM2ZGY4NjFlNWMyMTA2JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.Md8KHrBJ1VIC0gmUAR6jaOzq2AN852WxsxNg4BWUTII)
tbh i liked the "enclosure" we had around the variants before in this place:
![Screenshot 2023-12-07 at 12 21 35](https://private-user-images.githubusercontent.com/122294/288744335-1b247841-93b6-4325-8238-2d3a29589734.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQ0MzM1LTFiMjQ3ODQxLTkzYjYtNDMyNS04MjM4LTJkM2EyOTU4OTczNC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0xZTA0OTJkZTk0ODU0ZDFiN2ViN2IxMWZhYjZhYTJjMTU1ZTJlODNiMmYyOTQwYTVlNDliODM0ZTA5MGI5YzFkJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.9Vl9nbIGFKLHDeVarnIOcmEwP_s4Qt2KJWLRz5VZc5c)
- sometimes the filters section gets h-scrolled
![Screenshot 2023-12-07 at 12 26 15](https://private-user-images.githubusercontent.com/122294/288744588-c9592e7f-2f15-4750-8a9b-35d4bd6f32fd.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQ0NTg4LWM5NTkyZTdmLTJmMTUtNDc1MC04YTliLTM1ZDRiZDZmMzJmZC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1iYjZkZGE4NjgzZTZkNDRlNGNmN2IxZDA2MWI2YTU2M2FmNTA2YTI3OTY5M2U4MGJmNTRiOTBmY2Q1MWY5ZWNiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.FRiFCUUyOQBSqVYi0qu3YRGOqVegEiIiQW-R6Tbg4og)
- toolbar gradient in both sections: scrolled contents are partially visible in the background:
![Screenshot 2023-12-07 at 12 15 18](https://private-user-images.githubusercontent.com/122294/288746076-3d307063-9e72-4b44-9024-116bf8b9d84f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQ2MDc2LTNkMzA3MDYzLTllNzItNGI0NC05MDI0LTExNmJmOGI5ZDg0Zi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1mYTM3MTI5M2M4YmE4NmRhZjA5YzUzZGZmMmUwNzExMjQ4NmFkMGU4ZGE0OTU2ZjU0OTU5YTA5Y2IzMDk0NzdmJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.kBmZVmlDa96cl-jHnM8vouKxAnueaf8ZOeZ-LI8l-t4)
![Screenshot 2023-12-07 at 12 17 48](https://private-user-images.githubusercontent.com/122294/288746247-5f160a25-5532-492c-8637-cb97e54ceb8a.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQ2MjQ3LTVmMTYwYTI1LTU1MzItNDkyYy04NjM3LWNiOTdlNTRjZWI4YS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT03OWY5OTA5Y2VlYzg2MTJkNWExODU0MWM3NTczOWYwY2U2ODUxMDRmZTlhNzRiYzI0NWQzMzQyYmJjZTFjMjlhJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.wTTw5gZ7v0h3c8huo2h4KHbkr7Sl9CQlIUFkuAe0udM)
i think the transparent part should start just below the toolbar contents:
![Screenshot 2023-12-07 at 12 15 10](https://private-user-images.githubusercontent.com/122294/288746105-0940b8db-8b4e-4f07-83f3-f95fff10558d.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQ2MTA1LTA5NDBiOGRiLThiNGUtNGYwNy04M2YzLWY5NWZmZjEwNTU4ZC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1jZDlkNTk5ODJmMGY2ZTI0ZWJkYzE1ZDBhMjE0ODMxZDZiZmUxNzk3ZTdjYTdmYTQ4YzNmMGY1YmZmYjM4NTY4JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.Ik-zd1Q7hqAkMJO7Z5jLwRVMoGHyCwz1iYQmmv62xGg)
- seems like there's some z-index issue here (preview bg is visible above the toolbar):
![Screenshot 2023-12-07 at 12 15 31](https://private-user-images.githubusercontent.com/122294/288746888-ae8e8d33-1c3f-448f-9627-88f807aeb7b4.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5MTU2NjMsIm5iZiI6MTcxOTkxNTM2MywicGF0aCI6Ii8xMjIyOTQvMjg4NzQ2ODg4LWFlOGU4ZDMzLTFjM2YtNDQ4Zi05NjI3LTg4ZjgwN2FlYjdiNC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzAyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwMlQxMDE2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1jYzA4ZDA5NThjYjI1MzExOTE5Y2UyMjQwYWU5NDg3Yzc3YzJkNTNiZWRmOTg1YmQzOTAyY2Y4Njc3YmNkZGRmJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.IdySKAZ4O29KA1X5_Z5p0W_WhCfm6sZHgZxarB_gaqw)
also, while you're at it, it might make sense to fix the enum options wrapping to multiple lines (simple white-space: nowrap would help) |
After reviewing:
I have reluctantly concluded that @fiskus should not spend any more time on this PR, so he can start working on MOAB deliverables with Dima. My rationale is that successfully shipping MOAB in January is more important than a highly-polished faceted search this month. @kevinemoore has already said that the observed cosmetic issues, though serious, are not blockers. The reality is that most customers will only install a December release in their 'staging' stacks, and may well provide feedback that obsolete portions of this PR. @nl0 I believe your options are:
Please let me know at Monday's D&D which way you are leaning. Thanks. |