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

Snackbar enable actions #713

Merged

Conversation

susannalandstrom
Copy link
Contributor

Background

Currently actions were not working in snackbars. This PR enables adding actions. Also small improvements to alignment of snackbar/alert content.

Related to this ticket.

@susannalandstrom susannalandstrom requested a review from a team as a code owner December 22, 2023 11:28
@susannalandstrom susannalandstrom merged commit e20f7fe into lyytioy:next Dec 22, 2023
1 check passed
@susannalandstrom susannalandstrom deleted the snackbar-enable-actions branch December 22, 2023 11:46
Copy link
Contributor

@grzegorz-bach grzegorz-bach left a comment

Choose a reason for hiding this comment

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

Ahh too late 😁 But something to consider in the future :)

ref={ref}
severity={severity}
variant={variant}
sx={{ alignItems: 'center', '& .MuiAlert-action': { pt: 0, pl: 5 } }}
Copy link
Contributor

@grzegorz-bach grzegorz-bach Dec 22, 2023

Choose a reason for hiding this comment

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

I would destructure sx prop and put it here to avoid overriding default sx props accidentally :)

Suggested change
sx={{ alignItems: 'center', '& .MuiAlert-action': { pt: 0, pl: 5 } }}
sx={{ alignItems: 'center', '& .MuiAlert-action': { pt: 0, pl: 5 }, ...sx }}

@susannalandstrom susannalandstrom mentioned this pull request Dec 22, 2023
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.

3 participants