Fix: Show search & setting buttons at channel level in quiz resource selection #13198
Conversation
github-actions
bot
added
APP: Coach
Build Artifacts
|
nucleogenesis
changed the title
|
Hi @nucleogenesis - could you have a look while the build assets are not generated? |
|
Re-run triggered, should be good to go now (only APK is still building) @pcenov 🙂 |
|
Hi @nucleogenesis - unfortunately now if I have bookmarked resources then the "Settings" and "Search" buttons are duplicated:
|
| @@ -19,6 +19,25 @@ | |||
| </div> | |||
| </template> | |||
| <template #default="{ isScrolled }"> | |||
| <QuizResourceSelectionHeader | |||
There was a problem hiding this comment.
Hi @nucleogenesis! I think this should be solved in the context of the SelectionIndex component, there we already have the buttons and the search button handler, the only thing is that we dont show them when bookmarksCount === 0.
Adding this here makes that when we have bookmarks we see these buttons duplicated
And this issue also happens with lessons too, so it should be fixed there too. Fixing this in the SelectionIndex component should fix it for both of them because we use this component for lessons and quizzes 👐 .
There was a problem hiding this comment.
Whoops, seems like @pcenov and I were reviewing it at the same time 😆
There was a problem hiding this comment.
We are doubling down on everything! 😄
Summary
Uses
QuizResourceSelectionHeaderon the resource selection side panel's index (aka channels listing page) to display the filters & search buttons.References
Fixes #13144
Reviewer guidance
You should see Settings & Search filters at the top of the page listing the channels. They should work as expected.