New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Password reset button for managers #335
Conversation
ephios/core/templates/core/userprofile_confirm_password_reset.html
Outdated
Show resolved
Hide resolved
ephios/core/templates/core/userprofile_confirm_password_reset.html
Outdated
Show resolved
Hide resolved
@@ -19,6 +19,13 @@ <h1>{% trans "Edit user" %}</h1> | |||
<h1>{% trans "Add new user" %}</h1> | |||
{% endif %} | |||
</div> | |||
{% if userprofile %} | |||
<div class="float-right d-flex"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like that the button is so close to the field below and it appears kinda lost up there. Maybe put it next to delete button which triggers a "similar" action? Also I'd prefer to have another color for the button as this now indicates the same destructivity as deleting the user.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…n-for-managers' into enhancement/password-reset-button-for-managers
closes #287