Repository navigation
[Remove Vuetify from Studio] View/Exit fullscreen buttons/links in file preview聽#5442
Copy link
Copy link
Closed
Labels
DEV: frontendcommunity-contribution-in-progressgood first issueSelf-contained, straightforward, low-complexitySelf-contained, straightforward, low-complexityhacktoberfesthelp wantedOpen source contributors welcomeOpen source contributors welcome
Description
馃檪 Looking for an issue? Welcome! This issue is open for contribution. If this is the first time you鈥檙e requesting an issue, please:
- Read Contributing guidelines carefully. Pay extra attention to Using generative AI. Pull requests and comments that don鈥檛 follow the guidelines won鈥檛 be answered.
- Confirm that you鈥檝e read the guidelines in your comment.
Sub-issue of #5060.
Complexity: Low
Summary
Migrate the following file preview View/Exit fullscreen buttons/links from Vuetify to Kolibri Design System.
VBtn and shared/views/ActionLink that is built with Vuetify components are currently used to show them. To remove these Vuetify dependencies , replace the usage of VBtn and ActionLink in this specific location with the most suitable KDS button or link components. Do not modify ActionLink. Ensure that all buttons / links work exactly as before.
How to get there
- Login as
a@a.comwith passworda - Go to Channels > Published Channel
- Click Sample video
- In the side panel, click View fullscreen
- Close fullscreen
- Click Edit
- In the Details tab, click View fullscreen
Guidance
- Find detailed guidance with many code examples in KDS documentation
- Read the project for more useful references
Out of Scope
- Do not refactor any other areas of the codebase
- Do not modify
ActionLink
Expected UI/UX changes
- Minor visual differences naturally stemming from the use of KDS
Acceptance criteria
General
- The specification above is followed.
- Except for "Expected UI/UX changes," there are no functional or visual differences in user experience.
- There are no
::v-deepor/deep/selectors. - All user interactions are manually tested with no regressions.
- Pull request includes screenshots.
a11y and i18n
See the project's "Guidance" for useful references.
- Implementation meets a11y standards
- All components are LTR and RTL compliant (preview with
pnpm run devserversince:hotdoesn't render RTL properly) - All user-facing strings are translated properly
- The
notranslateclass been added to elements that shouldn't be translated by Google Chrome's automatic translation feature (e.g. user-generated text) - Mobile experience is reasonable
Unit tests
- If there is a unit test suite already, it is meaningfully updated (even if tests don't fail)
- If there is no unit test suite, a new one is created. Do not use obsolete
@vue/test-utilsapproach. Instead, use@testing-library/vue(Vue Testing Library).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
DEV: frontendcommunity-contribution-in-progressgood first issueSelf-contained, straightforward, low-complexitySelf-contained, straightforward, low-complexityhacktoberfesthelp wantedOpen source contributors welcomeOpen source contributors welcome

Activity
vtushar06 commentedon Oct 9, 2025
Hiii @MisRob, I would love to resolve this Issue, can you please assign this to me. Thanks
MisRob commentedon Oct 9, 2025
Hi @vtushar06, I will assign you, thank you.
vtushar06 commentedon Oct 9, 2025
Hi @MisRob, Issue resolved successfully and under PR #5447, Please whenever possible look that out. Thanks