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

"tailwindify" inline styles #33

Open
thescientist13 opened this issue Oct 9, 2022 · 0 comments
Open

"tailwindify" inline styles #33

thescientist13 opened this issue Oct 9, 2022 · 0 comments
Labels
enhancement Improvement to existing feature good first issue Good for newcomers

Comments

@thescientist13
Copy link
Contributor

thescientist13 commented Oct 9, 2022

As part of completing tasks like #30 and #31, there were some cases where using style directly on an element was done.

<img
  class="block w-20 rounded-full text-center"
  style="margin:0 auto"
  src="${avatar}"
  alt="Photo of ${name} (${title})"
/>

In a Tailwind world, it would be nice to using classes as much as possible, or creating custom rules, or whatever the best practice is.


Also, for playing around some colors were added to tailwind.config.js but these should already be available as CSS custom properties. I wonder if there is a way to avoid duplicating those color values so they don't have to be updated in one place?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement to existing feature good first issue Good for newcomers
Projects
Status: 🔖 Ready
Development

No branches or pull requests

1 participant