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

LATCH-987 Fix Snackbar button color #304

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

jaranaz
Copy link
Contributor

@jaranaz jaranaz commented Sep 22, 2023

🎟️ Jira ticket

https://jira.tid.es/browse/LATCH-987

πŸ₯… What's the goal?

Component Snackbar is using token textLink for its action button, instead of the right token textLinkSnackbar

🚧 How do we do it?

Create new MisticaButtonStyle type called .misticaSnackbar with the right color token.

πŸ§ͺ How can I verify this?

Screenshot 2023-09-22 at 12 06 50

Screenshot 2023-09-22 at 12 07 01

@jaranaz jaranaz self-assigned this Sep 22, 2023
@jaranaz jaranaz requested review from a team, pbartolome, WanaldinoTelefonica and yceballost and removed request for a team September 22, 2023 10:10
@github-actions
Copy link

github-actions bot commented Sep 22, 2023

Screenshot tests report

βœ”οΈ All passing

@jaranaz jaranaz force-pushed the LATCH-987-fix-snackbar-button-color branch from d3c015f to 35ce5b7 Compare September 22, 2023 10:16
static func misticaSnackbar(
small: Bool = false,
bleedingAlignment: ButtonBleedingAlignment = .none,
withChevron: Bool = false
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO this should not be avalable as chevron is only available on link style

Copy link
Contributor

@WanaldinoTelefonica WanaldinoTelefonica left a comment

Choose a reason for hiding this comment

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

πŸ‘

@jaranaz jaranaz merged commit 7054e3d into main Sep 25, 2023
2 checks passed
@jaranaz jaranaz deleted the LATCH-987-fix-snackbar-button-color branch September 25, 2023 12:03
tuentisre pushed a commit that referenced this pull request Oct 16, 2023
# [26.2.0](v26.1.0...v26.2.0) (2023-10-16)

### Bug Fixes

* **Button:** change colors for disabled style ([#308](#308)) ([e829160](e829160))
* **Button:** change colors for disabled style ([#310](#310)) ([fb108a1](fb108a1))
* **Button:** Change cornerCurve from continuous to circular ([#307](#307)) ([26a73b3](26a73b3))
* **Snackbar:** change color for action button ([#304](#304)) ([7054e3d](7054e3d))
* **Tag:** change foreground and background colors according to specs ([#309](#309)) ([6a9f441](6a9f441))

### Features

* **Button:** Added accesibility to right image (chevron) ([05d77d9](05d77d9))
@tuentisre
Copy link
Collaborator

πŸŽ‰ This PR is included in version 26.2.0 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

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.

5 participants