-
Notifications
You must be signed in to change notification settings - Fork 75
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
Api token page UX issues #1224
Api token page UX issues #1224
Conversation
062fb5e
to
e404e54
Compare
@@ -95,7 +96,11 @@ class TokenPage extends React.Component<RouteComponentProps, IState> { | |||
<ClipboardCopy>{tokenData.refresh_token}</ClipboardCopy> | |||
</div> | |||
) : ( | |||
<Button onClick={() => this.loadToken()}>{t`Load token`}</Button> | |||
<div> |
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.
should this also have className='load-token'
?
There's one more
Apart from that (and the potentially missing load-token from insights), LGTM 👍 :) |
…, API token management.
e404e54
to
d18d29d
Compare
@himdel The classname="load-token" isn't needed on the button in insights mode, Susan says that button looks fine. It was just in standalone. I removed the scss import because it isn't being used. :) |
Hello :)
This pr address UX issues on the Token API page discussed here: https://issues.redhat.com/browse/AAH-826
API token management
.Before:
After: