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
CONSOLE-2361: Replace Bootstrap float utility class with PF equivalent #9517
CONSOLE-2361: Replace Bootstrap float utility class with PF equivalent #9517
Conversation
Wondering if we can't maintain the layout here without adding a new partial just to float two items? |
Seems like https://www.patternfly.org/v4/layouts/level would give us what we want here. |
f35e670
to
304b8d9
Compare
Switched to use |
Code changes look good. Maybe some screenshots to help QE verify? |
/label qe-approved |
This PR is verified on private build and did not find any issue, the layout and view of ''Download YAML" button is not being impacted on this changes |
<LevelItem> | ||
<Button | ||
type="button" | ||
className="pf-u-float-right" |
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.
Whoops. Looks like this was forgotten.
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.
yanked it
304b8d9
to
969409b
Compare
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhamilto, sg00dwin The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
Format for "Try It" "Download YAML" buttons.