Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
There are a number of places in core...
... which put links in status, warning and error messages.
It would be helpful to have links in status, warning and error messages, since they typically have different background colors from the main theme, and links are often different than standard link colors.
Proposed resolution
Make the styleguide module output at least one link in the status, warning and error messages.
Remaining tasks
Write patchReview & RTBC- Commit
- Release a new version of Styleguide that includes this functionality.
User interface changes
A link is added to the status, warning and error messages.
API changes
None.
Original report by @jessehs
It would be helpful to have links in status, warning and error messages, since they typically have different background colors from the main theme, and links are often different than standard link colors.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 2217329-add-link-to-messages-1.patch | 1.94 KB | nevergone |
Comments
Comment #1
jessehsPatch adds an optional $link argument to the styleguide_sentence() function that places a link at the beginning of the sentence.
Comment #2
nevergone CreditAttribution: nevergone commented#1: PHP Notice message:
Notice: Undefined offset: 2 in styleguide_sentence() function (styleguide/styleguide.module line 429).
Corrected patch with theme_link().
Comment #3
mparker17Code looks good and functions properly. As far as I'm concerned, this is RTBC!
Comment #4
mparker17Updated issue summary, now with more motivation :)
Comment #6
dead_arm