Uh oh!
There was an error while loading. Please reload this page.
Use KTransition instead of VSlideYTransition in 'Request more space' form - #5194
Conversation
radinamatic
commented
Jul 22, 2025
@yeshwanth235 Thank you for your contribution! ❤️ Please remember what we mentioned previously regarding the more descriptive PR titles, thank you! |
MisRob
commented
Jul 24, 2025
Hi @yeshwanth235, thanks! Code is as expected. For manual testing - after we merge #5210 (I expect that within next few days), would you rebase your pull request on top of the latest |
yeshwanth235
commented
Jul 24, 2025
Sure @MisRob |
MisRob
commented
Jul 25, 2025
@yeshwanth235#5210 is now merged |
yeshwanth235
commented
Jul 25, 2025
Update the PR @MisRob |
There was a problem hiding this comment.
Thanks @yeshwanth235, I confirm it looks as expected.
If you'd like to keep contributing to this project, you're welcome to medium complexity issues if you'd like - from few issues you've resolved recently I've had an impression you have built a good grasp on how KDS in relationship to Studio works.
Uh oh!
There was an error while loading. Please reload this page.
yeshwanth235
commented
Jul 28, 2025
Thanks @MisRob. Sure, will start contributing to medium complexity issues |
yeshwanth235
commented
Jul 28, 2025
Hi @MisRob |
Hi @yeshwanth235, I've just assigned last one. I have several in preparation though - they will all be posted in the next two days or so :) |
MisRob
commented
Jul 29, 2025
If you'd like @yeshwanth235, here's one that's available - I left a note that it's reserved for you, but first have a look if that's something you'd be interested in #5217 |
Summary
Updated VSlideYTransition to KTransition
References
Fixes#5081
Reviewer guidance
Manual testing
Check whether form is show on click of open form
Check whether form is getting closed on click of close form
Note: With the release of the new KDS version containing the KTransition fix, this PR is ready for merging