Problem/Motivation

The current code assumes that that input box is quite wide (240px), at least wider than the style in Drupal 7 core (182px). As a result, the text for the password strength (e.g. Password strength:Very Weak) gets broken down into 2 lines.

Proposed resolution

Maybe don't restrict the width of the text, and don't do any right align?

Remaining tasks

* The progress bar overlays the field in an odd way on admin/config/system/password-strength and the password strength words wrap a line
* In the "Seven" admin theme, when editing users, the progress bar overlays the field too much and the strength words wrap a line

User interface changes

API changes

Comments

scor’s picture

Issue summary: View changes
scor’s picture

Issue summary: View changes
StatusFileSize
new20.23 KB
coltrane’s picture

Project: Password Strength2 » Password Strength
Version: » 7.x-1.x-dev

(moving to Password Strength project)

greggles’s picture

Status: Active » Needs review
StatusFileSize
new165.59 KB
new482 bytes

OK, I think this css fixup gets 2 bugs (demonstrated in the attached screenshot).

Basically the problem is that the font-size was being applied at too high of a wrapper level. Which might override with the font-size associated with the password input which might then make that box have a narrower width.

coltrane’s picture

The CSS changes look good on account registration form but not on the user edit form or the Password Strength configuration form.

The strength only appears inline on user edit form when it's level "weak". All others are not lined up.

Configuration form starts with strength meter over the form input when empty.

The strength level is also not lined up once displayed.

coltrane’s picture

Actually I was able to see the user edit and config form oddities without this patch applied so I think its an overall styling issue and not something caused by this patch.

The config form meter bar appears over the field whenever the collapsible fieldset is collapsed which happens when the default strength is > 0.

greggles’s picture

Issue summary: View changes

I've updated the issue summary to discuss those remaining issues under the "remaining tasks".

In my testing this patch doesn't cause any regressions to the layout of the fields.

I believe that the attached patch fixes the registration form and editing an account for non-admins. I suggest committing it as-is since it fixes the most important case (user-facing display). The other items can be followup tasks.

greggles’s picture

Status: Needs work » Needs review

Rather this.

  • coltrane committed 92cdd49 on 7.x-1.x authored by greggles
    Issue #2298875 by greggles: Fix styling issues for password strength...
coltrane’s picture

Status: Needs review » Active

Committed #4 patch. Will leave open to look into remaining issues.

scor’s picture

Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new939 bytes

For the strength to be in one line, we're just going to have to increase the length of the password from 16em to 20em, which is included in this patch.

I've also fixed the strength bar. The additional margin was caused by additional pixels added in the javascript.

greggles’s picture

Can you post before/after screenshots?

scor’s picture

StatusFileSize
new30.32 KB
new28.86 KB

Sure!


scor’s picture

StatusFileSize
new943 bytes
new30.74 KB

I made a 1 pixel adjustment.


  • scor committed 3bf8123 on 6.x-2.x
    Backport Issue #2298875 by greggles: Fix styling issues for password...

  • coltrane committed 03adb40 on 7.x-1.x authored by scor
    Issue #2298875 by scor, greggles: Fix styling issues for password...
coltrane’s picture

Status: Needs review » Fixed

Committed. Thanks!

scor’s picture

Version: 7.x-1.x-dev » 6.x-2.x-dev
Status: Fixed » Patch (to be ported)

moving to 6.x to make sure all patches were committed there too.

scor’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new1.06 KB

D6 needs a bit of extra CSS love.

scor’s picture

Status: Needs review » Fixed

  • scor committed 40128f6 on 6.x-2.x
    Backport Issue #2298875 by greggles, scor: Fix styling issues for...

Status: Fixed » Closed (fixed)

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