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

Fixed not been able to control ReadOnlyMode property using one-way data binding (#133) #134

Merged
merged 1 commit into from
Mar 30, 2023

Conversation

Filipsi
Copy link
Contributor

@Filipsi Filipsi commented Mar 30, 2023

This pull request fixes issue #133 that prevents ReadOnlyMode property from been controlled by one-way data binding.
There are however more issues with the same underlining problem that remain as they are.

The problem is that the editor is changing it's own its own dependency properties, which resets the value source configured by the user of the control.

Control should always use SetCurrentValue to set the value of its own dependency properties, since it changes the effective value of the property, but keeps existing triggers, data bindings, and styles.

…ta binding

A control should always use SetCurrentValue to set the value of its own dependency properties to preserve value source.
@abbaye
Copy link
Owner

abbaye commented Mar 30, 2023

Thank for your help and support 😊

@abbaye abbaye merged commit c9f1edf into abbaye:master Mar 30, 2023
@Filipsi
Copy link
Contributor Author

Filipsi commented Mar 30, 2023

Thanks for accepting the pull request. Are you planning on making a new NuGet release sometime soon?
I would like to use the base offset feature added in #128 alongside with this fix.

@abbaye
Copy link
Owner

abbaye commented Mar 30, 2023

Yes is possible to make a new release of the Nuget package

@myd7349 myd7349 mentioned this pull request Feb 7, 2025
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 this pull request may close these issues.

2 participants