-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
User Details View Accessibility Improvement #6969
User Details View Accessibility Improvement #6969
Conversation
@poornimanayar this makes so much sense! Could I ask you to resolve the merge conflicts on this one, so we can review it? Please note that I have changed the base branch from |
# Conflicts: # src/Umbraco.Web.UI.Client/src/views/components/umb-mini-list-view.html
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.
Hi @poornimanayar ... thank you for merging things up.
It looks like you have accidentally broken the changes made in #7107 - the "change photo" button now shows again for users without a photo:
Could you please have another look at this?
I have added that back, i was clearly having a bad merge day yday |
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.
Thanks @poornimanayar, this works perfectly ⭐️
...and merged 👍 |
On the user details screen, the "Change Photo" and "Remove Photo" are currently
<a>
. Changed them to be link style buttons.