Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#28097 closed Bug (fixed)

Incorrect layout of ReadOnlyPasswordHashWidget widget in user admin

Reported by: Claude Paroz Owned by: Tim Graham
Component: contrib.auth Version: 1.11
Severity: Release blocker Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

With the new template-based widget rendering, the admin password widget display is a bit awkward. See attached screenshot. Also, I'm not sure if the the nesting of divs is wanted.

Attachments (1)

admin_password_widget.png​ (12.8 KB) - added by Claude Paroz 6 years ago.

Download all attachments as: .zip

Change History (7)

Changed 6 years ago by Claude Paroz

Attachment: admin_password_widget.png​ added

comment:1 Changed 6 years ago by Aymeric Augustin

This looks like a regression to me.

comment:2 Changed 6 years ago by Tim Graham

Component: contrib.admin → contrib.auth
Owner: changed from nobody to Tim Graham
Status: new → assigned
Summary: Strange layout of password widget in user admin → Incorrect layout of ReadOnlyPasswordHashWidget widget in user admin
Triage Stage: Unreviewed → Accepted

comment:3 Changed 6 years ago by Tim Graham

Has patch: set

​PR

comment:4 Changed 6 years ago by Claude Paroz

Triage Stage: Accepted → Ready for checkin

Thanks for the fix!

comment:5 Changed 6 years ago by GitHub <noreply@…>

Resolution: → fixed
Status: assigned → closed

In dff559f:

Fixed #28097 -- Fixed layout of ReadOnlyPasswordHashWidget.

comment:6 Changed 6 years ago by Tim Graham <timograham@…>

In cf1e682:

[1.11.x] Fixed #28097 -- Fixed layout of ReadOnlyPasswordHashWidget.

Backport of dff559ff83a6aac14de0bbca58101786edf4195f from master

Note: See TracTickets for help on using tickets.
Back to Top