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

refactor(Button): enable loading state for button #253

Merged
merged 3 commits into from
Sep 16, 2024

Conversation

araujobarret
Copy link
Contributor

Description

Fixes loading states similar to what was done in #243.
Co-authored by @gkartalis

Simulator.Screen.Recording.-.iPhone.14.-.2024-09-16.at.16.56.41.mp4

@araujobarret araujobarret self-assigned this Sep 16, 2024
@artsy-peril artsy-peril bot added the Version: Minor A deploy for new features label Sep 16, 2024
@araujobarret araujobarret changed the title wip button refactor(Button): enable loading state for button Sep 16, 2024

{loading ? (
{loading && (
Copy link
Member

Choose a reason for hiding this comment

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

🤔 can't believe that this was the issue 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it wasn't we we missing the loader colors and the transparencies to the text too

Copy link
Member

Choose a reason for hiding this comment

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

I get that the colors were missing but what about the size of the button? It is not affected by the colors right?

@araujobarret
Copy link
Contributor Author

Added a missing fix on the follow button here 1929b17

result:

Simulator.Screen.Recording.-.iPhone.14.-.2024-09-16.at.17.26.58.mp4

@araujobarret araujobarret merged commit 8f6a59c into main Sep 16, 2024
6 checks passed
@araujobarret araujobarret deleted the gkartalis/fix-btn-issues branch September 16, 2024 15:51
@artsyit
Copy link
Collaborator

artsyit commented Sep 16, 2024

🚀 PR was released in v13.2.30 🚀

@artsyit artsyit added the released This issue/pull request has been released. label Sep 16, 2024
araujobarret added a commit that referenced this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released. Version: Minor A deploy for new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants