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

IsSelected vs IsActive behavior changed from 3.x to 4.1/4.2? #163

Closed
derammo opened this issue Jun 2, 2020 · 4 comments · Fixed by #243
Closed

IsSelected vs IsActive behavior changed from 3.x to 4.1/4.2? #163

derammo opened this issue Jun 2, 2020 · 4 comments · Fixed by #243

Comments

@derammo
Copy link

derammo commented Jun 2, 2020

I have a collection of tabbed documents. When I dynamically add a LayoutDocument child to the LayoutDocumentPane, I would like the new document to be the active/selected tab.

In 3.x, setting IsSelected to true resulted in the new LayoutDocument being selected and receiving focus.

In 4.x (I tried 4.10 and 4.20) the same behavior requires setting IsActive to true also? Is this as designed?

@Dirkster99
Copy link
Owner

This sounds like a bug to me. Can you give me a sample application please (you can attach a zip file to the issue) and let me know with which version this used to work and where it stopped working?

Thanx for raising the issue Drk

@derammo
Copy link
Author

derammo commented Jun 6, 2020

It is unlikely that I will find time soon to do this, as I am working on getting a release out myself. :)

@MariusMichelPrimes
Copy link

I had similar issues in our application. I added

protected override void OnSelectionChanged(SelectionChangedEventArgs e)
{
    base.OnSelectionChanged(e);

    if (_model.SelectedContent != null)
        _model.SelectedContent.IsActive = true;
}

to the LayoutDocumentPaneControl.cs file and it seems to behave like before.

@Dirkster99
Copy link
Owner

Closed without intention -> Re-open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants