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

UI design #338

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

UI design #338

wants to merge 6 commits into from

Conversation

dechdev
Copy link
Contributor

@dechdev dechdev commented Oct 25, 2024

This PR adds TailwindCSS and ShadCN component library. It retains MaterialUI, as that will be kept until all components are transitioned to Tailwind.

This PR also formalizes RPC querying patterns for most methods in Paladin.

Base automatically changed from ui to main October 30, 2024 13:55
@gabriel-indik
Copy link
Contributor

This PR introduces a number of neat ideas for the layout, making the experience more streamlined. The switch out of MUI and into TailwindCSS+ShadeCN seems like a relatively big step for which we may want to investigate a bit more the pros and cons. From what I've seen so far, using these tools has an upfront cost (need additional plugins on VSCode, setting up build process for generating resources and learning curve) but once that is done, the resulting UI resources would be more compact. I'll continue to look into the differences between the stacks.

Copy link

This PR is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale label Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants