sei-protocol / sei-js

SeiJS is a monorepo that contains multiple NPM libraries for writing applications that interact with the Sei network.
44 stars 38 forks source link

Styled Components #101

Closed codebycarson closed 11 months ago

codebycarson commented 11 months ago

Added styled components and improved customizability

Example with custom theme

Screenshot 2023-11-27 at 1 44 06 PM
changeset-bot[bot] commented 11 months ago

🦋 Changeset detected

Latest commit: e436100ed52bf934e0c1caf6391c0db92131484f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ------------- | ----- | | @sei-js/react | Minor |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

codebycarson commented 11 months ago

Can be tested on version 0.0.0-internal-20231127193711

codecov-commenter commented 11 months ago

Codecov Report

Attention: 35 lines in your changes are missing coverage. Please review.

Comparison is base (373f739) 76.66% compared to head (e436100) 76.56%.

Files Patch % Lines
...t/src/lib/components/WalletConnectButton/styles.ts 47.82% 12 Missing :warning:
...components/WalletSelectModal/WalletSelectModal.tsx 58.33% 10 Missing :warning:
...act/src/lib/components/WalletSelectModal/styles.ts 87.03% 7 Missing :warning:
...onents/WalletConnectButton/WalletConnectButton.tsx 33.33% 6 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #101 +/- ## ========================================== - Coverage 76.66% 76.56% -0.11% ========================================== Files 33 35 +2 Lines 450 512 +62 Branches 91 122 +31 ========================================== + Hits 345 392 +47 - Misses 105 120 +15 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

besated commented 11 months ago

Had a question, but otherwise looks good! Thanks for making these changes

codebycarson commented 11 months ago

Had a question, but otherwise looks good! Thanks for making these changes

Hey, no not required, if not passed it defaults to the standard white and grey. I'll update the README to make this more clear