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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev server needs to be restarted for Tailwind config changes to be picked up #4913

Closed
1 task
Princesseuh opened this issue Sep 29, 2022 · 4 comments · Fixed by #4947
Closed
1 task

Dev server needs to be restarted for Tailwind config changes to be picked up #4913

Princesseuh opened this issue Sep 29, 2022 · 4 comments · Fixed by #4947
Assignees
Labels
- P4: important Violate documented behavior or significantly impacts performance (priority)

Comments

@Princesseuh
Copy link
Member

What version of astro are you using?

1.3.1

Are you using an SSR adapter? If so, which one?

N/A

What package manager are you using?

pnpm

What operating system are you using?

macOS

Describe the Bug

Changes inside a Tailwind config are only picked up when the dev server is restarted, this is annoying and not a good DX when wanting to make quick iterations.

Link to Minimal Reproducible Example

https://stackblitz.com/edit/github-mdx6br?file=tailwind.config.cjs,src%2Fpages%2Findex.astro&on=stackblitz

Participation

  • I am willing to submit a pull request for this issue.
@Princesseuh Princesseuh added the - P5: urgent Fix build-breaking bugs affecting most users, should be released ASAP (priority) label Sep 29, 2022
@rishi-raj-jain
Copy link
Contributor

I wonder if this is supported by any framework out there right now?

@bluwy
Copy link
Member

bluwy commented Sep 29, 2022

The tailwind integration could add a Vite plugin that watches the tailwind.config.js and trigger a server restart like how svelte.config.js does

@matthewp
Copy link
Contributor

@rishi-raj-jain for context, this is a common source of support threads, so we'd like to support it even if we are paving new ground in doing so.

@JuanM04
Copy link
Contributor

JuanM04 commented Sep 30, 2022

I can pick this up if no one has done it yet

@JuanM04 JuanM04 self-assigned this Sep 30, 2022
@matthewp matthewp added - P4: important Violate documented behavior or significantly impacts performance (priority) and removed - P5: urgent Fix build-breaking bugs affecting most users, should be released ASAP (priority) labels Oct 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
- P4: important Violate documented behavior or significantly impacts performance (priority)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants