-
Notifications
You must be signed in to change notification settings - Fork 271
[Feature][0.9][Merged] Alert messages can now have HTML content #251
Conversation
border-color: #f8d053; | ||
} | ||
|
||
/* FLEXBOX HELPERS */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has NOTHING to do with the current PR, but it's needed and harmless, so I say let's leave it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needed by what though?
border-color: #f8d053; | ||
} | ||
|
||
/* FLEXBOX HELPERS */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needed by what though?
new PNotify({ | ||
// title: 'Regular Notice', | ||
text: "{{ $message }}", | ||
text: "{!! str_replace('"', "'", $message) !!}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup. Just double-checked to be sure: https://cl.ly/3C2V1s0i1Y26
Merged into upgrade branch #259 . We'll leave this PR and branch open, and push fixes here if anything comes up in testing. |
So that you can do things like:
This is NOT a breaking change, it's backwards-compatible, but we should make sure we don't use this feature ourselves in the code, until it's adopted. For this reason, I think it should be launched with the 3.4 update.