Skip to content

Fixes editor has wrong height when toggling tabs with breadcrumbs visible - #97502

Merged
Benjamin Pasero (bpasero) merged 3 commits into
microsoft:masterfrom
jeanp413:fix-97021
May 12, 2020
Merged

Benjamin Pasero (bpasero) merged 3 commits into
microsoft:masterfrom
jeanp413:fix-97021

Conversation

@jeanp413

Copy link
Copy Markdown
Contributor

This PR fixes #97021

@bpasero
Benjamin Pasero (bpasero) merged commit 4c467c1 into microsoft:master May 12, 2020
@bpasero Benjamin Pasero (bpasero) added this to the May 2020 milestone May 12, 2020
@bpasero

Copy link
Copy Markdown
Contributor

Thanks. I made changes on top to make this simpler: instead of calling layout from redraw, simply relayout when we detect the options change.

@jeanp413
Jean Pierre (jeanp413) deleted the fix-97021 branch May 12, 2020 16:20
@jeanp413

Copy link
Copy Markdown
Contributor Author

Awesome, much better 👍
Maybe some calls to relayout in tabsTitleControl are redundant now

private updateBreadcrumbsControl(): void {
if (this.breadcrumbsControl && this.breadcrumbsControl.update()) {
// relayout when we have a breadcrumbs and when update changed
// its hidden-status
this.group.relayout();
}

@bpasero

Copy link
Copy Markdown
Contributor

Jean Pierre (@jeanp413) not sure. the code I touched handles the transition between tabs being enabled or not while that method is about enabling breadcrumbs or not. I still think both cases need layout calls individually.

@github-actions github-actions Bot locked and limited conversation to collaborators Jun 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Editor has wrong height when toggling tabs with breadcrumbs visible

3 participants