Skip to content

Conversation

@Prashant-thakur77
Copy link
Contributor

@Prashant-thakur77 Prashant-thakur77 commented Nov 5, 2025

Fixes #5529

Also contains additional updates by @MisRob after design feedback that finalizes channel collections table for production.

Summary

This PR replaces the old Vuetify-based ChannelSetList with a new StudioCollectionsTable built using Kolibri Design System (KDS) components. The goal is to continue the process of removing Vuetify from Studio and modernizing the UI to match current Kolibri patterns.
Screencast From 2025-11-05 11-01-24.webm

References

Sub-issue of #5060.

Reviewer guidance

How to get there
Login as [email protected] with password a
Go to Channels > Collections

How to run Kolibri
Find detailed guidance with many code examples in KDS documentation
Read #5060 for more useful references

@MisRob MisRob self-requested a review November 5, 2025 07:05
@MisRob MisRob self-assigned this Nov 5, 2025
@Prashant-thakur77 Prashant-thakur77 force-pushed the channel-collections-table-refactor branch from 9a740f6 to cc52c1e Compare November 8, 2025 19:33
@Prashant-thakur77
Copy link
Contributor Author

Hlo @MisRob i just read the slack thread related to the Positioning of elements
image
I think this will get resolved in this pr only.
as currently this is how it works.
Screencast From 2025-11-11 18-07-10.webm
Just wanted to let you know in case of any prob regarding it. :).Thanks

@MisRob
Copy link
Member

MisRob commented Nov 12, 2025

Hi @Prashant-thakur77, I will follow-up on everything next week. Thanks for preparing the table for this work.

@Prashant-thakur77 Prashant-thakur77 force-pushed the channel-collections-table-refactor branch from cc52c1e to 89e1c81 Compare November 19, 2025 17:19
Copy link
Member

@MisRob MisRob left a comment

Choose a reason for hiding this comment

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

Thanks a lot @Prashant-thakur77 - this is aligned to requirements, and well implemented. I'm leaving some cleanup notes.

Since we already have clarity on how the interface should look like, in addition I am also posting next steps that will actually finalize this page so we can merge this work. Those notes, however, are beyond the scope of the original issue whose main task was to prepare a draft - I consider that done. If you don't wish or can't work on those, that would be completely fine. I can finish it myself - just let me know what works best.

@Prashant-thakur77
Copy link
Contributor Author

Prashant-thakur77 commented Nov 24, 2025

I consider that done. If you don't wish or can't work on those, that would be completely fine. I can finish it myself - just let me know what works best.

@MisRob
Thanks a lot for the feedback! I’m glad the draft meets the requirements.

I’d be happy to help further and can definitely work on the next steps as well.Firstly i will complete the necessary cleanup mentioned above.

@Prashant-thakur77 Prashant-thakur77 force-pushed the channel-collections-table-refactor branch from f2d33fc to 26fc944 Compare November 27, 2025 07:57
@Prashant-thakur77
Copy link
Contributor Author

Prashant-thakur77 commented Nov 27, 2025

@MisRob I have done the changes required ,and regarding one below:

Would you make the token field take the full height of a table row, like here in the current version?

I have done it by apllying -ve margin to studiocopytoken as the extra space was because of the uiFeedback box of the Ktextbox .The desired behaviour is achieved but i have not changed the 8px padding of the cells of the kTable.Do tell if it is required:)
Screenshot From 2025-11-27 11-56-39
Screenshot From 2025-11-27 12-31-52
Screenshot From 2025-11-27 12-32-03

Do tell if they are according to the scope or if for the height issue some other solution is to be used:)

  1. The caption prop's translated string is still showing on top of the table,but i think it might be because of the visually hidden class issue.

@MisRob
Copy link
Member

MisRob commented Nov 27, 2025

This all looks very good @Prashant-thakur77

@MisRob MisRob changed the title [Remove Vuetify from Studio] Channel collections table (first draft version) [Remove Vuetify from Studio] Channel collections table Dec 8, 2025
@MisRob
Copy link
Member

MisRob commented Dec 8, 2025

Hi @Prashant-thakur77, all latest updates are great. I will now take this work over and fine-tune as I discuss final details with our design team. In any case, what you submitted was already very close to the final version and helped us a lot. Thanks!

@Prashant-thakur77
Copy link
Contributor Author

Prashant-thakur77 commented Dec 8, 2025

@MisRob glad to help, currently i am working on the side pannel pr ,but if there is any other issue you would like me to work on ,i will be glad to help:)..

@MisRob
Copy link
Member

MisRob commented Dec 8, 2025

Hi @Prashant-thakur77, are you on Slack? There's one task that I would be happy to entrust you with, but we should chat first whether it'd be interesting for you.

@Prashant-thakur77
Copy link
Contributor Author

Prashant-thakur77 commented Dec 8, 2025

Sure @MisRob i am in slack and would love to talk about the project you are talking:)

@MisRob MisRob force-pushed the channel-collections-table-refactor branch 10 times, most recently from e7ccbec to c2dcbd8 Compare December 18, 2025 16:47
@MisRob MisRob force-pushed the channel-collections-table-refactor branch from c2dcbd8 to 75550ac Compare December 18, 2025 17:20
Copy link
Member

@MisRob MisRob left a comment

Choose a reason for hiding this comment

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

Thanks one more time @Prashant-thakur77!

I pushed some visual fine-tuning - all changes I pushed were related only to design feedback.

@MisRob MisRob merged commit f6564c2 into learningequality:unstable Dec 18, 2025
13 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] Channel collections table

2 participants