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

style(ui): [toaster] add colors for different states #8

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

okt-limonikas
Copy link
Collaborator

@okt-limonikas okt-limonikas commented Jan 12, 2024

Adds colors for different states and variants of toast e.g warning/error/success

@okt-limonikas okt-limonikas changed the title Add toaster colors style(ui): [toaster] add colors for different states Jan 12, 2024
@okt-limonikas okt-limonikas self-assigned this Jan 12, 2024
@okt-limonikas okt-limonikas marked this pull request as ready for review January 12, 2024 19:23
@okt-limonikas okt-limonikas added the ✨ Enhancement New feature or request label Jan 12, 2024
@okt-limonikas okt-limonikas requested review from arybchik and k0nstantin and removed request for arybchik January 12, 2024 19:23
@@ -19,7 +19,11 @@ const Toaster = ({ ...props }: ToasterProps) => {
actionButton:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Sorry, which states, which view, why?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It's not one view, it's for notification system in case we show some message.
Currently it's in monochrome tone so added colours

Commit message improved

Screenshot 2024-01-15 at 05 59 47

@okt-limonikas okt-limonikas requested review from okt-kostik and removed request for k0nstantin January 15, 2024 03:55
Added different icon colors for different toast types e.g. success|warning|info|error, so it's easier to distinguish if something failed, or it's some general information or tip message

Signed-off-by: Danil Kostromin <[email protected]>
@okt-limonikas okt-limonikas merged commit 2955b94 into main Jan 17, 2024
1 check passed
@okt-limonikas okt-limonikas deleted the limonikas/toaster-colors branch January 17, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants