Skip to content

Conversation

Prashant-thakur77
Copy link
Contributor

Fixes #5421

Summary

Replaced Vuetify VBtn elements with Kolibri Design System KButton elements in trash and folder creation modals. Added KButtonGroup containers to maintain proper element spacing and applied CSS to prevent button element wrapping on mobile screens.

Desktop Users

Successfully replaced Vuetify VBtn elements with Kolibri Design System KButton elements in trash and folder creation modals. Added KButtonGroup containers to maintain proper element spacing and applied CSS to prevent button element wrapping on mobile screens
Screenshot From 2025-10-03 14-58-06
Screenshot From 2025-10-03 15-09-49
image

Mobile Users

Screenshot From 2025-10-03 15-17-39 Screenshot From 2025-10-03 14-57-33

References

• Parent issue: #5060

Reviewer guidance

Login as [email protected] with password a
Go to Channels > Published Channel
Click Add > New folder and create the folder
Go back to the channel page
Select the folder
Click Remove
Open trash from the ... dropdown at the top of the page

@MisRob
Copy link
Member

MisRob commented Oct 3, 2025

Thank you @Prashant-thakur77, we will assign a reviewer next week.

@LianaHarris360 LianaHarris360 self-assigned this Oct 6, 2025
@Prashant-thakur77
Copy link
Contributor Author

hey @LianaHarris360 ,do tell if any changes are required.I will update accordingly.

@Prashant-thakur77 Prashant-thakur77 force-pushed the remove-vuetify-buttons-trash branch from 4620da3 to fd33d5b Compare October 18, 2025 10:28
@Prashant-thakur77 Prashant-thakur77 deleted the remove-vuetify-buttons-trash branch October 18, 2025 10:40
@Prashant-thakur77 Prashant-thakur77 restored the remove-vuetify-buttons-trash branch October 18, 2025 10:40
@MisRob
Copy link
Member

MisRob commented Oct 20, 2025

Hi @Prashant-thakur77, we saw your note and plan to review. Thanks.

@MisRob MisRob requested a review from LianaHarris360 October 20, 2025 11:00
Copy link
Member

@LianaHarris360 LianaHarris360 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this @Prashant-thakur77. My apologies for the delayed review. When testing the changes on mobile, the buttons at the bottom of the TrashModal stack awkwardly, the .button-group styling isn't being applied. Everything else looks good and works correctly.

@LianaHarris360 LianaHarris360 dismissed their stale review October 20, 2025 17:06

No longer applicable

Copy link
Member

@LianaHarris360 LianaHarris360 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes in this PR are correct and caused no regressions.

@LianaHarris360 LianaHarris360 merged commit 9190969 into learningequality:unstable Oct 20, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Remove Vuetify from Studio] Buttons in New folder and Trash

3 participants