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

Mix of tailwind and bem #1017

Open
asvae opened this issue Jan 24, 2024 · 2 comments
Open

Mix of tailwind and bem #1017

asvae opened this issue Jan 24, 2024 · 2 comments
Labels
d1: super easy Task for newbies. Players take half normal damage from monsters and the environment. good first issue Right issue if you're new to vuestic refactoring Prettify code without introducing new features v1: minor This might be helpful, but not really

Comments

@asvae
Copy link
Member

asvae commented Jan 24, 2024

Right now we have some components that use bem, while majority of the application uses tailwind

I searched by "__" and this seems to be the list. Feel free to check other components.

We want to only use tailwind for vuestic-admin. So for all of these we need to replace bem implementation with tailwind classes (simplest way might be to use GPT to help with that).

image

@asvae asvae added good first issue Right issue if you're new to vuestic refactoring Prettify code without introducing new features d1: super easy Task for newbies. Players take half normal damage from monsters and the environment. v1: minor This might be helpful, but not really labels Jan 24, 2024
@AfganAbbas
Copy link

So, what is the task? @asvae

@asvae
Copy link
Member Author

asvae commented Jan 26, 2024

Hey @AfganAbbas I updated the issue so hopefully it's more clear. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
d1: super easy Task for newbies. Players take half normal damage from monsters and the environment. good first issue Right issue if you're new to vuestic refactoring Prettify code without introducing new features v1: minor This might be helpful, but not really
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants