-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Styles] Update the paste, review, and install filebeat screen styles #7033
[Styles] Update the paste, review, and install filebeat screen styles #7033
Conversation
027ed00
to
833e2d6
Compare
833e2d6
to
302d5fe
Compare
jenkins, test it |
.paste-samples { | ||
textarea { | ||
width: 100%; | ||
height: 250px; |
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.
Do we want to make this 450px like the pipeline-setup step? @alt74
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.
@alt74 for reference, here's how 250px and 450px look, respectively. To me 450px looks a bit odd here since it starts out as a giant empty box.
Added #7095 to address this |
LGTM |
@Bargs @BigFunger around 250 px height for the sample logs box works for me (1. Paste screen). A really large box creates the impression I have to paste a lot of example logs. |
Paste Screen
Review Screen
Install Filebeat Screen