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

styles(frontend): add theme package #4052

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Conversation

xudaotutou
Copy link
Contributor

@xudaotutou xudaotutou commented Oct 9, 2023

🤖 Generated by Copilot at 6434164

Summary

🎨🌐🧹

This pull request introduces a new @sealos/ui package that defines a custom theme and components for the sealos platform using the Chakra UI framework. It also updates the frontend apps to use the @sealos/ui package and remove unnecessary dependencies. Additionally, it adds internationalization support to the adminer and bytebase apps using the i18next framework and the next-i18next plugin.

We're building the sealos platform, with UI and i18n
We're optimizing the performance, removing the dependencies
We're extending the Chakra theme, with colors and components
We're simplifying the Dockerfile, using the node alpine

Walkthrough

  • Simplify and update the Dockerfile for the frontend app (link, link, link)
  • Rename the frontend package from kf-web to sealos (link)
  • Add a new @sealos/ui package that defines a custom theme and components for the frontend apps (link, link, link, link, link, link, link)
  • Add scripts and dependencies for generating TypeScript typings for the custom theme (link, link)
  • Import and extend the custom theme in the desktop app (link, link)
  • Remove unnecessary dependencies from the desktop and applaunchpad apps (link, link, link, link)
  • Add path alias for the @ui/* modules in the desktop app (link)
  • Add dependencies for internationalization and localization in the adminer and bytebase apps (link, link)
  • Remove unnecessary dependencies from the costcenter app (link, link)
  • Replace external CSS modules with Chakra UI style props and pseudo elements in the costcenter app (link, link, link, link, link, link, link, link, link, link, link)

@pull-request-size
Copy link

Whoa! Easy there, Partner!

This PR is too big. Please break it up into smaller PRs.

@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Oct 9, 2023

🤖 Generated by lychee action

Summary

Status Count
🔍 Total 973
✅ Successful 400
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 572
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

All modified lines are covered by tests ✅

📢 Thoughts on this report? Let us know!.

@xudaotutou xudaotutou changed the title styles(chore): add theme package styles(frontend): add theme package Oct 10, 2023
@@ -3,7 +3,8 @@
"lib": ["dom", "dom.iterable", "esnext"],
"baseUrl": ".",
"paths": {
"@/*": ["./src/*"]
"@/*": ["./src/*"],
"@ui/*": ["../../packages/ui/src/*"]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't seem to need to be written and can be introduced directly by the package name

@c121914yu c121914yu merged commit d6b1f6d into labring:main Oct 10, 2023
21 checks passed
bxy4543 pushed a commit to bxy4543/sealos that referenced this pull request Oct 13, 2023
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.

3 participants