make use of component-level CSS custom properties in Claro's form--password-confirm stylesheet

I have tested the MR #3822,
Confirm password variables now shifted into local scope from :root

Nothing broken in the UI side

Week

Fair

Good

Strong

The MR is largely declaring variables within style blocks and using them in the following lines. These are block scoped so there's no way to override or reuse them, so these wind up being redundant and unnecessary

For example the MR

  &.is-weak {
    --password-strength-bar-weak-border-color: var(--color-maximumred);
    --password-strength-bar-weak-bg-color: var(--color-maximumred);

    border-color: var(--password-strength-bar-weak-border-color);
    background-color: var(--password-strength-bar-weak-bg-color);
  }

Provides no advantage over

  &.is-weak {
    border-color: var(--color-maximumred);
    background-color: var(--color-maximumred);
  }

working on it

Worked upon the suggestion given in #7, I believe that declaring variables in the styles block is redundant and unnecessary. I have addressed this issue in commit #8. Please take a look

Does appear point #7 has been addressed

Posted question in the MR

For the open thread.

Addressed feedback from comment #12, please review

variables updated.

I'm triaging RTBC issues → .

The issue summary doesn't have a proposed resolution but this does appear simple enough than maybe it is isn't needed here. I didn't find any unanswered questions. Leaving at RTBC.

Committed c945327 and pushed to 11.x. Also cherry-picked to 10.2.x. Thanks!

lauriii → closed merge request !3822

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