-
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
[Session View] Update User icon to the new one #131101
Comments
@codearos can you help me with a summary for me to add on the EuiIcon PR, of why we're proposing a change to the user Icon? |
@opauloh Sure.
|
Thanks, @codearos Pushed a PR into Eui proposing the Icon change: elastic/eui#5877 |
Hi @opauloh we have validated this issue on 8.3.0 BC2 but mentioned icon of user is not there. we will going to re-test the issue on BC3 in case pr commit got missed to there in BC2 . Build Details:
Screen-Cast: Expected: ✔️ Actual: ❌ c.c @MadameSheema |
Hi @karanbirsingh-qasource, Just checked and it looks like the EUI upgrade PR in Kibana which contains the EuiIcon changes won't make the Since it's not critical, I'm going to target this issue for |
thanks for digging into this @opauloh |
thanks @opauloh for looking into the comment and sharing the update 👍 |
Summary
The AWP Design team proposes a new user Icon for the Session View, as stated on the ➡️ Figma file, as the Eui Library already have a User Icon we're going to propose a change instead of adding a new user Icon
Screenshot of the new User Icon:
The text was updated successfully, but these errors were encountered: