-
Notifications
You must be signed in to change notification settings - Fork 882
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
Fixed constructors of LayoutAnchorableFloatingWindowControl and Layou… #1528
Conversation
…tDocumentFloatingWindowControl
This is dublicate with Issue #1442 - please read the attached thread to understand why this is not fixed in version 3.5 of this project (the short version is that the current version of the open source version of the toolkit is no longer actively maintained). |
Hi Dirk, Thank you for pointing that out. I should have checked before adding my pull request. I was already aware of Xceed's policy to delay fixes in open source version of wpftoolkit with two versions. I will most probably give a try to your fork! :) |
Yes, you could say they implement an early crash adding and late fixing policy. Unfortunately, its a useless contribution and you only find out until after you incorporated it in your application which crashs on all kinds of random interactions 👎 thats when they want you to pay for it 💯 but whats even more fun: the current version is not much better in terms of quality then the open source version - so, you end up wasting your time and your money... |
Hi all, The OpenSource version is still maintained and will always be. We love to ear what the community is using, how they work with our controls and if they find any bugs so we can fix them. You can try the Plus version for free for 45 days and you have all the benefits of the paying customers: The OpenSource version is not updated quite often, but we are thinking about a way to improve this. We understand your concerns of late fixes, but you need to understand that this is a version which is updated for free ! |
I have to say the constant ‘plugging’ for the paid ‘Plus’ version is a little frustrating when dealing with open-source versions. Yes, we know there is a paid version with more features and enhancements. But perhaps we don’t need that. We just need what’s in the open-source version. To say you’re delaying fixes in the open source version but not in the plus version, especially when those fixes are made by the open source community I feel is a little disingenuous to the open source community and the philosophy thereof. Don’t get me wrong. I understand Xceed is a company that’s trying to make a profit and it is nice to release an open source version at all, and of course you provide better support when a customer has paid, but bug fixing is not something that should be delayed, let alone by several versions, let alone when those fixes are contributed back to the open source project. That’s holding the open source code hostage if you don’t pay. After all, if someone fixes something in the open source version, we’re assuming that fix would, if needed, also make it back into the paid version, right? So that means we’ve taken the time to fix something that you’re benefitting from in a paid product while purposely not fixing it in the open source product where it was originally applied. Again, this just tastes so wrong to me. When you contribute something to open source, you are making somewhat of a promise to the open source community that you are working with them on it. You are all contributing to make it better. You also agree to, as the owner of the repo, stay on top of such things. After all, the only thing that has to happen is testing, then merging. It should not delay a fix to wait for something in the pro version, which is unrelated. By your own admission, they are several versions behind, therefore it’s a different product. Please start to work on committing bug fixes to the OS versions sooner.- It helps everyone and shows us you’re a company that cares about the code, not just cold profits. |
I understand your point MarqueIV and as said in another post, things will change for the Community version of the Toolkit. Stay tuned, more versions are coming real soon !! |
It seems to me that in both LayoutAnchorableFloatingWindowControl and LayoutDocumentFloatingWindowControl classes the constructors with single parameter should call the other constructor of the respective class instead of the base class as otherwise at least _model field won't get initialized and there are problems afterwards.