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

LayoutDocument CanClose is false but still showing close button #196

Closed
zaksmolen opened this issue Sep 2, 2020 · 4 comments
Closed

LayoutDocument CanClose is false but still showing close button #196

zaksmolen opened this issue Sep 2, 2020 · 4 comments

Comments

@zaksmolen
Copy link

zaksmolen commented Sep 2, 2020

It looks like there was stuff done in regard to this with LayoutAnchorables but was the same not fixed inLayoutDocuments?

We just updated our Xceed DockingManager, freaked out, cursed a bunch, and then very happily found this fork. It seems to be pretty much a drop-in replacement, but one slight difference I have noticed so far is that LayoutDocuments with CanClose set to false still show the closing "x" where they used to not show an "x" and didn't even take space for it. The "x" doesn't work, it does not close the window, but it's still showing up or leaving a placeholder space for it. If I change some things to LayoutAnchorables the "x" does not show, but there are other issues and the LayoutDocuments seem to work better for my needs. Here's a comparison between the same code with old and new libraries:
Old:
image

New:
image

This isn't an especially major issue, but knowing our users, they'll all try to click an X if it's there, and complain when it doesn't do anything! I'm happy to just hide it with styling or anything like that as well.

@Dirkster99
Copy link
Owner

Dirkster99 commented Sep 4, 2020

I think I found the issue in the binding, which does not consider the situation in which a LayoutDocumentControl (LayoutDocument) does not provide a CanHide property, and so we've overseen the situation for the LayoutDocument since we were focused on LayoutAnchorable.

Fortunately, the fix seems to be rather simple as outlined in PR #198 (relevant changes are only FallbackValue=False) - could you please test this for me?

@bdachev What's your opinion about this fix?

@bdachev
Copy link
Contributor

bdachev commented Sep 4, 2020

Dirk, we don't have such case in our software so to be honest I haven't thought of it - sorry my fault. I was wondering if it wouldn't be simpler to just add CanHide property to LayoutDocument class always returning false. But proposed solution seems valid as well and if you have tested and it works as expected please go for it!

@OgreTransporter
Copy link

I use nuget version 4.50.1 and in that version the bug is still present.

@Khaos66
Copy link
Contributor

Khaos66 commented Mar 19, 2021

I think I had the same issue. But as I wanted to style the tabs differently I had to copy and change the styles and templates anyway ;)

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

No branches or pull requests

5 participants