-
Notifications
You must be signed in to change notification settings - Fork 80
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
Library Home - Filter by Capa Problem Type #1196
Comments
@jmakowski1123 If it doesn't require any additional work, would you like us to implement this detailed capa filtering on the Studio course search modal too, or just in libraries? |
@jmakowski1123 @lizc577 @sdaitzman @marcotuts Just one question about the filter behavior: Is it necessary to add the filter related data in the type filter? Like this? (ref) |
@ChrisChV Can you clarify what you're asking? I'm not really sure what you mean. |
In the https://www.loom.com/share/2ad01436fa13478abec6dc3a2fb9e0a3?sid=ec3247a8-362d-4074-b874-668f0b599562 |
@ChrisChV Good question. Personally, I think it should appear there too. It would usually just be a number though e.g. "Problem (+2)" or "Multiple Choice (+3)" |
Thanks! I updated this on #1207 |
@jmakowski1123 @lizc577 @sdaitzman @marcotuts This is ready for AC testing on the sandbox You can test it with this library |
I noticed a minor inconsistency with the 'select all' functionality. If you select a few problem types, then click select all, then leave the dropdown. re-open the drop down and de-select some, the main "problem" filter will look as if none are selected. Screenshare.-.2024-09-09.6_04_43.PM.mp4 |
@jmakowski1123 @lizc577 @sdaitzman @marcotuts Fixed! You can check it on the sandbox. |
Looks good! |
Update the "Type" filter to allow optionally filtering by more specific capa problem type.
Note that this data is already in the search index.
The text was updated successfully, but these errors were encountered: