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

Makes tailwind config settings easier. #275

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

Conversation

doong-jo
Copy link
Contributor

@doong-jo doong-jo commented Feb 17, 2024

Hello, thank you for your outstanding results.

The way to create and manage a separate file is to write down a long relative path or put aliases to import the file, which is not a very good developer experience.

This provides a way to initialize the tailwind config without creating a separate file (libs/tailwind.js).
As a result, developers can import 'twnrc' through a single line of configuration and use it immediately.

Below is speculated to be an issue caused by tailwind config not being set up properly. I think this PR change will make it easier to set up the config and make developers less bogged down.

#191

@doong-jo doong-jo changed the title Configure tailwind config Makes tailwind config settings easier. Feb 17, 2024
@jaredh159
Copy link
Owner

i'm really intrigued by this idea. i think it is an improvement to the API, and i want to carefully consider it. i'm a little bit concerned about initialization/configuration race conditions (making sure that no usage of importing twrnc from the bare import is executed before the one-time config happens). but if i can satisfy myself that it's not a problem, i'm really open to this (or something very similar).

might be a few days before i take a detailed look at it, but wanted to at least let you know i'm interested in this and open to merging. appreciate the time you put in to this, thanks!

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.

None yet

2 participants