-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Address review feedback for redesigned user profile page #132645
Comments
Pinging @elastic/kibana-security (Team:Security) |
@azasypkin if you want, feel free to split this out into separate issues -- it looks like there is some low-hanging fruit that either @jeramysoucy or @kc13greiner could pick up if you wanted to divide and conquer this. Up to you! |
It's a great idea! I'm going to outline actionable items and then we can deal with them together with @jeramysoucy and/or @kc13greiner. |
I'll start on this! |
Im going to start on this now: #127624 (review) |
Starting on this |
I'm going to remove |
Sounds great to me! Let me know what I can do to help.
…On Thu, Jun 9, 2022 at 7:14 PM Aleh Zasypkin ***@***.***> wrote:
@azasypkin <https://github.com/azasypkin> if you want, feel free to split
this out into separate issues -- it looks like there is some low-hanging
fruit that either @jeramysoucy <https://github.com/jeramysoucy> or
@kc13greiner <https://github.com/kc13greiner> could pick up if you wanted
to divide and conquer this. Up to you!
It's a great idea! I'm going to outline actionable items and then we can
deal with them together with @jeramysoucy <https://github.com/jeramysoucy>
and/or @kc13greiner <https://github.com/kc13greiner>.
—
Reply to this email directly, view it on GitHub
<#132645 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYY7Z7A3DHRWEBSVRG3TYKTVOIQ6BANCNFSM5WQEMW6Q>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Feedback to address (#127624 (review)):
✔️ Fix left/right padding of the
Profile
page template Redesigned user profile page #127624 (review)✔️ Simplify form label markup (or confirm that proposed simplification doesn't work): Redesigned user profile page #127624 (comment) + Introduce user profiles #132522 (comment)
✔️ Elastic Cloud managed users shouldn't be able to see avatar editor (we need to collaborate with @MichaelMarcialis on this one)
Profile
page, that might or might not be what we want.Profile
,User settings
andPreferences
words)* Support for viewing another user's avatar Support for viewing another user's avatar #132636The text was updated successfully, but these errors were encountered: