-
Notifications
You must be signed in to change notification settings - Fork 0
feat: Nuxt UI Version 3 einsetzen #24
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: master
Are you sure you want to change the base?
Conversation
The commit includes version upgrades for several packages in package.json. These upgrades include "@nuxt/devtools", "@nuxt/ui", "nuxt", "sass", "@nuxt/content", "@nuxt/ui-pro", "@vueuse/nuxt", and "puppeteer".
This commit includes the following changes: - Refactored the code to improve readability. - Enhanced the image wrapper styling. - Updated the gradient color scale in the shine class. - Corrected the import statement format. - Improved the logic of the getImageAbsolutePath function. - Added a new line at the end of the file.
… color and xl size
This commit updates the app.vue file to wrap the NuxtLayout and NuxtPage components with UApp and UMain components for improved page structure. It also makes minor formatting improvements for better readability.
Updated the UI configuration in app.config.ts to include container settings and color definitions. This change also includes adjustments to the light and dark mode color variables.
This commit includes an extensive update to the CSS styling parameters in `main.css`. Changes include the addition of several font sizes, color variables, and updates to some existing UI variables. These changes aim to improve the overall look and feel of the application.
Added new UI configurations for buttons and size in the app.config.ts file. This includes default variants, slots, and size attributes for xs, sm, md, lg, and xl. Also updated the primary and secondary color settings.
The Carousel.vue component has been updated to improve user experience and accessibility. Button elements have been refactored to use UButton component for better consistency and maintainability. The on-click events have been adjusted to provide a smoother transition. The usePointerSwipe event has been updated to provide better responsiveness. Code readability has also been improved with better formatting and organization.
Updated the ImageFigure.vue component to enhance UI. This includes changes in class ordering, button UI, button hover effects, and the transition properties. The script setup has also been updated for TypeScript.
Update all instances of the Button component in the index.vue file to use the UButton component. This includes changing component properties such as color, size, and variant for better styling. Additionally, update the text color class for the Headline component.
Updated UI elements on the team page for improved user experience, including changes to image display, button styles, and layout adjustments.
Updated multiple dependencies to their latest versions, including @nuxt/content, @nuxt/image, @nuxt/ui-pro, Tailwind, and others. These updates improve compatibility, add new features, and align with the latest ecosystem standards.
The `package.json` file has been updated to include the latest versions of several dependencies. This includes minor version updates for `@nuxt/devtools`, `sass`, `@nuxt/content`, `nuxt`, `@nuxt/image`, and `@nuxt/ui-pro`.
Updated various package dependencies to newer versions in the `pnpm-lock.yaml` file. This includes improvements and bug fixes provided by the latest package releases across multiple dependencies.
Updated import statements in main.css. Removed unnecessary variables and updated the '--ui-primary' variable color.
This commit refactors the code in the `error.vue` file to enhance readability. Changes include formatting and reordering of elements, and restructuring of some code blocks for better understanding.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 35 out of 36 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- assets/css/main.css: Language not supported
Comments suppressed due to low confidence (2)
app/router.options.ts:6
- [nitpick] Consider extracting the fixed offset value '60' into a named constant to improve readability and maintainability.
const top = 60
components/Button.vue:1
- Confirm that removal of the custom Button component is intentional and that all its usages have been replaced with the corresponding Nuxt UI component.
<template> ...
components/Image.vue
Outdated
color.scale($color, $alpha: -70%) 0%, | ||
color.scale($color, $alpha: -30%) 100% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verify that using negative alpha values with 'color.scale' is supported in your Sass version, as this may lead to unexpected color outputs.
color.scale($color, $alpha: -70%) 0%, | |
color.scale($color, $alpha: -30%) 100% | |
rgba($color, 0.3) 0%, | |
rgba($color, 0.7) 100% |
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 35 out of 36 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- assets/css/main.css: Language not supported
Comments suppressed due to low confidence (1)
components/Image.vue:84
- [nitpick] Review the new gradient values: please verify that replacing fade_out with color.scale using $alpha: 0.3 and $alpha: 0.7 produces the intended visual effect for the overlay.
color.scale($color, $alpha: 0.3) 0%,
Updated the hardcoded scroll position offset in the router options with a constant value for better maintainability and readability.
Removed unnecessary `@font-face` declarations for various languages and added a new `--ui-foreground` color variable. This change will help to simplify the CSS file and make it more maintainable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 35 out of 37 changed files in this pull request and generated 1 comment.
Files not reviewed (2)
- assets/css/main.css: Language not supported
- assets/css/tailwind.css: Language not supported
Comments suppressed due to low confidence (2)
content/blog/wie-koennen-blockchains-unternehmen-bereichern.md:15
- [nitpick] The heading level was changed from '##' to '#', which may affect the document structure and accessibility. Please confirm that this change is intentional and consistent with your content hierarchy.
# **Block…was?**
components/Image.vue:84
- The use of percentage values for the $alpha parameter in the color.scale function appears unusual; typically, alpha values are unitless (e.g., 0.3). Please verify that the intended transparency is achieved with these values.
color.scale($color, $alpha: 0.3%) 0%,
app/router.options.ts
Outdated
setTimeout(() => resolve(findHashPosition(to.hash)), 50) | ||
} else { | ||
nuxtApp.hooks.hookOnce('page:finish', () => { | ||
setTimeout(() => resolve(findHashPosition(to.hash)), 50) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] Using a fixed 50ms delay for hash-based scrolling might lead to inconsistent behavior on slower devices or complex pages. Consider using a more robust or configurable delay to ensure proper scroll alignment.
setTimeout(() => resolve(findHashPosition(to.hash)), 50) | |
} else { | |
nuxtApp.hooks.hookOnce('page:finish', () => { | |
setTimeout(() => resolve(findHashPosition(to.hash)), 50) | |
setTimeout(() => resolve(findHashPosition(to.hash)), calculateDynamicDelay()) | |
} else { | |
nuxtApp.hooks.hookOnce('page:finish', () => { | |
setTimeout(() => resolve(findHashPosition(to.hash)), calculateDynamicDelay()) |
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 35 out of 37 changed files in this pull request and generated no comments.
Files not reviewed (2)
- assets/css/main.css: Language not supported
- assets/css/tailwind.css: Language not supported
Comments suppressed due to low confidence (2)
components/Image.vue:84
- The updated use of 'color.scale' with an alpha value expressed in percentages may not produce the intended opacity gradient compared to the previous fade_out() calls. Please verify that using 0.3% and 0.7% yields the same visual result as before.
color.scale($color, $alpha: 0.3%) 0%,
app.config.ts:18
- [nitpick] The syntax 'max-w-(--container-3xl)' looks unusual and may not be interpreted correctly by your CSS framework. Consider verifying or adjusting the syntax to meet the expected format (e.g. using square brackets) for dynamic custom property values.
container: { base: 'max-w-(--container-3xl) mx-auto px-4 sm:px-6 lg:px-8', },
Updated headline types and their respective styles for better clarity and consistency. Adjusted container width classes to align with design specifications. These changes improve the overall visual structure and responsiveness.
Upgraded `sass`, `nuxt`, `@vueuse/nuxt`, and `puppeteer` to their respective newer versions. This ensures compatibility, fixes potential bugs, and includes the latest updates for improved stability and features.
This commit updates various dependencies in `pnpm-lock.yaml` to their latest versions, including `nuxt`, `@vueuse/nuxt`, `sass`, `puppeteer`, and others. It ensures compatibility with new features, bug fixes, and performance improvements in these packages.
Replaced the static scroll delay with a dynamic calculation based on page complexity. This ensures smoother scrolling behavior, adapting to the content size while maintaining reasonable performance limits.
Adjusted the bottom offset for the button on 2xl screens to ensure proper alignment. This resolves visual inconsistencies in the layout for larger viewports.
This commit includes the addition of a page transition animation and a routing delay in the app.vue file. The page transition animation is set to transform property with a duration of 0.4s with ease-in-out timing function. The routing delay is set to 1000ms. The overflow property has also been changed from overflow-x-hidden to overflow-hidden.
No description provided.