Skip to content

Conversation

@Koc
Copy link
Contributor

@Koc Koc commented Dec 28, 2025

We have a large table with many multi line text fields. The problem is that currently RichEditor component created too early (even before we start editing field). And this early creation adds an tons of event listeners to handle click outside. And sometimes it sends too many updates to BE (even if field wasn't changed).

My PR delays creation of the RichEditor component: now we creating it only for inline editing, not earlier.

Related to #2212 and #2174

@Koc Koc requested review from blizzz and enjeck as code owners December 28, 2025 13:57
@Koc Koc added the performance Performance issues and optimisations label Jan 4, 2026
@enjeck enjeck force-pushed the feature/speedup-inline-edit-for-rich-editor branch from 6fdb05c to 0bd34f3 Compare January 9, 2026 11:59
@Koc Koc force-pushed the feature/speedup-inline-edit-for-rich-editor branch from 0bd34f3 to e45828c Compare January 10, 2026 20:42
@Koc Koc enabled auto-merge (squash) January 10, 2026 20:47
@enjeck enjeck force-pushed the feature/speedup-inline-edit-for-rich-editor branch from e45828c to 7e247e9 Compare January 12, 2026 05:47
@Koc Koc merged commit 34b91c0 into main Jan 12, 2026
56 of 58 checks passed
@Koc Koc deleted the feature/speedup-inline-edit-for-rich-editor branch January 12, 2026 07:08
@enjeck
Copy link
Contributor

enjeck commented Jan 12, 2026

/backport to stable1.0

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Jan 12, 2026
@backportbot
Copy link

backportbot bot commented Jan 12, 2026

The backport to stable1.0 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable1.0
git pull origin stable1.0

# Create the new backport branch
git checkout -b backport/2240/stable1.0

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 7e247e91

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/2240/stable1.0

Error: Failed to cherry pick commits: error: no cherry-pick or revert in progress
fatal: cherry-pick failed


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-request Pending backport by the backport-bot performance Performance issues and optimisations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants