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

Basic configuration for chakra UI #28

Merged
merged 12 commits into from
Nov 20, 2023
Merged

Basic configuration for chakra UI #28

merged 12 commits into from
Nov 20, 2023

Conversation

kkosiorowska
Copy link
Contributor

@kkosiorowska kkosiorowska commented Nov 14, 2023

We would like to be able to change the theme mode for the dApp. The mode should be detected for ledger live apps. Let's use an external chakra UI lib to create custom styles for components. This should make it easier to switch the theme for the components. Ten PR adds a basic configuration for chakra UI lib.

Chakra UI library has already been used in previous projects. It has prepared components and you can easily manage theme changes in the application by this lib. This should make development faster and easier.

What has been done

  • Using the ChakraProvider
  • Initialization of colors for themes
  • Detecting the correct theme for the ledger live app.

UI

This PR only sets up the configuration for the chakra Ui library. Please ignore the differences between the button designs. The styles of the components will be fixed in the next PRs.

Dark mode
Screenshot 2023-11-14 at 12 10 35

Light mode
Screenshot 2023-11-14 at 12 10 27

Screen.Recording.2023-11-14.at.14.55.35.mov

@kkosiorowska kkosiorowska self-assigned this Nov 14, 2023
@kkosiorowska kkosiorowska mentioned this pull request Nov 14, 2023
@kkosiorowska kkosiorowska marked this pull request as ready for review November 14, 2023 14:11
@kkosiorowska kkosiorowska marked this pull request as draft November 16, 2023 13:56
@kkosiorowska kkosiorowska marked this pull request as ready for review November 16, 2023 14:04
Base automatically changed from ledger-live-app to main November 16, 2023 14:30
There was a problem with the `import/no-extraneous-dependencies` rule for the new Chakra UI packages. Let's disable it at this step.
Copy link
Contributor

@r-czajkowski r-czajkowski left a comment

Choose a reason for hiding this comment

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

Left a few comments to take a look.

We would like to be able to change the theme mode for the dApp. The mode should be detected for ledger live apps. Let's use an external chakra UI lib to create custom styles for components.
Currently, the designs for the dark mode are not yet fully ready. Let's hide it under the feature flag to make sure that the page is displayed correctly.
r-czajkowski
r-czajkowski previously approved these changes Nov 20, 2023
Copy link
Contributor

@r-czajkowski r-czajkowski left a comment

Choose a reason for hiding this comment

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

:shipit:

@r-czajkowski r-czajkowski merged commit 4cf4d8c into main Nov 20, 2023
@r-czajkowski r-czajkowski deleted the init-components branch November 20, 2023 11:11
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.

3 participants