Skip to content

Conversation

@webdev03
Copy link
Contributor

@webdev03 webdev03 commented Dec 20, 2021

This adds live reloading with electron-reloader for a better development experience so now working with Tailwind actually makes sense, because if you didn't have live reloading, you would have to kill the electron process again and again. It also works with the normal builds, as I wrapped it in a try catch block, because electron-reloader is a Dev Dependency. Tell me if you want it to be removed.
I haven't changed other styles to tailwind, though, tell me if I should.

You should also credit MystPi for helping with implementation!

I'd also like to add that instead of running npm run start run npm run dev if you want to work with Tailwind at the same time. When starting to develop and you aren't touching anything with Tailwind styling, please run npm run tailwind at least once.

@webdev03
Copy link
Contributor Author

Sorry, my editor messed up the formatting, that's why I wanted prettier in the old freecat, but since you say it broke Freecat, I didn't add it.

@hello-smile6
Copy link
Contributor

Sorry, my editor messed up the formatting, that's why I wanted prettier in the old freecat, but since you say it broke Freecat, I didn't add it.

I have an opinion on that, but don't want to offend anyone, so won't say it.

@hello-smile6
Copy link
Contributor

This adds live reloading with `electron-reloader` for a better development experience so now working with Tailwind actually makes sense, because if you didn't have live reloading, you would have to kill the electron process again and again. It also works with the normal builds, as I wrapped it in a try catch block, because `electron-reloader` is a Dev Dependency. Tell me if you want it to be removed. I haven't changed other styles to tailwind, though, tell me if I should.

Fixes #15 

You should also credit MystPi for helping with implementation!

Suggested edits. That way, the issue will auto-close.

@jdev082 jdev082 merged commit 30c05ee into CatalystDevOrg:master Dec 20, 2021
@webdev03
Copy link
Contributor Author

Thank you so much for merging! Should I work on the other styles now?

@hello-smile6
Copy link
Contributor

Thank you so much for merging! Should I work on the other styles now?

Could you work on SpellCheckJS instead?

@webdev03
Copy link
Contributor Author

Why? This is the Catalyst repository.

@hello-smile6
Copy link
Contributor

Why? This is the Catalyst repository.

Because I need help with it!

@webdev03 webdev03 deleted the tailwind branch December 23, 2021 05:18
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.

3 participants