Skip to content

Conversation

@arikorn
Copy link
Contributor

@arikorn arikorn commented Jun 30, 2025

move the deprecation @includes inside the css rule

completes issue #979

move the deprecation `@includes` inside the css rule

completes issue coreui#979
@mrholek
Copy link
Member

mrholek commented Jun 30, 2025

@arikorn, thank you for your PR. There is a simpler way to do this. Since we don't use those mixins in our codebase, we can remove them from this file scss/_mixins.scss. Can you update your PR?

@arikorn
Copy link
Contributor Author

arikorn commented Jun 30, 2025

Thanks @mrholek. Pushed the change. I left the other files in this PR alone since I'm assuming you'll remove them anyway if you want. (Or if you do keep those files, it's still better to have the @include inside the css rule...)

@mrholek mrholek changed the title Fix: spurious deprecation warnings in scss code fix: spurious deprecation warnings in SCSS code Jun 30, 2025
@mrholek mrholek merged commit 2892ee5 into coreui:main Jun 30, 2025
@arikorn
Copy link
Contributor Author

arikorn commented Jun 30, 2025

Thank you @mrholek!

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.

2 participants