-
Notifications
You must be signed in to change notification settings - Fork 293
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
chore: update style of xterm terminal #8986
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Sonia Sandler <[email protected]>
Signed-off-by: Sonia Sandler <[email protected]>
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.
hello, it is not possible to include the padding in the terminal component ?
AFAIK we should not have to include css file each time we're including a component
@benoitf I went through the |
Signed-off-by: Sonia Sandler <[email protected]>
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.
side note not for this commit: should we remove also import import '@xterm/xterm/css/xterm.css';
from the various place
as it's imported by TerminalWindow
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. Shoudn't we have the same padding left and 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.
LGTM
What does this PR do?
This PR adds padding to all terminal instances
Screenshot / video of UI
before:
after:
What issues does this PR fix or reference?
Closes #8079
How to test this PR?