-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Web: add a tooltip summary for aws oidc configure step #46934
Conversation
Is there a better way to display the IAM role? The editor feels too big for a tooltip, and having two scrollbars show up feels like a poor UX. |
The script will describe specifics like the trust policy. I think the UI tooltip should just summarize in plain sentences |
Yeah, I think it could just be something like "configures an IAM role called |
I think showing the IAM policy here would be helpful too, for user reference. Can we borrow the "sliding out side panel" UX trick from TAG and show it in the panel that slides off of the right edge to view the full policy? |
754f60d
to
06d35a3
Compare
i discussed this with kenny, xin, and dave: we agreed to keep the tooltip for now (with wider width, to reduce horizontal scrolling), until we finalize how we want to better represent these type of information eg: side bar. (there are other places in discover where we use same tooltip for similar info) |
06d35a3
to
f7744ac
Compare
Sounds good to me. (Some day we really should get the ACE editor to use a light syntax theme when Teleport is in light mode). |
this is for enrolling aws oidc integration flow
checkout story: https://localhost:9003/?path=/story/teleport-integrations-enroll-awsoidc--flow