It would be great to have css classes in the statusmessages so you can add some coloring for the pass / fail status indications

Comments

AndersNielsen created an issue. See original summary.

AndersNielsen’s picture

Status: Active » Needs review
StatusFileSize
new819 bytes

Patch attached

aohrvetpv’s picture

CSS classes need to be namespaced (i.e., prefixed with the module name) to prevent possible collisions with other classes. That's my understanding at least. Modified patch attached.

aohrvetpv’s picture

StatusFileSize
new465 bytes
paddy.doyle’s picture

Looks good to me.

AndersNielsen’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me as well

nerdstein’s picture

Status: Reviewed & tested by the community » Needs work

This looks good, especially for customizing the experience. The patch looks like it needs re-rolled; it does not apply

voleger’s picture

Status: Needs work » Needs review
StatusFileSize
new1.28 KB

rerolled patch

  • nerdstein committed 58a7403 on 8.x-3.x authored by voleger
    Issue #2858693 by AohRveTPV, AndersNielsen, voleger: Add css class for...
nerdstein’s picture

Status: Needs review » Fixed

Thanks, @voleger and all who contributed. Merging and marking as fixed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.