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

fix: field content cannot be discarded or updated correctly #4

Merged
merged 2 commits into from
Jul 5, 2021

Conversation

schettn
Copy link
Member

@schettn schettn commented Jul 5, 2021

Checklist:

  • Have you added an explanation of what your changes do and why you'd like them to be included?
  • Have you updated or added documentation for the change?
  • Have you tested your changes with successful results?

Type of Changes:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation (non-breaking change which adds documentation)
  • Breaking change (fix or feature that would cause existing functionality to change)

What is the current behavior? (link to any open issues here)

What is the new behavior (if this is a feature change)?

Other information:

schettn added 2 commits July 5, 2021 02:31
Because of EditableField content do not update when the editing content changes the useEffect of the
Editor is not triggered on discard. By adding a resetTrigger this issue has been fixed.
When editing a StreamField the cursor now no longer jumps to the left side which made editing
impossible.
@schettn schettn changed the title Fix/fields rendering fix: field content cannot be discarded or updated correctly Jul 5, 2021
@schettn schettn merged commit 90270fd into next Jul 5, 2021
schettn added a commit that referenced this pull request Jul 17, 2021
* fix(editablefield): add resetTrigger to reset the Editor on discard

Because of EditableField content do not update when the editing content changes the useEffect of the
Editor is not triggered on discard. By adding a resetTrigger this issue has been fixed.

* fix(editor): cursor no longer jumps to left side on editing

When editing a StreamField the cursor now no longer jumps to the left side which made editing
impossible.
@schettn schettn deleted the fix/fields-rendering branch July 17, 2021 09:17
kleberbaum pushed a commit to getcronit/jaen-v3 that referenced this pull request Aug 8, 2024
feat: add lens compatibility with jaen and ZITADEL
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.

1 participant