Proposed Solution for Next.js Issue

[cholazzzb]

Hi, I just read this closed issue Remove CSS import in order to support Next.js. I come up with the idea that this issue can be solved with CSS-in-JS, for example using the simplest one is inline styles or style={{color: 'red'}} in the component. However, I see that the library makes use of CSS variables. Maybe the CSS variable can be moved inside the component with js as well? What are your thoughts?

[nick-keller]

One of the advantage of CSS variables is letting the user customize them at which ever level it desire.
Same for CSS classes, very easy to override a particular class.

I think the only solution would be to introduce a breaking change and force the users to import the style themself.

[LoncaSorin]

Couldn't we follow the example from here https://www.npmjs.com/package/react-datasheet-grid-nextjs? This works, but it doesn't have the latest updates

[nick-keller]

Hello @LoncaSorin , this will be live in less than 24h, thx for your support!

[LoncaSorin]

You're welcome, the library is great and I really wanted to use it. Keep up the good work!

[nick-keller]

Alright this is now live ;)

[nick-keller]

That has not changed since v3, I guess I need to compile imports into requires, if you have any insight that'd help a lot.

[LoncaSorin]

Sorry, but I don't. I will ask some of my colleagues and if I get an answer, I will come back and tell you.

[LoncaSorin]

@nick-keller sorry, I don't want to put pressure on you or something because you were great and answered so fast, but tomorrow I will need to present the library to the team in order to decide with which one should we go and it would really help if I could tell them how much time do you think it would take to compile the imports into requires? Or if you consider doing that
In any case, what you've built here is amazing and I would love to use it

[nick-keller]

@LoncaSorin we should be good now ;)

[LoncaSorin]

Omg, it's working, thank you, you're amazing!
I would send you a beer, I swear :))