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

welcome view content layout is broken? #95508

Closed
sandy081 opened this issue Apr 17, 2020 · 2 comments
Closed

welcome view content layout is broken? #95508

sandy081 opened this issue Apr 17, 2020 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded workbench-views Workbench view issues
Milestone

Comments

@sandy081
Copy link
Member

welcome view layout is handled by the ViewPane parent class

protected layoutBody(height: number, width: number): void {
this.viewWelcomeContainer.style.height = `${height}px`;
this.viewWelcomeContainer.style.width = `${width}px`;
this.scrollableElement.scanDomNode();
}

None of the sub classes (views) are calling super.layoutBody(). Does it mean that if any of these views has welcome view content then welcome view is not layout correctly?

@joaomoreno
Copy link
Member

I actually went through all the subclasses and added super.renderBody() to all renderBody() methods. Forgot to do the same for layoutBody...

@joaomoreno joaomoreno added this to the April 2020 milestone Apr 17, 2020
@joaomoreno joaomoreno added bug Issue identified by VS Code Team member as probable bug workbench-views Workbench view issues labels Apr 17, 2020
@sandy081 sandy081 added the verified Verification succeeded label Apr 30, 2020
@sandy081
Copy link
Member Author

Verified by code review

@github-actions github-actions bot locked and limited conversation to collaborators Jun 1, 2020
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 verified Verification succeeded workbench-views Workbench view issues
Projects
None yet
Development

No branches or pull requests

2 participants