-
-
Notifications
You must be signed in to change notification settings - Fork 361
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
refactor: move styles to assets #7394
Conversation
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
AI-Generated Summary: This pull request involves comprehensive restructuring of the project's directory for style assets, more specifically moving style related files from the 'styles' directory to 'assets/styles'. Various vue and scss component files have their import paths updated to reflect this change in the project directory structure. The update chiefly impacts the import statements for 'variables' and certain component style files which have been shifted to the 'assets/styles/abstracts' and 'assets/styles/components' directory from 'styles/abstracts' and 'styles/components' respectively. In files like 'ListingCartModal.vue', 'GalleryItemTransfer.vue', 'CollectionInfo.vue', 'UnlockableCollectionBanner.vue' and many others, a single line import update can be seen from '@/styles/abstracts/variables' to '@/assets/styles/abstracts/variables'. Other files like '_popup.scss', '_search.scss', and '_carousel.scss', were merely moved to the 'assets/styles/components' directory without any content modifications. The 'nuxt.config.js' file also sees an adjustment in the global CSS file path from 'styles/index.scss' to '@/assets/styles/index.scss'. It is essential to inspect each component for potential style-related issues due to this large-scale modification. |
Reviewpad Report
|
Code Climate has analyzed commit 2d131a7 and detected 0 issues on this pull request. View more on Code Climate. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
why?
|
Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type
Context
main
andnuxt
branchDid your issue had any of the "$" label on it?
Screenshot 📸
Copilot Summary
🤖 Generated by Copilot at b733447
Moved all styles files to the
assets
folder and updated the import path for the variables file. This refactor aimed to improve the structure and organization of the project.🤖 Generated by Copilot at b733447