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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change PostCSS to v8 #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arturparkhisenko
Copy link

  • Changed postcss dependency version to 8 馃啎, plugin migration guide.
  • Changed Node.js version to 10. Note: PostCSS v8 dropped Node.js 6.x, 8.x, 11.x, and 13.x versions support.

Closes #12

@arturparkhisenko
Copy link
Author

@larsenwork do you know who is a maintainer there who can review this PR?

@jonnitto
Copy link

@larsenwork Any news on this?

@arturparkhisenko
Copy link
Author

@larsenwork can you please review it?

And as a part of the Hacktoberfest OSS initiative, I wanted to ask you to add the label hacktoberfest-accepted if that PR will be accepted and merged.

@arturparkhisenko
Copy link
Author

Last year I did open it when I hoped to resolve this dependency thing, but because of waiting, we ended writing our own function based on multiple sources. If it'll help anyone:

@thomasaull
Copy link

@arturparkhisenko Would you be willing to share your solution?

@arturparkhisenko
Copy link
Author

@thomasaull Hi, unfortunately, I cannot share it, the project is not open source, but what can help to create the solution:

Those two code parts are a bit simpler to read than the plugin.
In addition to the just easing function, we did write the ease-in-out-sine function for steps progress (so it's not hardcoded and the amount of steps is a variable) and implemented support for transparency. Last but not least if it's a JS function then chroma.js it's a library that might help a lot. We did write our own using SASS and relied on built-ins.

@Ambient-Impact
Copy link

Ambient-Impact commented Nov 2, 2022

@arturparkhisenko Glad to see some development on this!

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.

Compatibility with PostCSS 8
4 participants