Problem/Motivations

`password-confirm-message` element that is used by user registration form is generated from JS code. This means we can not inject `description` CSS class in it. We want this element to look the same way as a description. Because reusing existing CSS classes is not an option we will need to add these rules manually.

CommentFileSizeAuthor
#5 3158035-5-password-confirm-message.patch684 bytespivica
#4 3158035-4-password-confirm-message.patch737 bytespivica
#4 interdiff-3158035-2-4.txt976 bytespivica
#2 3158035-password-confirm-message.patch999 bytespivica

Comments

pivica created an issue. See original summary.

pivica’s picture

pivica commented
Status: Active » Needs review
StatusFileSize
new 3158035-password-confirm-message.patch999 bytes

Here is a patch.

pivica’s picture

pivica commented
Related issues: +#3008285: Add support for Bootstrap floating labels

Note that this patch is dependent on the latest patch in #3008285: Add support for Bootstrap floating labels.

pivica’s picture

pivica commented
StatusFileSize
new interdiff-3158035-2-4.txt976 bytes
new 3158035-4-password-confirm-message.patch737 bytes

Seems that using of text-emphasis-variant('.description', $text-muted) is creating some problems in additional rules because of !important. Let's remove it and use plain color rule.

pivica’s picture

pivica commented
StatusFileSize
new 3158035-5-password-confirm-message.patch684 bytes

I needed to reverse dependency here, so this patch is now going first and then #3008285: Add support for Bootstrap floating labels which has some additional fixes based on patch in this issue.

  • pivica committed 0e05b1a on 8.x-1.x 
    Issue #3158035 by pivica: password-confirm-message should behave in the...
pivica’s picture

pivica commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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