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

fix: Use VItemGroup to display settings list again #1855

Merged
merged 2 commits into from
Jan 19, 2023

Conversation

Janbong
Copy link
Contributor

@Janbong Janbong commented Jan 19, 2023

v-list-item-group seems to be not available anymore in Vuetify 3. I can't find any information on it. We use v-item-group (not v-list-group which was causing the settings not to show up) instead.
We use templates with v-slot:prepend and v-slot:append to place the icons correctly before or behind the actual settings name and description.

v-list-item-group is not available anymore in Vuetify 3.
We use v-item-group instead.
We use templates with v-slot:prepend and v-slot:append to place the
icons correctly before or behind the actual settings name and
description.
@jellyfin-bot jellyfin-bot added the vue Pull requests that edit or add Vue files label Jan 19, 2023
Copy link
Member

@ferferga ferferga left a comment

Choose a reason for hiding this comment

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

One small thing and it's ready to go.

This should have been reported as a warning by your code editor. If it didn't, tell us so we can try to help you configure VSCode appropiately :) (Codespaces are a quick and good way to start up quickly as well).

The vite branch is full of linter and type errors right now, but it's not that we don't care about them and adding code with warnings is good, it's just that it's a work in progress :).

<v-avatar>
<v-icon :icon="linkItem.icon" />
</v-avatar>
<template v-slot:prepend>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<template v-slot:prepend>
<template #prepend>

This is the first occurence in the PR, but you used this syntax everywhere else. Revisit those as well please.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

My VSCode did not warn me about it.
I am using the development container and installed all suggested extensions.
Something else I should have configured?

Copy link
Member

Choose a reason for hiding this comment

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

@Janbong Did you wait for npm install to complete?

@jellyfin-bot
Copy link

Cloudflare Pages deployment

Latest commit 4680235
Status 🔄 Deploying...
Preview URL Not available
Type 🔀 Preview

View bot logs

@sonarcloud
Copy link

sonarcloud bot commented Jan 19, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Janbong Janbong requested a review from ferferga January 19, 2023 15:37
Copy link
Member

@ferferga ferferga left a comment

Choose a reason for hiding this comment

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

Thank you very much for your PR! ❤

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vue Pull requests that edit or add Vue files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants