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

Replace customStyle property with style #3715

Merged
merged 1 commit into from
Dec 19, 2023
Merged

Replace customStyle property with style #3715

merged 1 commit into from
Dec 19, 2023

Conversation

xpaczka
Copy link
Contributor

@xpaczka xpaczka commented Dec 19, 2023

Resolves #2126

What has been done

Change all customStyle to the style property,

Latest build: extension-builds-3715 (as of Tue, 19 Dec 2023 12:38:08 GMT).

@xpaczka xpaczka self-assigned this Dec 19, 2023
Copy link
Contributor

@jagodarybacka jagodarybacka left a comment

Choose a reason for hiding this comment

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

🔥 🚀 ✨


// Want to return a ReactElement to make this maximally easy to integrate,
// whereas children can be a ReactNode; Fragment will let us achieve that.
// eslint-disable-next-line react/jsx-no-useless-fragment
if (isLoaded) return <>{children}</>

return (
<div className={classNames("skeleton")} data-testid="loading_skeleton">
<div
className={classNames("skeleton")}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
className={classNames("skeleton")}
className="skeleton"

@jagodarybacka jagodarybacka merged commit 1e89e02 into main Dec 19, 2023
7 checks passed
@jagodarybacka jagodarybacka deleted the custom-style branch December 19, 2023 13:38
@xpaczka xpaczka mentioned this pull request Dec 27, 2023
xpaczka added a commit that referenced this pull request Dec 29, 2023
## What's Changed
* Enable swaps on Sepolia testnet by @michalinacienciala in
#3710
* Prevent read-only address submission when validating input by @xpaczka
in #3705
* Support video avatars by @xpaczka in
#3700
* Replace `customStyle` property with `style` by @xpaczka in
#3715
* Show tooltips on hovering over icons with no labels by @xpaczka in
#3711
* Minor UI issues by @xpaczka in
#3716
* Fix hiding unverified assets by @jagodarybacka in
#3708
* v0.54.0 by @xpaczka in
#3706
* chore(ui): typo fix by @IssouChancla in
#3717
* Switch e2e tests run on Goerli to Sepolia by @michalinacienciala in
#3719

## New Contributors
* @xpaczka made their first contribution in
#3705
* @IssouChancla made their first contribution in
#3717

**Full Changelog**:
v0.54.0...v0.55.0

Latest build:
[extension-builds-3720](https://github.com/tahowallet/extension/suites/19343332470/artifacts/1136165985)
(as of Wed, 27 Dec 2023 09:28:52 GMT).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update customStyles property
2 participants