Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

Update style import in Remix guide #2

Closed
wants to merge 1 commit into from
Closed

Update style import in Remix guide #2

wants to merge 1 commit into from

Conversation

TobiasHansen
Copy link

When following the Remix guide, I get the following error when building the app

mantine-v7-remix-template git:(master) ✗ npm run build

> build
> remix build

Building Remix app in production mode...
The path "@mantine/core/styles.css?__remix_sideEffect__" is imported in  but "@mantine/core/styles.css?__remix_sideEffect__" was not found in your node_modules. Did you forget to install it?
Built in 833ms

The error goes away by importing the styles through a stylesheet link. I am no expert on the exact differences here, but compared the css output and it resulted in the same output.

@rtivital
Copy link
Owner

rtivital commented Jun 2, 2023

This seems to be an issue with the latest versions Remix – remix-run/remix#6502
Not really sure whether the fix should be implemented this way

@TobiasHansen
Copy link
Author

ah, right, sounds like its a Remix-bug. Looks like they just merged a fix, so will probably be corrected in the next nightly. Will test again tomorrow.

@rtivital
Copy link
Owner

rtivital commented Jun 2, 2023

Thanks, please let me know if you find that we need to change something on Mantine side

@TobiasHansen
Copy link
Author

@rtivital Can confirm the issue is fixed in Remix v1.17.0, closing this issue :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants