Skip to content
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

[DO NOT MERGE, FOR QA ONLY] Test KSelect refactor #12079

Closed
wants to merge 1 commit into from

Conversation

MisRob
Copy link
Member

@MisRob MisRob commented Apr 11, 2024

Summary

learningequality/kolibri-design-system#549 introduces internal KSelect refactor. I previewed a few places in Kolibri, but it's a significant refactoring so @pcenov could you please do a more thorough test run of places in Kolibri where we use selects (https://design-system.learningequality.org/kselect/)?

The main focus of QA at this point is just confirming that something major is not broken in regards to selects functionality. There's no need to report issues with colors, etc. because later I will merge rebrand work with this select work and there will be final test run.

Thanks so much.

@pcenov
Copy link
Member

pcenov commented Apr 15, 2024

Hi @MisRob the only issue I was able to identify is that most of the drop-downs are now opening upwards instead of downwards:

drop-downs.mp4

@MisRob
Copy link
Member Author

MisRob commented Apr 15, 2024

Thank you, @pcenov. I've just tried whether the issue is present on my rebrand PR which doesn't contain any KDS updates tested in this PR and the bug is there too. So I think it's either caused by some recent KDS releases or perhaps it was there even before. I will report as separate issue.

Screenshot from 2024-04-15 17-55-09

@MisRob
Copy link
Member Author

MisRob commented Jul 26, 2024

^ Above issue reported here learningequality/kolibri-design-system#690

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants