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

use tailwind css instead of picocss #33

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

AliYar-Khan
Copy link

@AliYar-Khan AliYar-Khan commented Oct 11, 2024

Issue Reference(s):
Fixes #5
/claim #5

Migration to Tailwind CSS

had to install tailwind package as dev dependency and on start of server generate the CSS from main.css to tailwind.css
Also the main.css is also convert to tailwind classes and use the utility provided by tailwind to apply them. Reason for doing this was because in main.js file we have used the class name to add event listeners on the elements like button and it could have break the functionality.

@AliYar-Khan AliYar-Khan force-pushed the migration/picocss-to-tailwindcss branch from eef26bb to 1342914 Compare October 11, 2024 20:10
Sign

Signed-off-by: Ali Yar Khan <[email protected]>
@AliYar-Khan AliYar-Khan force-pushed the migration/picocss-to-tailwindcss branch from a69ef93 to 853acc8 Compare October 11, 2024 20:17
@AliYar-Khan
Copy link
Author

/claim #5

@AliYar-Khan
Copy link
Author

@nkkko please take a look here to my work and let me know if there any changes required

@AliYar-Khan
Copy link
Author

@nkkko Please review the changes and let me know if anything is unclear. Will really appreciate your feedback.

@nkkko
Copy link
Member

nkkko commented Nov 5, 2024

Sorry for the late reply, have been busy. The interface is messed up, especially buttons, see img:
image

It still loads Pico?
image

@AliYar-Khan
Copy link
Author

I will look into these not sure why it is happening !!!

@mojafa
Copy link
Contributor

mojafa commented Dec 11, 2024

@AliYar-Khan I’ve updated main with some changes that affect this PR. Could you please rebase your branch onto the latest main and resolve any conflicts that come up? Once that’s done, we can review the updated changes and move forward.

@AliYar-Khan
Copy link
Author

@mojafa I did updated it

@mojafa
Copy link
Contributor

mojafa commented Dec 17, 2024

@AliYar-Khan looks like we still have pico
Screenshot 2024-12-17 at 23 26 43

@mojafa
Copy link
Contributor

mojafa commented Dec 17, 2024

Screenshot 2024-12-17 at 23 27 30

@mojafa
Copy link
Contributor

mojafa commented Dec 17, 2024

Screenshot 2024-12-17 at 23 32 58

@AliYar-Khan
Copy link
Author

@mojafa yes I am not sure why it is there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate from Picocss to Tailwind and Cleanup Frontend Code
3 participants