Skip to content

Conversation

@dgdavid
Copy link
Contributor

@dgdavid dgdavid commented Mar 26, 2025

Refactor the alert design to stop rendering them as "boxes", which were cluttering the interface. Instead, they now use a left border and a gradient background to maintain visual hints about the scope of their content without the boxy appearance.

Before After
localhost_8080_ (29) localhost_8080_ (28)

Refactor the alert design to stop rendering them as "boxes", which were
cluttering the interface. Instead, they now use a left border and a
gradient background to maintain visual hints about the scope of their
content without the boxy appearance.
@dgdavid dgdavid requested a review from ancorgs March 26, 2025 21:29

$pf-alert-modifiers: "custom", "info", "success", "warning", "danger";

@each $modifier in $pf-alert-modifiers {
Copy link
Contributor

Choose a reason for hiding this comment

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

Wow!

Copy link
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

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

LGTM

@imobachgs imobachgs merged commit 88c40d5 into storage-lvm Mar 27, 2025
3 checks passed
@imobachgs imobachgs deleted the change-alerts-look-and-feel branch March 27, 2025 11:04
@imobachgs imobachgs mentioned this pull request Mar 27, 2025
imobachgs added a commit that referenced this pull request Mar 27, 2025
@imobachgs imobachgs mentioned this pull request Apr 22, 2025
imobachgs added a commit that referenced this pull request Apr 22, 2025
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.

4 participants