Skip to content

Keep RichText webview always running and load notes without reloading the page - #3427

Open
daniele-verducci wants to merge 2 commits into
mainfrom
improv/webview-always-running
Open

daniele-verducci wants to merge 2 commits into
mainfrom
improv/webview-always-running

Conversation

@daniele-verducci

@daniele-verducci daniele-verducci commented Nov 4, 2026 •

Copy link
Copy Markdown
Collaborator

The Webview is no more tied to a NoteDirectEditFragment instance, but it's shared and lazy-loaded.
First time a note is opened, WebView is instanced and loads the direct editing url (about 5 secs on emulator)
Following times, the WebView is only shown and the direct editing url loaded via OCA.Text.openDirectEditing() (about 1 second in the emulator)

⚠️ Requires an instance with the text app version nextcloud/text#9404 to work.
The text app must be built via its make command (requires a container with npm)

Screen_recording_20261104_154039.webm

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

…attach to the running/lazy-loaded wv and load the page

Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 31 complexity · 0 duplication

Metric Results
Complexity 31
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant