KDS to Studio: Replace ResponsiveDialog by KModal in SavedSearchesModal#3063
Conversation
| </VList> | ||
| </KModal> | ||
|
|
||
| <MessageDialog |
There was a problem hiding this comment.
I haven't replaced MessageDialog and EditSearchModal with KModal because @marcellamaki already has some work in progress related to that.
marcellamaki
left a comment
There was a problem hiding this comment.
This looks good to me! Nice work Misha :)
|
Tested today at https://hotfixes.studio.learningequality.org/ and the Saved searches modal is displayed correctly. Currently the saved searches are not the same for my different channels for example in Channel A I might have Search 1 and Search 2 while in Channel 2 I might have no saved searches at all or totally different saved searches. Is this intentional? |
This does not seem to be the case on my side, the same list of all my saved searches (plus the duplicates 😛) appears in all the channels. |
Yeah, I'm not able to replicate that issue today as well. How bizarre... :) |
Summary
Description of the change(s) you made
Replace
ResponsiveDialogbyKModalinSavedSearchesModalManual verification steps performed
Screenshots
Does this introduce any tech-debt items?
Nothing that I am aware of regarding Studio code.
I've discovered a bug in KDS that has been reported here.
Reviewer guidance
How can a reviewer test these changes?
Please follow "Manual verification steps"
Are there any risky areas that deserve extra testing?
Please check that you can edit and remove saved searches
References
Part of learningequality/kolibri-design-system#157
Contributor's Checklist
PR process:
CHANGELOGlabel been added to this PR. Note: items with this label will be added to the CHANGELOG at a later timedocslabel has been added if this introduces a change that needs to be updated in the user docs?requirements.txtfiles also included in this PRStudio-specifc:
notranslateclass been added to elements that shouldn't be translated by Google Chrome's automatic translation feature (e.g. icons, user-generated text)pages,components, andlayoutsdirectories as described in the docsTesting:
Reviewer's Checklist
This section is for reviewers to fill out.
yarnandpip)