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.

Sign up for GitHub

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

Jump to bottom

Removed the check for non-floating content #451

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Removed the check for non-floating content #451

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Removed the invalid check for non-floating content before clearing th… 
…e PreviousContainer during LayoutRoot.CollectGarbage.
  • Loading branch information
frsa committed Aug 27, 2023
commit b7431a378fa2578ce9f0b779fb3fae4e78a32aef
4 changes: 2 additions & 2 deletions source/Components/AvalonDock/Layout/LayoutRoot.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -383,9 +383,9 @@ public void CollectGarbage()
//for each pane that is empty
foreach (var emptyPane in this.Descendents().OfType<ILayoutPane>().Where(p => p.ChildrenCount == 0))
{
//...set null any reference coming from contents not yet hosted in a floating window
//...set null any reference coming from contents
foreach (var contentReferencingEmptyPane in this.Descendents().OfType<LayoutContent>()
.Where(c => ((ILayoutPreviousContainer)c).PreviousContainer == emptyPane && !c.IsFloating))
.Where(c => ((ILayoutPreviousContainer)c).PreviousContainer == emptyPane))
{
if (contentReferencingEmptyPane is LayoutAnchorable anchorable &&
!anchorable.IsVisible)
Expand Down