Skip to content
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

Lines separating splitted terminal disappear when enable/disable menu bar #79226

Closed
alanlivio opened this issue Aug 15, 2019 · 2 comments
Closed
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities terminal Integrated terminal issues verified Verification succeeded
Milestone

Comments

@alanlivio
Copy link

alanlivio commented Aug 15, 2019

  • VSCode Version:
    $ code -version
    1.37.0
    036a6b1
    x64

  • OS Version:
    $ lsb_release -a
    No LSB modules are available.
    Distributor ID: Ubuntu
    Description: Ubuntu 19.04
    Release: 19.04
    Codename: disco

  • Steps to Reproduce:

  1. use integrated terminal at right and splited
  2. enable and disable menu bar
  3. the lines separating the splitted terminal disappear (see figures)
    Screenshot from 2019-08-15 12-42-20
    Screenshot from 2019-08-15 12-42-42
  • Does this issue occur when all extensions are disabled?: Yes/No
    Yes
@Tyriar
Copy link
Member

Tyriar commented Aug 16, 2019

This appears to happen only when the first row of the lower split pane sits on top of the sash. I guess it's a problem with the internals of the pane not sizing correctly.

@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities terminal Integrated terminal issues labels Aug 16, 2019
@Tyriar Tyriar added this to the Backlog milestone Aug 16, 2019
@jeanp413
Copy link
Contributor

This is caused by the border-top in split-view-view from this css rule

.monaco-workbench .panel.integrated-terminal .monaco-split-view2.vertical .split-view-view:not(:first-child) {
border-top-width: 1px;
border-top-style: solid;
}

which isn't taken into account when calculating the height of the terminal

const innerHeight = height - bottom;

Seems there's no easy way to get that value, maybe just always subtract 1 px from the total height const innerHeight = height - bottom - 1;

@Tyriar Tyriar modified the milestones: Backlog, October 2019 Oct 9, 2019
@Tyriar Tyriar closed this as completed in b2cb8ba Oct 25, 2019
@connor4312 connor4312 added the verified Verification succeeded label Oct 30, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities terminal Integrated terminal issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants