MudAlert: Fix content alignment issue #8734#8735
Conversation
| .mud-alert-position { | ||
| flex: 1; | ||
| display: flex; | ||
| align-items:center; |
There was a problem hiding this comment.
nit: missing space between align-items: and center;
|
Thanks! |
|
As I understand this fixed this issue #8734 but it wasn't mentioned in the PR |
|
@danielchalmers , this change has caused a visual difference from v6. Previously, the severity icon was aligned to the top (start) of the alert content. It is now aligned in the center of the mud-alert container. This is only noticeable with an alert that wraps to more than one line... This was caused by the addition of "align-items: center;" to the .mud-alert-position css class. While it's probably a subjective preference, I much prefer the icon to align with the top of the text for long alerts as it did in v6. A workaround is to add |
|
@gabephudson Could you post a before and after? |
|
@gabephudson Thanks! I agree with you that it should be at the top. Do you want to submit a PR? CSS should be in here: MudBlazor/src/MudBlazor/Styles/components/_alert.scss Lines 105 to 109 in b89bbe8 |
|
I'm an ex-programmer and a highly technical product person now for our org, so I've never summitted a PR or have much practice with Git/GitHub. That said, I would love to learn. So I'm going to give it a try. :) Let me know if there is anything out the box I may need to do, otherwise I'll have ChatGPT walk me through the process, lol. ;) |
|
Here is a rough list of things to do:
|
|
Thanks for the guidance, everyone! I have submitted the simplest of PRs. :) First time for everything. Hopefully I executed the PR correctly. |


before
not alignment
"mud-alert-position justify-sm-start"
fixed
mud-alert-position align-center justify-sm-start