-
Notifications
You must be signed in to change notification settings - Fork 11
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
[IDEA][BLAZOR] Editable user control should stop updating value when focused and value binding should be switched to 'Edit' property of primitive type #31
Merged
PTKu
merged 6 commits into
dev
from
9-_IDEA_BLAZOR_Editable_user_control_should_stop_updating_value_when_focused_and_value_binding_should_be_switched_to_Edit_property_of_primitive_type
Jan 31, 2023
Merged
[IDEA][BLAZOR] Editable user control should stop updating value when focused and value binding should be switched to 'Edit' property of primitive type #31
PTKu
merged 6 commits into
dev
from
9-_IDEA_BLAZOR_Editable_user_control_should_stop_updating_value_when_focused_and_value_binding_should_be_switched_to_Edit_property_of_primitive_type
Jan 31, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@PTKu there are some tests fails, probably because of our change of Edit property in OnlinerBase`1.cs file. Can you please look at it? |
PTKu
commented
Jan 31, 2023
src/ix.blazor/src/Ix.Presentation.Blazor.Controls/RenderableContent/RenderableComponentBase.cs
Outdated
Show resolved
Hide resolved
src/ix.connectors/src/Ix.Connector/ValueTypes/Onlines/OnlinerBase`1.cs
Outdated
Show resolved
Hide resolved
Yes, that's good, I've see that... we must uncomment the EditValue it is highlighted in the comments int this PR |
…top_updating_value_when_focused_and_value_binding_should_be_switched_to_Edit_property_of_primitive_type
…ng_value_when_focused_and_value_binding_should_be_switched_to_Edit_property_of_primitive_type' of https://github.com/ix-ax/ix into 9-_IDEA_BLAZOR_Editable_user_control_should_stop_updating_value_when_focused_and_value_binding_should_be_switched_to_Edit_property_of_primitive_type
PTKu
deleted the
9-_IDEA_BLAZOR_Editable_user_control_should_stop_updating_value_when_focused_and_value_binding_should_be_switched_to_Edit_property_of_primitive_type
branch
January 31, 2023 14:40
PTKu
added a commit
that referenced
this pull request
Feb 4, 2023
* adds posibility to update version in an apax file * add publishing of Ix.Compiler package * push Ix.Compiler.Abstractions package (#39) * [IDEA][BLAZOR] Editable user control should stop updating value when focused and value binding should be switched to 'Edit' property of primitive type (#31) * Create draft PR for #9 * added edit property binding for control presentation type --------- Co-authored-by: Specter-13 <Specter-13@users.noreply.github.com> Co-authored-by: Specter-13 <56168909+Specter-13@users.noreply.github.com> * [DOCUMENTATION] Add conceptual overview of IX (#41) * Create draft PR for #40 * [documentation] documention update - short conceptual overview - fixed API doc generation * additional examples Blazor renderer --------- Co-authored-by: PTKu <PTKu@users.noreply.github.com> * Create sample of various structures for the renders for additional UI testing (#43) * Create draft PR for #30 * added GroupLayout type attribute, changed way of sass compiling, fix grouplayouts in st files * post merge refactoring * refactoring of templates, fixed tests * fixed test * add docs for group layout --------- Co-authored-by: Specter-13 <Specter-13@users.noreply.github.com> Co-authored-by: Specter-13 <56168909+Specter-13@users.noreply.github.com> * [IDEA] More integration tests for Blazor renderer (#48) * Create draft PR for #47 * Added tests for each layouts and groups. --------- Co-authored-by: Brano5 <Brano5@users.noreply.github.com> Co-authored-by: Branko Zachemsky <bzachemsky@gmail.com> * fixes readme in blazor template (#49) * [fix] bulk read request test reduces due to '#19' --------- Co-authored-by: Specter-13 <Specter-13@users.noreply.github.com> Co-authored-by: Specter-13 <56168909+Specter-13@users.noreply.github.com> Co-authored-by: PTKu <PTKu@users.noreply.github.com> Co-authored-by: Brano5 <Brano5@users.noreply.github.com> Co-authored-by: Branko Zachemsky <bzachemsky@gmail.com>
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #9