update

.coderabbit.yaml

@@ -12,99 +12,84 @@ reviews:
   auto_apply_labels: true
   labeling_instructions:
     - label: "accessibility"
-      instructions: "Apply when the PR contains changes related to accessibility, screen readers, keyboard navigation, or ARIA attributes"
+      instructions: "Apply when the PR contains changes related to accessibility, screen readers, keyboard navigation, or ARIA attributes."
     - label: "android"
-      instructions: "Apply when the PR modifies files under packages/app-mobile/android/. Or when the PR modifies files under packages/app-mobile and the change is specific to Android only"
+      instructions: "Apply when the PR contains changes specific to the Android platform or Android app."
     - label: "api"
-      instructions: "Apply when the PR modifies files under packages/lib/services/rest/"
+      instructions: "Apply when the PR modifies the Joplin API, REST endpoints, or API-related code."
     - label: "bug"
-      instructions: "Apply when the PR fixes a bug or unexpected behaviour"
+      instructions: "Apply when the PR fixes a bug or unexpected behaviour."
     - label: "ci"
-      instructions: "Apply when the PR modifies files under .github/workflows/ or .circleci/"
+      instructions: "Apply when the PR modifies CI/CD configuration, GitHub Actions workflows, or build pipelines."
     - label: "cli"
-      instructions: "Apply when the PR modifies files under packages/app-cli/, except if all the modified files are under packages/app-cli/tests/"
+      instructions: "Apply when the PR contains changes specific to the Joplin CLI (command-line) application."
     - label: "clipper"
-      instructions: "Apply when the PR modifies files under packages/app-clipper/"
+      instructions: "Apply when the PR contains changes to the Joplin Web Clipper browser extension."
     - label: "database"
-      instructions: "Apply when the PR is mainly about modifying database schema, migrations, or database-related logic"
+      instructions: "Apply when the PR modifies database schema, migrations, or database-related logic."
     - label: "desktop"
-      instructions: "Apply when the PR modifies files under packages/app-desktop/"
+      instructions: "Apply when the PR contains changes specific to the Joplin desktop (Electron) application."
     - label: "documentation"
-      instructions: "Apply when the PR modifies files under readme/"
+      instructions: "Apply when the PR adds or updates documentation, README files, or code comments."
     - label: "draw"
-      instructions: "Apply when the PR modifies files under packages/default-plugins and relates to the JS-Draw drawing plugin"
+      instructions: "Apply when the PR contains changes related to the drawing or sketching feature."
     - label: "editor"
-      instructions: "Apply when the PR modifies files under packages/editor/ or packages/app-mobile/components/NoteEditor/"
+      instructions: "Apply when the PR contains changes to the note editor (CodeMirror, TinyMCE, or the editor infrastructure)."
     - label: "enhancement"
-      instructions: "Apply when the PR adds a new feature or improves existing functionality (not a bug fix)"
+      instructions: "Apply when the PR adds a new feature or improves existing functionality (not a bug fix)."
     - label: "export"
-      instructions: "Apply when the PR is mainly about changes to the export functionality (PDF, HTML, JEX, etc.)"
+      instructions: "Apply when the PR contains changes to export functionality (PDF, HTML, JEX, etc.)."
     - label: "import"
-      instructions: "Apply when the PR is mainly about changes to the import functionality (Evernote, Markdown, etc.)"
+      instructions: "Apply when the PR contains changes to import functionality (Evernote, Markdown, etc.)."
     - label: "iOS"
-      instructions: "Apply when the PR modifies files under packages/app-mobile/ios/. Or when the PR modifies files under packages/app-mobile and the change is specific to iOS only"
+      instructions: "Apply when the PR contains changes specific to the iOS platform or iOS app."
     - label: "linux"
-      instructions: "Apply when the PR is mainly about changes specific to Linux"
+      instructions: "Apply when the PR contains changes specific to Linux."
     - label: "linux/wayland"
-      instructions: "Apply when the PR is mainly about changes specific to Linux Wayland"
+      instructions: "Apply when the PR contains changes specific to Linux Wayland."
     - label: "macOS"
-      instructions: "Apply when the PR is mainly about changes specific to macOS"
+      instructions: "Apply when the PR contains changes specific to macOS."
     - label: "markdown-editor"
-      instructions: "Apply when the PR modifies files under packages/editor/CodeMirror"
+      instructions: "Apply when the PR contains changes to the Markdown editor or Markdown rendering."
     - label: "mobile"
-      instructions: "Apply when the PR modifies files under packages/app-mobile/"
+      instructions: "Apply when the PR contains changes to the mobile app (iOS or Android)."
+    - label: "multi-window"
+      instructions: "Apply when the PR contains changes related to multi-window support."
     - label: "OCR"
-      instructions: "Apply when the PR contains changes related to OCR (optical character recognition) functionality"
+      instructions: "Apply when the PR contains changes related to OCR (optical character recognition) functionality."
     - label: "performance"
-      instructions: "Apply when the PR improves performance, reduces memory usage, or optimises speed"
+      instructions: "Apply when the PR improves performance, reduces memory usage, or optimises speed."
     - label: "plugins"
-      instructions: "Apply when the PR modifies files under packages/lib/services/plugins/ or packages/plugin-repo-cli/"
+      instructions: "Apply when the PR contains changes to the plugin system, plugin API, or specific plugins."
     - label: "Regression"
-      instructions: "Apply when the linked issue, if any, has the Regression label"
+      instructions: "Apply when the linked issue, if any, has the Regression label."
     - label: "renderer"
-      instructions: "Apply when the PR modifies files under packages/renderer/ or packages/turndown/"
+      instructions: "Apply when the PR contains changes to the note renderer or how notes are displayed."
     - label: "search"
-      instructions: "Apply when the PR is mainly about changes to the search functionality"
+      instructions: "Apply when the PR contains changes to search functionality."
     - label: "security"
-      instructions: "Apply when the PR is mainly about addressing a security vulnerability or improving security"
+      instructions: "Apply when the PR addresses a security vulnerability or improves security."
     - label: "server"
-      instructions: "Apply when the PR modifies files under packages/server/"
+      instructions: "Apply when the PR contains changes to Joplin Server."
     - label: "Sharing"
-      instructions: "Apply when the PR is mainly about changes to the note or notebook/folder sharing features"
+      instructions: "Apply when the PR contains changes to note or notebook sharing features."
     - label: "sync"
-      instructions: "Apply when the PR modifies files under packages/lib/services/synchronizer/, packages/lib/Sync*.ts or packages/lib/services/e2ee/"
+      instructions: "Apply when the PR contains changes to synchronisation logic or sync targets."
     - label: "tags"
-      instructions: "Apply when the PR is mainly about changes to the tag management or tagging functionality"
+      instructions: "Apply when the PR contains changes to tag management or tagging functionality."
     - label: "transcribe"
-      instructions: "Apply when the PR modifies files under packages/transcribe"
+      instructions: "Apply when the PR contains changes to audio transcription functionality."
     - label: "translation"
-      instructions: "Apply when the PR modifies files under packages/tools/locales/ or **/locales/"
+      instructions: "Apply when the PR adds or updates translations or localisation strings."
     - label: "Voice typing"
-      instructions: "Apply when the PR is mainly about changes to the voice typing functionality"
+      instructions: "Apply when the PR contains changes to voice typing functionality."
     - label: "web"
-      instructions: "Apply when the PR modifies files under packages/app-web/. Or when the PR modifies files under packages/app-mobile and the change is specific to the web app only"
+      instructions: "Apply when the PR contains changes to the Joplin web application or web-related features."
     - label: "windows"
-      instructions: "Apply when the PR is mainly about changes specific to Windows"
-
-  pre_merge_checks:
-    description:
-      mode: "warning"
-    custom_checks:
-      - name: "PR Description Must Follow Guidelines"
-        mode: "error"
-        instructions: |
-          Fail if the pull request description does not include clear sections for:
-          - Problem or user-impact description
-          - A high-level Solution explanation
-          - Any Test Plan or verification steps
-
-          The description should align with our PR guidelines
-          at https://github.com/joplin/gsoc/blob/master/pull_request_guidelines.md
-          and should not just restate the diff or implementation details.
+      instructions: "Apply when the PR contains changes specific to Windows."
 knowledge_base:
   code_guidelines:
     enabled: true
     filePatterns:
       - "readme/dev/coding_style.md"
       - "readme/dev/index.md"
-      - "CLAUDE.md"

.dockerignore

@@ -17,4 +17,3 @@ packages/server/db-*.sqlite
 packages/server/dist/
 packages/server/logs/
 packages/server/temp/
-packages/transcribe/.env

.env-sample

@@ -31,7 +31,6 @@
 # QUEUE_DATABASE_PASSWORD=transcribe
 # QUEUE_DATABASE_PORT=5431
 # HTR_CLI_IMAGES_FOLDER=/home/user/images_storage
-# HTR_CLI_MODELS_FOLDER=/home/user/transcribe_models
 
 # =============================================================================
 # DEV CONFIG EXAMPLE

.env-transcribe-sample

@@ -1,33 +1,35 @@
-# Joplin Transcribe Configuration
-#
-# Copy this file to .env-transcribe and update the values.
-
 # =============================================================================
 # Required
+# -----------------------------------------------------------------------------
 # =============================================================================
 
-# Set a secure API key for authentication
-API_KEY=changeme
+SERVER_PORT=4567
 
-# =============================================================================
-# Optional (defaults are set in the Docker image)
-# =============================================================================
+API_KEY=random-string
+QUEUE_TTL=900000
+QUEUE_RETRY_COUNT=2
+QUEUE_MAINTENANCE_INTERVAL=30000
+IMAGE_MAX_DIMENSION=400
 
-# Server port (default: 4567)
-# SERVER_PORT=4567
+HTR_CLI_DOCKER_IMAGE=joplin/htr-cli:latest
+# Fullpath to images folder e.g.:
+#HTR_CLI_IMAGES_FOLDER=/home/user/joplin/packages/transcribe/images
+HTR_CLI_IMAGES_FOLDER=
 
-# Maximum image dimension for processing (default: 400)
-# IMAGE_MAX_DIMENSION=400
-
-# Queue driver: sqlite (default) or pg
+QUEUE_DRIVER=pg
 # QUEUE_DRIVER=sqlite
 
-# =============================================================================
-# PostgreSQL settings (only if QUEUE_DRIVER=pg)
-# =============================================================================
+FILE_STORAGE_MAINTENANCE_INTERVAL=3600000
+FILE_STORAGE_TTL=604800000 # one week
 
-# QUEUE_DATABASE_NAME=transcribe
-# QUEUE_DATABASE_USER=transcribe
-# QUEUE_DATABASE_PASSWORD=transcribe
-# QUEUE_DATABASE_PORT=5432
-# QUEUE_DATABASE_HOST=localhost
+# =============================================================================
+# Queue driver
+# -----------------------------------------------------------------------------
+# =============================================================================
+#
+# QUEUE_DATABASE_NAME=./queue.sqlite3
+QUEUE_DATABASE_NAME=transcribe
+QUEUE_DATABASE_USER=transcribe
+QUEUE_DATABASE_PASSWORD=transcribe
+QUEUE_DATABASE_PORT=5432
+QUEUE_DATABASE_HOST=localhost

.eslintignore

@@ -169,7 +169,6 @@ packages/app-desktop/bridge.js
 packages/app-desktop/checkForUpdates.js
 packages/app-desktop/commands/copyDevCommand.js
 packages/app-desktop/commands/copyToClipboard.js
-packages/app-desktop/commands/createAccessibleDocument.js
 packages/app-desktop/commands/editProfileConfig.js
 packages/app-desktop/commands/emptyTrash.js
 packages/app-desktop/commands/exportDeletionLog.test.js
@@ -303,7 +302,6 @@ packages/app-desktop/gui/NoteEditor/commands/focusElementNoteTitle.js
 packages/app-desktop/gui/NoteEditor/commands/focusElementNoteViewer.js
 packages/app-desktop/gui/NoteEditor/commands/focusElementToolbar.js
 packages/app-desktop/gui/NoteEditor/commands/index.js
-packages/app-desktop/gui/NoteEditor/commands/pasteAsMarkdown.js
 packages/app-desktop/gui/NoteEditor/commands/pasteAsText.js
 packages/app-desktop/gui/NoteEditor/commands/showLocalSearch.js
 packages/app-desktop/gui/NoteEditor/commands/showRevisions.js
@@ -341,11 +339,9 @@ packages/app-desktop/gui/NoteEditor/utils/useWindowCommandHandler.js
 packages/app-desktop/gui/NoteList/NoteList2.js
 packages/app-desktop/gui/NoteList/commands/focusElementNoteList.js
 packages/app-desktop/gui/NoteList/commands/index.js
-packages/app-desktop/gui/NoteList/utils/UseAutoScroll.test.js
 packages/app-desktop/gui/NoteList/utils/canManuallySortNotes.js
 packages/app-desktop/gui/NoteList/utils/types.js
 packages/app-desktop/gui/NoteList/utils/useActiveDescendantId.js
-packages/app-desktop/gui/NoteList/utils/useAutoScroll.js
 packages/app-desktop/gui/NoteList/utils/useDragAndDrop.js
 packages/app-desktop/gui/NoteList/utils/useFocusNote.js
 packages/app-desktop/gui/NoteList/utils/useFocusVisible.js
@@ -611,6 +607,10 @@ packages/app-desktop/services/plugins/hooks/useViewIsReady.js
 packages/app-desktop/services/plugins/hooks/useWebviewToPluginMessages.js
 packages/app-desktop/services/plugins/types.js
 packages/app-desktop/services/restart.js
+packages/app-desktop/services/sortOrder/PerFolderSortOrderService.test.js
+packages/app-desktop/services/sortOrder/PerFolderSortOrderService.js
+packages/app-desktop/services/sortOrder/notesSortOrderUtils.test.js
+packages/app-desktop/services/sortOrder/notesSortOrderUtils.js
 packages/app-desktop/services/spellChecker/SpellCheckerServiceDriverNative.js
 packages/app-desktop/tools/bundleJs.js
 packages/app-desktop/tools/copy7Zip.js
@@ -694,6 +694,8 @@ packages/app-mobile/components/ExtendedWebView/index.js
 packages/app-mobile/components/ExtendedWebView/index.web.js
 packages/app-mobile/components/ExtendedWebView/types.js
 packages/app-mobile/components/ExtendedWebView/utils/useCss.js
+packages/app-mobile/components/FeedbackBanner.test.js
+packages/app-mobile/components/FeedbackBanner.js
 packages/app-mobile/components/FolderPicker.js
 packages/app-mobile/components/Icon.js
 packages/app-mobile/components/IconButton.js
@@ -738,6 +740,7 @@ packages/app-mobile/components/ScreenHeader/Menu.js
 packages/app-mobile/components/ScreenHeader/WarningBanner.test.js
 packages/app-mobile/components/ScreenHeader/WarningBanner.js
 packages/app-mobile/components/ScreenHeader/WarningBox.js
+packages/app-mobile/components/ScreenHeader/WebBetaButton.js
 packages/app-mobile/components/ScreenHeader/index.js
 packages/app-mobile/components/SearchInput.js
 packages/app-mobile/components/SelectDateTimeDialog.js
@@ -901,7 +904,6 @@ packages/app-mobile/contentScripts/markdownEditorBundle/useWebViewSetup.js
 packages/app-mobile/contentScripts/markdownEditorBundle/utils/useCodeMirrorPlugins.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/Renderer.test.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/Renderer.js
-packages/app-mobile/contentScripts/rendererBundle/contentScript/index.handleAnchorClick.test.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/index.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/types.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/utils/addPluginAssets.js
@@ -1263,7 +1265,6 @@ packages/lib/SyncTargetRegistry.js
 packages/lib/Synchronizer.js
 packages/lib/TaskQueue.js
 packages/lib/WebDavApi.js
-packages/lib/WelcomeUtils.test.js
 packages/lib/WelcomeUtils.js
 packages/lib/array.js
 packages/lib/callbackUrlUtils.test.js
@@ -1292,7 +1293,6 @@ packages/lib/components/EncryptionConfigScreen/utils.test.js
 packages/lib/components/EncryptionConfigScreen/utils.js
 packages/lib/components/shared/NoteEditor/WarningBanner/onRichTextDismissLinkClick.js
 packages/lib/components/shared/NoteEditor/WarningBanner/onRichTextReadMoreLinkClick.js
-packages/lib/components/shared/NoteEditor/WarningBanner/useEditorTypeMigrationBanner.js
 packages/lib/components/shared/NoteList/getEmptyFolderMessage.js
 packages/lib/components/shared/NoteRevisionViewer/getHelpMessage.js
 packages/lib/components/shared/NoteRevisionViewer/useDeleteHistoryClick.js
@@ -1431,7 +1431,6 @@ packages/lib/services/AlarmServiceDriverNode.js
 packages/lib/services/BaseService.js
 packages/lib/services/CommandService.test.js
 packages/lib/services/CommandService.js
-packages/lib/services/DecryptionWorker.test.js
 packages/lib/services/DecryptionWorker.js
 packages/lib/services/ExternalEditWatcher.js
 packages/lib/services/ExternalEditWatcher/utils.js
@@ -1520,7 +1519,6 @@ packages/lib/services/interop/InteropService_Importer_Md.test.js
 packages/lib/services/interop/InteropService_Importer_Md.js
 packages/lib/services/interop/InteropService_Importer_Md_frontmatter.test.js
 packages/lib/services/interop/InteropService_Importer_Md_frontmatter.js
-packages/lib/services/interop/InteropService_Importer_OneNote.postprocessHtml.test.js
 packages/lib/services/interop/InteropService_Importer_OneNote.test.js
 packages/lib/services/interop/InteropService_Importer_OneNote.js
 packages/lib/services/interop/InteropService_Importer_Raw.test.js
@@ -1554,7 +1552,6 @@ packages/lib/services/ocr/OcrService.js
 packages/lib/services/ocr/drivers/OcrDriverTesseract.js
 packages/lib/services/ocr/drivers/OcrDriverTranscribe.test.js
 packages/lib/services/ocr/drivers/OcrDriverTranscribe.js
-packages/lib/services/ocr/utils/createAccessiblePdf.js
 packages/lib/services/ocr/utils/filterOcrText.test.js
 packages/lib/services/ocr/utils/filterOcrText.js
 packages/lib/services/ocr/utils/types.js
@@ -1576,7 +1573,6 @@ packages/lib/services/plugins/api/JoplinCommands.js
 packages/lib/services/plugins/api/JoplinContentScripts.js
 packages/lib/services/plugins/api/JoplinData.js
 packages/lib/services/plugins/api/JoplinFilters.js
-packages/lib/services/plugins/api/JoplinFs.js
 packages/lib/services/plugins/api/JoplinImaging.js
 packages/lib/services/plugins/api/JoplinInterop.js
 packages/lib/services/plugins/api/JoplinPlugins.js
@@ -1675,10 +1671,6 @@ packages/lib/services/share/ShareService.test.js
 packages/lib/services/share/ShareService.js
 packages/lib/services/share/invitationRespond.js
 packages/lib/services/share/reducer.js
-packages/lib/services/sortOrder/PerFolderSortOrderService.test.js
-packages/lib/services/sortOrder/PerFolderSortOrderService.js
-packages/lib/services/sortOrder/notesSortOrderUtils.test.js
-packages/lib/services/sortOrder/notesSortOrderUtils.js
 packages/lib/services/spellChecker/SpellCheckerService.js
 packages/lib/services/spellChecker/SpellCheckerServiceDriverBase.js
 packages/lib/services/style/cssToTheme.test.js
@@ -1897,11 +1889,11 @@ packages/tools/fuzzer/utils/SeededRandom.js
 packages/tools/fuzzer/utils/diffSortedStringArrays.test.js
 packages/tools/fuzzer/utils/diffSortedStringArrays.js
 packages/tools/fuzzer/utils/extractResourceIds.js
-packages/tools/fuzzer/utils/getBinaryDiffDebugMessage.js
 packages/tools/fuzzer/utils/getNumberProperty.js
 packages/tools/fuzzer/utils/getProperty.js
 packages/tools/fuzzer/utils/getStringProperty.js
 packages/tools/fuzzer/utils/hangingIndent.js
+packages/tools/fuzzer/utils/logDiffDebug.js
 packages/tools/fuzzer/utils/openDebugSession.js
 packages/tools/fuzzer/utils/randomId.test.js
 packages/tools/fuzzer/utils/randomId.js
@@ -1948,7 +1940,6 @@ packages/tools/update-readme-contributors.js
 packages/tools/update-readme-download.test.js
 packages/tools/update-readme-download.js
 packages/tools/update-readme-sponsors.js
-packages/tools/updateCanary.js
 packages/tools/updateMarkdownDoc.js
 packages/tools/utils/discourse.test.js
 packages/tools/utils/discourse.js

.eslintrc.js

@@ -214,6 +214,7 @@ module.exports = {
 				'packages/tools/**',
 				'packages/app-mobile/tools/**',
 				'packages/app-desktop/tools/**',
+				'packages/transcribe/src/tools/**',
 			],
 			'rules': {
 				'no-console': 'off',

.github/PULL_REQUEST_TEMPLATE

@@ -1,12 +1,6 @@
 <!--
 
-Before contributing, please read the contribution guidelines: https://github.com/laurent22/joplin/blob/dev/readme/dev/index.md
-
-If this is a Google Summer of Code pull request, please read the [GSoC pull request guidelines](https://github.com/joplin/gsoc/blob/master/pull_request_guidelines.md).
-
----
-
-**Pull request title**: Please prefix the title with the platform you are targetting.
+Please prefix the title with the platform you are targetting:
 
 Here are some examples of good titles:
 
@@ -26,4 +20,6 @@ If it's not related to any platform (such as a translation, change to the docume
 
 Then please append the issue that you've addressed or fixed. Use "Resolves #123" for new features or improvements and "Fixes #123" for bug fixes.
 
+AND PLEASE READ THE GUIDE: https://github.com/laurent22/joplin/blob/dev/readme/dev/index.md
+
 -->

.github/workflows/build-android.yml

@@ -4,10 +4,6 @@
 name: react-native-android-build-apk
 on: [push, pull_request]
 
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: true
-
 jobs:
   AssembleRelease:
     if: github.repository == 'laurent22/joplin'

.github/workflows/build-macos-m1.yml

@@ -1,10 +1,5 @@
 name: Build macOS M1
 on: [push, pull_request]
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: true
-
 jobs:
   Main:
     # We always process desktop release tags, because they also publish the release

.github/workflows/github-actions-main.yml

@@ -1,10 +1,5 @@
 name: Joplin Continuous Integration
 on: [push, pull_request]
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: true
-
 jobs:
   Main:
     # We always process server or desktop release tags, because they also publish the release

.github/workflows/ui-tests.yml

@@ -1,10 +1,5 @@
 name: Joplin UI tests
 on: [push, pull_request]
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: true
-
 permissions:
   contents: read
 jobs:

.gitignore

@@ -142,7 +142,6 @@ packages/app-desktop/bridge.js
 packages/app-desktop/checkForUpdates.js
 packages/app-desktop/commands/copyDevCommand.js
 packages/app-desktop/commands/copyToClipboard.js
-packages/app-desktop/commands/createAccessibleDocument.js
 packages/app-desktop/commands/editProfileConfig.js
 packages/app-desktop/commands/emptyTrash.js
 packages/app-desktop/commands/exportDeletionLog.test.js
@@ -276,7 +275,6 @@ packages/app-desktop/gui/NoteEditor/commands/focusElementNoteTitle.js
 packages/app-desktop/gui/NoteEditor/commands/focusElementNoteViewer.js
 packages/app-desktop/gui/NoteEditor/commands/focusElementToolbar.js
 packages/app-desktop/gui/NoteEditor/commands/index.js
-packages/app-desktop/gui/NoteEditor/commands/pasteAsMarkdown.js
 packages/app-desktop/gui/NoteEditor/commands/pasteAsText.js
 packages/app-desktop/gui/NoteEditor/commands/showLocalSearch.js
 packages/app-desktop/gui/NoteEditor/commands/showRevisions.js
@@ -314,11 +312,9 @@ packages/app-desktop/gui/NoteEditor/utils/useWindowCommandHandler.js
 packages/app-desktop/gui/NoteList/NoteList2.js
 packages/app-desktop/gui/NoteList/commands/focusElementNoteList.js
 packages/app-desktop/gui/NoteList/commands/index.js
-packages/app-desktop/gui/NoteList/utils/UseAutoScroll.test.js
 packages/app-desktop/gui/NoteList/utils/canManuallySortNotes.js
 packages/app-desktop/gui/NoteList/utils/types.js
 packages/app-desktop/gui/NoteList/utils/useActiveDescendantId.js
-packages/app-desktop/gui/NoteList/utils/useAutoScroll.js
 packages/app-desktop/gui/NoteList/utils/useDragAndDrop.js
 packages/app-desktop/gui/NoteList/utils/useFocusNote.js
 packages/app-desktop/gui/NoteList/utils/useFocusVisible.js
@@ -584,6 +580,10 @@ packages/app-desktop/services/plugins/hooks/useViewIsReady.js
 packages/app-desktop/services/plugins/hooks/useWebviewToPluginMessages.js
 packages/app-desktop/services/plugins/types.js
 packages/app-desktop/services/restart.js
+packages/app-desktop/services/sortOrder/PerFolderSortOrderService.test.js
+packages/app-desktop/services/sortOrder/PerFolderSortOrderService.js
+packages/app-desktop/services/sortOrder/notesSortOrderUtils.test.js
+packages/app-desktop/services/sortOrder/notesSortOrderUtils.js
 packages/app-desktop/services/spellChecker/SpellCheckerServiceDriverNative.js
 packages/app-desktop/tools/bundleJs.js
 packages/app-desktop/tools/copy7Zip.js
@@ -667,6 +667,8 @@ packages/app-mobile/components/ExtendedWebView/index.js
 packages/app-mobile/components/ExtendedWebView/index.web.js
 packages/app-mobile/components/ExtendedWebView/types.js
 packages/app-mobile/components/ExtendedWebView/utils/useCss.js
+packages/app-mobile/components/FeedbackBanner.test.js
+packages/app-mobile/components/FeedbackBanner.js
 packages/app-mobile/components/FolderPicker.js
 packages/app-mobile/components/Icon.js
 packages/app-mobile/components/IconButton.js
@@ -711,6 +713,7 @@ packages/app-mobile/components/ScreenHeader/Menu.js
 packages/app-mobile/components/ScreenHeader/WarningBanner.test.js
 packages/app-mobile/components/ScreenHeader/WarningBanner.js
 packages/app-mobile/components/ScreenHeader/WarningBox.js
+packages/app-mobile/components/ScreenHeader/WebBetaButton.js
 packages/app-mobile/components/ScreenHeader/index.js
 packages/app-mobile/components/SearchInput.js
 packages/app-mobile/components/SelectDateTimeDialog.js
@@ -874,7 +877,6 @@ packages/app-mobile/contentScripts/markdownEditorBundle/useWebViewSetup.js
 packages/app-mobile/contentScripts/markdownEditorBundle/utils/useCodeMirrorPlugins.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/Renderer.test.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/Renderer.js
-packages/app-mobile/contentScripts/rendererBundle/contentScript/index.handleAnchorClick.test.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/index.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/types.js
 packages/app-mobile/contentScripts/rendererBundle/contentScript/utils/addPluginAssets.js
@@ -1236,7 +1238,6 @@ packages/lib/SyncTargetRegistry.js
 packages/lib/Synchronizer.js
 packages/lib/TaskQueue.js
 packages/lib/WebDavApi.js
-packages/lib/WelcomeUtils.test.js
 packages/lib/WelcomeUtils.js
 packages/lib/array.js
 packages/lib/callbackUrlUtils.test.js
@@ -1265,7 +1266,6 @@ packages/lib/components/EncryptionConfigScreen/utils.test.js
 packages/lib/components/EncryptionConfigScreen/utils.js
 packages/lib/components/shared/NoteEditor/WarningBanner/onRichTextDismissLinkClick.js
 packages/lib/components/shared/NoteEditor/WarningBanner/onRichTextReadMoreLinkClick.js
-packages/lib/components/shared/NoteEditor/WarningBanner/useEditorTypeMigrationBanner.js
 packages/lib/components/shared/NoteList/getEmptyFolderMessage.js
 packages/lib/components/shared/NoteRevisionViewer/getHelpMessage.js
 packages/lib/components/shared/NoteRevisionViewer/useDeleteHistoryClick.js
@@ -1404,7 +1404,6 @@ packages/lib/services/AlarmServiceDriverNode.js
 packages/lib/services/BaseService.js
 packages/lib/services/CommandService.test.js
 packages/lib/services/CommandService.js
-packages/lib/services/DecryptionWorker.test.js
 packages/lib/services/DecryptionWorker.js
 packages/lib/services/ExternalEditWatcher.js
 packages/lib/services/ExternalEditWatcher/utils.js
@@ -1493,7 +1492,6 @@ packages/lib/services/interop/InteropService_Importer_Md.test.js
 packages/lib/services/interop/InteropService_Importer_Md.js
 packages/lib/services/interop/InteropService_Importer_Md_frontmatter.test.js
 packages/lib/services/interop/InteropService_Importer_Md_frontmatter.js
-packages/lib/services/interop/InteropService_Importer_OneNote.postprocessHtml.test.js
 packages/lib/services/interop/InteropService_Importer_OneNote.test.js
 packages/lib/services/interop/InteropService_Importer_OneNote.js
 packages/lib/services/interop/InteropService_Importer_Raw.test.js
@@ -1527,7 +1525,6 @@ packages/lib/services/ocr/OcrService.js
 packages/lib/services/ocr/drivers/OcrDriverTesseract.js
 packages/lib/services/ocr/drivers/OcrDriverTranscribe.test.js
 packages/lib/services/ocr/drivers/OcrDriverTranscribe.js
-packages/lib/services/ocr/utils/createAccessiblePdf.js
 packages/lib/services/ocr/utils/filterOcrText.test.js
 packages/lib/services/ocr/utils/filterOcrText.js
 packages/lib/services/ocr/utils/types.js
@@ -1549,7 +1546,6 @@ packages/lib/services/plugins/api/JoplinCommands.js
 packages/lib/services/plugins/api/JoplinContentScripts.js
 packages/lib/services/plugins/api/JoplinData.js
 packages/lib/services/plugins/api/JoplinFilters.js
-packages/lib/services/plugins/api/JoplinFs.js
 packages/lib/services/plugins/api/JoplinImaging.js
 packages/lib/services/plugins/api/JoplinInterop.js
 packages/lib/services/plugins/api/JoplinPlugins.js
@@ -1648,10 +1644,6 @@ packages/lib/services/share/ShareService.test.js
 packages/lib/services/share/ShareService.js
 packages/lib/services/share/invitationRespond.js
 packages/lib/services/share/reducer.js
-packages/lib/services/sortOrder/PerFolderSortOrderService.test.js
-packages/lib/services/sortOrder/PerFolderSortOrderService.js
-packages/lib/services/sortOrder/notesSortOrderUtils.test.js
-packages/lib/services/sortOrder/notesSortOrderUtils.js
 packages/lib/services/spellChecker/SpellCheckerService.js
 packages/lib/services/spellChecker/SpellCheckerServiceDriverBase.js
 packages/lib/services/style/cssToTheme.test.js
@@ -1870,11 +1862,11 @@ packages/tools/fuzzer/utils/SeededRandom.js
 packages/tools/fuzzer/utils/diffSortedStringArrays.test.js
 packages/tools/fuzzer/utils/diffSortedStringArrays.js
 packages/tools/fuzzer/utils/extractResourceIds.js
-packages/tools/fuzzer/utils/getBinaryDiffDebugMessage.js
 packages/tools/fuzzer/utils/getNumberProperty.js
 packages/tools/fuzzer/utils/getProperty.js
 packages/tools/fuzzer/utils/getStringProperty.js
 packages/tools/fuzzer/utils/hangingIndent.js
+packages/tools/fuzzer/utils/logDiffDebug.js
 packages/tools/fuzzer/utils/openDebugSession.js
 packages/tools/fuzzer/utils/randomId.test.js
 packages/tools/fuzzer/utils/randomId.js
@@ -1921,7 +1913,6 @@ packages/tools/update-readme-contributors.js
 packages/tools/update-readme-download.test.js
 packages/tools/update-readme-download.js
 packages/tools/update-readme-sponsors.js
-packages/tools/updateCanary.js
 packages/tools/updateMarkdownDoc.js
 packages/tools/utils/discourse.test.js
 packages/tools/utils/discourse.js

.yarn/patches/formidable-npm-2.1.2-40ba18d67f.patch

@@ -1,24 +0,0 @@
-# Resolves an issue in which notes and attachments larger than 16 KB
-# could become corrupted during the upload process.
-# See https://github.com/laurent22/joplin/issues/14343
-diff --git a/src/parsers/JSON.js b/src/parsers/JSON.js
-index 9a096c25778c7c68be1ddd9dd78faa85bd1d8ec3..6d6bfd2d3789313a7adc8966ab8e58c3d3167356 100644
---- a/src/parsers/JSON.js
-+++ b/src/parsers/JSON.js
-@@ -12,13 +12,14 @@ class JSONParser extends Transform {
-   }
- 
-   _transform(chunk, encoding, callback) {
--    this.chunks.push(String(chunk)); // todo consider using a string decoder
-+    this.chunks.push(chunk); // type: Uint8Array
-     callback();
-   }
- 
-   _flush(callback) {
-     try {
--      const fields = JSON.parse(this.chunks.join(''));
-+      const data = Buffer.concat(this.chunks);
-+      const fields = JSON.parse(data.toString('utf-8'));
-       Object.keys(fields).forEach((key) => {
-         const value = fields[key];
-         this.push({ key, value });

Assets/WebsiteAssets/rss.xml

@@ -1,24 +1,4 @@
-<?xml version="1.0" encoding="UTF-8"?><rss xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:atom="http://www.w3.org/2005/Atom" version="2.0"><channel><title><![CDATA[Joplin]]></title><description><![CDATA[Joplin, the open source note-taking application]]></description><link>https://joplinapp.org</link><generator>RSS for Node</generator><lastBuildDate>Mon, 23 Feb 2026 00:00:00 GMT</lastBuildDate><atom:link href="https://joplinapp.org/rss.xml" rel="self" type="application/rss+xml"/><pubDate>Mon, 23 Feb 2026 00:00:00 GMT</pubDate><item><title><![CDATA[Introducing our Warrant Canary]]></title><description><![CDATA[<p>We have introduced a publicly signed warrant canary for Joplin.</p>
-<p>A warrant canary is a regularly updated statement confirming that, as of the stated date, the project has not received secret legal orders, gag orders, or demands requiring the introduction of backdoors into the software or its infrastructure.</p>
-<p>The canary is:</p>
-<ul>
-<li>
-<p>Cryptographically signed using a dedicated OpenPGP key</p>
-</li>
-<li>
-<p>Updated every 60 days</p>
-</li>
-<li>
-<p>Published in plain text for independent verification</p>
-</li>
-</ul>
-<p>If the canary is not updated within its stated validity window, it should be considered expired.</p>
-<p>You can view and verify the current canary here:</p>
-<p><a href="https://raw.githubusercontent.com/laurent22/joplin/refs/heads/dev/readme/canary.txt">https://raw.githubusercontent.com/laurent22/joplin/refs/heads/dev/readme/canary.txt</a></p>
-<p>With additional information on how it is generated and managed there:</p>
-<p><a href="https://github.com/laurent22/joplin/blob/dev/readme/canary.md">https://github.com/laurent22/joplin/blob/dev/readme/canary.md</a></p>
-<p>This measure is intended to improve transparency and provide an additional signal to the community. It does not prevent legal orders, but it helps ensure that any material change in our legal status cannot occur silently.</p>
-]]></description><link>https://joplinapp.org/news/20260223-warrant-canary</link><guid isPermaLink="false">20260223-warrant-canary</guid><pubDate>Mon, 23 Feb 2026 00:00:00 GMT</pubDate><twitter-text></twitter-text></item><item><title><![CDATA[Joplin will come preloaded on the HMD Terra M]]></title><description><![CDATA[<div style="overflow: auto;">
+<?xml version="1.0" encoding="UTF-8"?><rss xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:atom="http://www.w3.org/2005/Atom" version="2.0"><channel><title><![CDATA[Joplin]]></title><description><![CDATA[Joplin, the open source note-taking application]]></description><link>https://joplinapp.org</link><generator>RSS for Node</generator><lastBuildDate>Tue, 10 Feb 2026 00:00:00 GMT</lastBuildDate><atom:link href="https://joplinapp.org/rss.xml" rel="self" type="application/rss+xml"/><pubDate>Tue, 10 Feb 2026 00:00:00 GMT</pubDate><item><title><![CDATA[Joplin will come preloaded on the HMD Terra M]]></title><description><![CDATA[<div style="overflow: auto;">
 <img src="https://raw.githubusercontent.com/laurent22/joplin/dev/Assets/WebsiteAssets/images/news/20260210-hmd-joplin-logo.png" width="200px" style="float: left; margin-right: 16px; margin-bottom: 16px;"/>
 <p>We’re happy to announce a collaboration with <a href="https://www.hmdsecure.com/">HMD Secure</a>, who will preload Joplin on their upcoming device, the HMD Terra M.</p>
 <p>This partnership brings Joplin to a new class of rugged, professional devices built for instant reliable communication, and reflects a shared focus on reliability, security, and long-term use.</p>
@@ -528,4 +508,15 @@ sys 0m38.013s</p>
 ]]></description><link>https://joplinapp.org/news/20230508-release-2-10</link><guid isPermaLink="false">20230508-release-2-10</guid><pubDate>Wed, 10 May 2023 12:00:00 GMT</pubDate><twitter-text>What&apos;s new in Joplin 2.10</twitter-text></item><item><title><![CDATA[Joplin will participate in JdLL 2023!]]></title><description><![CDATA[<p>On 1 and 2 April 2023, we will have a stand for Joplin at the <a href="https://www.jdll.org/">Journées du Logiciel Libre</a> in Lyon, France. The JdLL has been taking place in Lyon for 24 years and is a popular open source conference in France. We had a stand in 2020 and 2021 but that was cancelled due to Covid, so this year is a first for Joplin!</p>
 <p>Admission is free, so don't hesitate to come and meet us, exchange ideas and learn more about Joplin!</p>
 <p><img src="https://raw.githubusercontent.com/laurent22/joplin/dev/Assets/WebsiteAssets/images/news/20230202-jdll.jpg" alt="Joplin at JdLL 2023"></p>
-]]></description><link>https://joplinapp.org/news/20230302-jdll-2023</link><guid isPermaLink="false">20230302-jdll-2023</guid><pubDate>Thu, 02 Mar 2023 00:00:00 GMT</pubDate><twitter-text></twitter-text></item></channel></rss>
+]]></description><link>https://joplinapp.org/news/20230302-jdll-2023</link><guid isPermaLink="false">20230302-jdll-2023</guid><pubDate>Thu, 02 Mar 2023 00:00:00 GMT</pubDate><twitter-text></twitter-text></item><item><title><![CDATA[Introducing the "GitHub Actions Raw Log Viewer" extension for Chrome]]></title><description><![CDATA[<p>If you've ever used GitHub Actions, you will find that they provide by default a nice coloured output for the log. It looks good and it's even interactive! (You can click to collapse/expand blocks of text) But unfortunately it doesn't scale to large workflows, like we have for Joplin - the log can freeze and it will take forever to search for something. Indeed searching is done in &quot;real time&quot;... which mostly means it will freeze for a minute or two for each letter you type in the search box. Not great.</p>
+<p>Thankfully GitHub provides an alternative access: the raw logs. This is much better because they will open as plain text, without any styling or JS magic, which means you can use the browser native search and it will be fast.</p>
+<p>But now the problem is that raw logs look like this:</p>
+<p><img src="https://raw.githubusercontent.com/laurent22/joplin/dev/Assets/WebsiteAssets/images/news/20230116-ga-raw-log.png" alt="Raw log without extension"></p>
+<p>While it's not impossible to read, all colours that would display nicely in a terminal are gone and replaced by <a href="https://en.wikipedia.org/wiki/ANSI_escape_code">ANSI codes</a>. You can find what you need in there but it's not particularly easy.</p>
+<p>This is where the new <strong>GitHub Action Raw Log Viewer</strong> extension for Chrome can help. It will parse your raw log and convert the ANSI codes to proper colours. This results in a much more readable rendering:</p>
+<p><img src="https://raw.githubusercontent.com/laurent22/joplin/dev/Assets/WebsiteAssets/images/news/20230116-ga-raw-log-colored.png" alt="Raw log with extension"></p>
+<p>The extension is fast even for very large logs and it's of course easy to search for text since it simply works with your browser built-in search.</p>
+<p>The extension is open source, with the code available here: <a href="https://github.com/laurent22/github-actions-logs-extension">https://github.com/laurent22/github-actions-logs-extension</a></p>
+<p>And to install it, follow this link:</p>
+<p><a href="https://chrome.google.com/webstore/detail/github-action-raw-log-vie/lgejlnoopmcdglhfjblaeldbcfnmjddf"><img src="https://raw.githubusercontent.com/laurent22/joplin/dev/Assets/WebsiteAssets/images/news/20230116-extension-get-it-now.png" alt="Download GitHub Action Raw Log Viewer extension"></a></p>
+]]></description><link>https://joplinapp.org/news/20230116-github-actions-log-viewer</link><guid isPermaLink="false">20230116-github-actions-log-viewer</guid><pubDate>Mon, 16 Jan 2023 00:00:00 GMT</pubDate><twitter-text>Introducing the &quot;GitHub Action Raw Log Viewer&quot; extension for Chrome</twitter-text></item></channel></rss>

Assets/WebsiteAssets/templates/partials/socialFeeds.mustache

@@ -3,9 +3,9 @@
 		<a class="social-link-bluesky" href="https://bsky.app/profile/joplinapp.bsky.social" title="Joplin Bluesky feed"><i class="fa-brands fa-bluesky"></i></a>
 		<a class="social-link-mastodon" href="https://mastodon.social/@joplinapp" title="Joplin Mastodon feed"><i class="fab fa-mastodon"></i></a>
 		<a class="social-link-patreon" href="https://www.patreon.com/joplin" title="Joplin Patreon"><i class="fab fa-patreon"></i></a>
-		<a class="social-link-youtube" href="https://youtube.com/@joplinapp" title="Joplin YouTube channel"><i class="fab fa-youtube"></i></a>
 		<a class="social-link-discord" href="https://discord.gg/VSj7AFHvpq" title="Joplin Discord chat"><i class="fab fa-discord"></i></a>
 		<a class="social-link-linkedin" href="https://www.linkedin.com/company/joplin" title="Joplin LinkedIn Feed"><i class="fab fa-linkedin"></i></a>
+		<a class="social-link-lemmy" href="https://sopuli.xyz/c/joplinapp" title="Joplin Lemmy Community"><i class="fas fa-otter"></i></a>
 		<a class="social-link-github" href="https://github.com/laurent22/joplin/" title="Joplin GitHub repository"><i class="fab fa-github"></i></a>
 	</div>
 </div>

Assets/keys/joplin-canary-signing-key.asc

@@ -1,14 +0,0 @@
------BEGIN PGP PUBLIC KEY BLOCK-----
-
-mDMEaZWFlBYJKwYBBAHaRw8BAQdAIh3xQbjaS0EC+8WuKXNPjVF/ayq0/2GZlheR
-qj1G3Qe0RUpvcGxpbiBDYW5hcnkgU2lnbmluZyBLZXkgKFdhcnJhbnQgQ2FuYXJ5
-IEtleSkgPGNhbmFyeUBqb3BsaW5hcHAub3JnPoiZBBMWCgBBFiEE+CD4MG3QBaEC
-0YzVlGrp+lkV71MFAmmVhZQCGwMFCQPCZwAFCwkIBwICIgIGFQoJCAsCBBYCAwEC
-HgcCF4AACgkQlGrp+lkV71MZtwD/Ufd4OAcgkl5T6MSB+WDFg8BXvpaBZfNnZkoo
-LrOoqNAA/iqGiiBRoarlus2ATOiWhyXaEpRUQcEeaRhhqHW0BGcCuDgEaZWFlBIK
-KwYBBAGXVQEFAQEHQFORKWRLp4hDYzR8Q5IRyF9AIjoziR+sj4icUdvZx4Z6AwEI
-B4h+BBgWCgAmFiEE+CD4MG3QBaEC0YzVlGrp+lkV71MFAmmVhZQCGwwFCQPCZwAA
-CgkQlGrp+lkV71Nu+AD9Gw4qEmL8WNCNs7idc8CRpGpS2DhasNTV398lbKYzco0B
-ANlMrGlMc0w1KhuFxdU4fF3s/ktUUnjJwosxK94l5/MJ
-=C9VN
------END PGP PUBLIC KEY BLOCK-----

CLAUDE.md

@@ -1,17 +0,0 @@
-# Joplin Guidelines
-
-## Quick Reference
-
-- Tabs for indentation
-- Single quotes for strings  
-- Proper TypeScript types (avoid `any`)
-- Comments should be only with `//` and should not contain jsdoc syntax
-- If you duplicate a substantial block of code, add a comment above it noting the duplication and referencing the original location.
-- When creating Jest tests, there should be only one `describe()` statement in the file.
-- Focus on testing essential behaviour and edge cases — avoid adding tests for every minor detail.
-- Avoid duplicating code in tests; when testing the same logic with different inputs, use `test.each` or shared helpers instead of repeating similar test blocks.
-
-## Full Documentation
-
-- Coding style: [readme/dev/coding_style.md](readme/dev/coding_style.md)
-- Contributing: [CONTRIBUTING.md](CONTRIBUTING.md)

Dockerfile.transcribe

@@ -1,25 +1,26 @@
-FROM node:24-bookworm
+FROM node:24-bullseye
 
 RUN apt-get update \
     && apt-get install -y \
-    ca-certificates curl wget unzip \
-    python3 tini \
+    ca-certificates curl \
+    python3 tini
+
+## install docker
+RUN install -m 0755 -d /etc/apt/keyrings
+RUN curl -fsSL https://download.docker.com/linux/ubuntu/gpg -o /etc/apt/keyrings/docker.asc
+RUN chmod a+r /etc/apt/keyrings/docker.asc
+RUN echo \
+    "deb [arch=$(dpkg --print-architecture) signed-by=/etc/apt/keyrings/docker.asc] https://download.docker.com/linux/debian \
+    $(. /etc/os-release && echo bullseye) stable" | \
+    tee /etc/apt/sources.list.d/docker.list > /dev/null
+RUN apt-get update \
+    && apt-get install -y docker-ce docker-ce-cli containerd.io docker-buildx-plugin docker-compose-plugin \
     && rm -rf /var/lib/apt/lists/*
 
 ENV NODE_ENV=production
 
 RUN corepack enable
 
-# Download llama.cpp binary
-WORKDIR /opt/llama
-RUN wget -q https://github.com/ggml-org/llama.cpp/releases/download/b5449/llama-b5449-bin-ubuntu-x64.zip \
-    && unzip llama-b5449-bin-ubuntu-x64.zip \
-    && rm llama-b5449-bin-ubuntu-x64.zip \
-    && chmod +x /opt/llama/build/bin/llama-mtmd-cli
-
-# Create non-root user for security
-RUN groupadd -r transcribe && useradd -r -g transcribe -m transcribe
-
 WORKDIR /app
 
 COPY .yarn/releases ./.yarn/releases
@@ -43,21 +44,7 @@ RUN BUILD_SEQUENCIAL=1 yarn install --inline-builds \
     && yarn cache clean \
     && rm -rf .yarn/berry
 
-# Create data directory and set permissions
-RUN mkdir -p /data/images \
-    && chown -R transcribe:transcribe /data
-
 WORKDIR /app/packages/transcribe
 
-# Switch to non-root user
-USER transcribe
-
-# Set environment variables
-ENV HTR_CLI_BINARY_PATH=/opt/llama/build/bin/llama-mtmd-cli
-ENV LD_LIBRARY_PATH=/opt/llama/build/bin
-ENV DATA_DIR=/data
-ENV QUEUE_DRIVER=sqlite
-
 # Start the Node.js application
-ENTRYPOINT ["/usr/bin/tini", "--"]
 CMD ["yarn", "start"]

README.md

@@ -31,7 +31,7 @@ Please see the [donation page](https://github.com/laurent22/joplin/blob/dev/read
 # Sponsors
 
 <!-- SPONSORS-ORG -->
-<a href="https://seirei.ne.jp"><img title="Serei Network" width="256" src="https://joplinapp.org/images/sponsors/SeireiNetwork.png"/></a> <a href="https://citricsheep.com"><img title="Citric Sheep" width="256" src="https://joplinapp.org/images/sponsors/CitricSheep.png"/></a> <a href="https://sorted.travel/?utm_source=joplinapp"><img title="Sorted Travel" width="256" src="https://joplinapp.org/images/sponsors/SortedTravel.png"/></a> <a href="https://celebian.com"><img title="Celebian" width="256" src="https://joplinapp.org/images/sponsors/Celebian.png"/></a> <a href="https://bestkru.com"><img title="BestKru" width="256" src="https://joplinapp.org/images/sponsors/BestKru.png"/></a> <a href="https://www.socialfollowers.uk/buy-tiktok-followers/"><img title="Social Followers" width="256" src="https://joplinapp.org/images/sponsors/SocialFollowers.png"/></a> <a href="https://stormlikes.com/"><img title="Stormlikes" width="256" src="https://joplinapp.org/images/sponsors/Stormlikes.png"/></a> <a href="https://route4me.com"><img title="Route4Me" width="256" src="https://joplinapp.org/images/sponsors/Route4Me.png"/></a> <a href="https://topagency.webflow.io"><img title="WebDesignAgency" width="256" src="https://joplinapp.org/images/sponsors/WebDesignAgency.png" alt="topagency"/></a> <a href="https://essayshark.com"><img title="EssayShark - essay writers for hire" width="256" src="https://joplinapp.org/images/sponsors/EssayShark.png" alt="EssayShark - essay writers for hire"/></a> <a href="https://pokieslab1.com/real-money-pokies/"><img title="Australian Real Money Pokies" width="256" src="https://joplinapp.org/images/sponsors/PokiesLab.png" alt="Australian Real Money Pokies"/></a> <a href="https://pokiesman1.net/real-money-pokies/"><img title="Australian Real Money Pokies" width="256" src="https://joplinapp.org/images/sponsors/Pokiesman.png" alt="Australian Real Money Pokies"/></a> <a href="https://socialkings.online"><img title="Boost your reach and buy real followers" width="256" src="https://joplinapp.org/images/sponsors/SocialKings.png" alt="Boost your reach and buy real followers"/></a> <a href="https://essayservice.com/"><img title="For those in need of immediate academic assistance, EssayService offers a fast and reliable service to write my essay for me now, ensuring high-quality results within tight deadlines" width="256" src="https://joplinapp.org/images/sponsors/EssayService.png" alt="For those in need of immediate academic assistance, EssayService offers a fast and reliable service to write my essay for me now, ensuring high-quality results within tight deadlines"/></a> <a href="https://thenationonlineng.net/casino-en-ligne/casino-en-ligne-payant-au-canada/"><img title="casino en ligne le plus payant" width="256" src="https://joplinapp.org/images/sponsors/TheNationOnline.jpg" alt="casino en ligne le plus payant"/></a>
+<a href="https://seirei.ne.jp"><img title="Serei Network" width="256" src="https://joplinapp.org/images/sponsors/SeireiNetwork.png"/></a> <a href="https://citricsheep.com"><img title="Citric Sheep" width="256" src="https://joplinapp.org/images/sponsors/CitricSheep.png"/></a> <a href="https://sorted.travel/?utm_source=joplinapp"><img title="Sorted Travel" width="256" src="https://joplinapp.org/images/sponsors/SortedTravel.png"/></a> <a href="https://celebian.com"><img title="Celebian" width="256" src="https://joplinapp.org/images/sponsors/Celebian.png"/></a> <a href="https://bestkru.com"><img title="BestKru" width="256" src="https://joplinapp.org/images/sponsors/BestKru.png"/></a> <a href="https://www.socialfollowers.uk/buy-tiktok-followers/"><img title="Social Followers" width="256" src="https://joplinapp.org/images/sponsors/SocialFollowers.png"/></a> <a href="https://stormlikes.com/"><img title="Stormlikes" width="256" src="https://joplinapp.org/images/sponsors/Stormlikes.png"/></a> <a href="https://route4me.com"><img title="Route4Me" width="256" src="https://joplinapp.org/images/sponsors/Route4Me.png"/></a> <a href="https://topagency.webflow.io"><img title="WebDesignAgency" width="256" src="https://joplinapp.org/images/sponsors/WebDesignAgency.png" alt="topagency"/></a> <a href="https://www.slotozilla.com/nz/no-deposit-bonus"><img title="casino without making any upfront cost" width="256" src="https://joplinapp.org/images/sponsors/Slotozilla.png" alt="casino without making any upfront cost"/></a> <a href="https://writepaper.com/"><img title="best service to write my paper for me" width="256" src="https://joplinapp.org/images/sponsors/WritePaper.png" alt="best service to write my paper for me"/></a> <a href="https://paperwriter.com/"><img title="high-quality paper writing service PaperWriter" width="256" src="https://joplinapp.org/images/sponsors/PaperWriter.png" alt="high-quality paper writing service PaperWriter"/></a> <a href="https://www.bestetf.net/"><img title="BestETF" width="256" src="https://joplinapp.org/images/sponsors/BestEtf.png" alt="BestETF"/></a> <a href="https://freespinny.io/free-spins-no-deposit/"><img title="Freespinny.io Free Spins Bonus site" width="256" src="https://joplinapp.org/images/sponsors/Freespinny.png" alt="Freespinny.io Free Spins Bonus site"/></a> <a href="https://essayshark.com"><img title="EssayShark - essay writers for hire" width="256" src="https://joplinapp.org/images/sponsors/EssayShark.png" alt="EssayShark - essay writers for hire"/></a> <a href="https://pokieslab1.com/real-money-pokies/"><img title="Australian Real Money Pokies" width="256" src="https://joplinapp.org/images/sponsors/PokiesLab.png" alt="Australian Real Money Pokies"/></a> <a href="https://pokiesman1.net/real-money-pokies/"><img title="Australian Real Money Pokies" width="256" src="https://joplinapp.org/images/sponsors/Pokiesman.png" alt="Australian Real Money Pokies"/></a> <a href="https://domyessay.com"><img title="Essay writers DoMyEssay are dedicated to providing top-notch, custom-written papers that meet your academic requirements" width="256" src="https://joplinapp.org/images/sponsors/DoMyEssay.png" alt="DoMyEssay"/></a> <a href="https://essaypro.com/"><img title="best essay writing service" width="256" src="https://joplinapp.org/images/sponsors/EssayPro.png" alt="best essay writing service"/></a> <a href="https://socialkings.online"><img title="Boost your reach and buy real followers" width="256" src="https://joplinapp.org/images/sponsors/SocialKings.png" alt="Boost your reach and buy real followers"/></a> <a href="https://uk.notgamstop.com/bonuses/free-spins-no-deposit-no-gamstop/"><img title="free spins no deposit at NotGamstop" width="256" src="https://joplinapp.org/images/sponsors/NotGamStop.jpg" alt="free spins no deposit at NotGamstop"/></a> <a href="https://www.writemyessay.com/"><img title="writing service for students WriteMyEssay" width="256" src="https://joplinapp.org/images/sponsors/WriteMyEssay.png" alt="writing service for students WriteMyEssay"/></a> <a href="https://essayservice.com/"><img title="For those in need of immediate academic assistance, EssayService offers a fast and reliable service to write my essay for me now, ensuring high-quality results within tight deadlines" width="256" src="https://joplinapp.org/images/sponsors/EssayService.png" alt="For those in need of immediate academic assistance, EssayService offers a fast and reliable service to write my essay for me now, ensuring high-quality results within tight deadlines"/></a>
 <!-- SPONSORS-ORG -->
 
 * * *
@@ -39,9 +39,9 @@ Please see the [donation page](https://github.com/laurent22/joplin/blob/dev/read
 <!-- SPONSORS-GITHUB -->
 |       |       |       |       |
 | :---: | :---: | :---: | :---: |
-| <img width="50" src="https://avatars2.githubusercontent.com/u/552452?s=96&v=4"/></br>[andypiper](https://github.com/andypiper) | <img width="50" src="https://avatars2.githubusercontent.com/u/215668?s=96&v=4"/></br>[avanderberg](https://github.com/avanderberg) | <img width="50" src="https://avatars2.githubusercontent.com/u/67130?s=96&v=4"/></br>[chr15m](https://github.com/chr15m) | <img width="50" src="https://avatars2.githubusercontent.com/u/1177810?s=96&v=4"/></br>[felixstorm](https://github.com/felixstorm) |
-| <img width="50" src="https://avatars2.githubusercontent.com/u/8030470?s=96&v=4"/></br>[Galliver7](https://github.com/Galliver7) | <img width="50" src="https://avatars2.githubusercontent.com/u/4721118?s=96&v=4"/></br>[GPrimola](https://github.com/GPrimola) | <img width="50" src="https://avatars2.githubusercontent.com/u/64712218?s=96&v=4"/></br>[Hegghammer](https://github.com/Hegghammer) | <img width="50" src="https://avatars2.githubusercontent.com/u/42319182?s=96&v=4"/></br>[marcdw1289](https://github.com/marcdw1289) |
-| <img width="50" src="https://avatars2.githubusercontent.com/u/668977?s=96&v=4"/></br>[ugoertz](https://github.com/ugoertz) |       |       |       |
+| <img width="50" src="https://avatars2.githubusercontent.com/u/97193607?s=96&v=4"/></br>[Akhil-CM](https://github.com/Akhil-CM) | <img width="50" src="https://avatars2.githubusercontent.com/u/552452?s=96&v=4"/></br>[andypiper](https://github.com/andypiper) | <img width="50" src="https://avatars2.githubusercontent.com/u/215668?s=96&v=4"/></br>[avanderberg](https://github.com/avanderberg) | <img width="50" src="https://avatars2.githubusercontent.com/u/67130?s=96&v=4"/></br>[chr15m](https://github.com/chr15m) |
+| <img width="50" src="https://avatars2.githubusercontent.com/u/1177810?s=96&v=4"/></br>[felixstorm](https://github.com/felixstorm) | <img width="50" src="https://avatars2.githubusercontent.com/u/11947658?s=96&v=4"/></br>[KentBrockman](https://github.com/KentBrockman) | <img width="50" src="https://avatars2.githubusercontent.com/u/42319182?s=96&v=4"/></br>[marcdw1289](https://github.com/marcdw1289) | <img width="50" src="https://avatars2.githubusercontent.com/u/668977?s=96&v=4"/></br>[ugoertz](https://github.com/ugoertz) |
+|       |       |       |       |
 <!-- SPONSORS-GITHUB -->
 	
 # Community
@@ -61,14 +61,6 @@ Name | Description
 
 Please see the guide for information on how to contribute to the development of Joplin: https://github.com/laurent22/joplin/blob/dev/readme/dev/index.md
 
-## Warrant Canary Signing Key
-
-Fingerprint:
-
-F820 F830 6DD0 05A1 02D1  8CD5 946A E9FA 5915 EF53
-
-Public key: https://github.com/laurent22/joplin/raw/dev/Assets/keys/joplin-canary-signing-key.asc
-
 # Contributors
 
 Thank you to everyone who've contributed to Joplin's source code!

cspell.json

@@ -63,7 +63,6 @@
 		"/readme/_i18n",
 		"/readme/about/changelog/desktop.md",
 		"/readme/licenses.md",
-		"/readme/canary.txt",
 		"/readme/i18n",
 		"cspell.json",
 		"node_modules"

devbox.json

@@ -9,7 +9,7 @@
     "vips.dev": {
       "platforms": ["aarch64-darwin"],
     },
-    "nodejs":     "24.10.0",
+    "nodejs":     "24.9.0",
     "pkg-config": "latest",
     "python": "3.13.3",
     "bat":    "latest",

docker-compose.server.yml

@@ -84,8 +84,8 @@ services:
         profiles:
             - full
         volumes:
+            - /var/run/docker.sock:/var/run/docker.sock
             - ${HTR_CLI_IMAGES_FOLDER}:/app/packages/transcribe/images
-            - ${HTR_CLI_MODELS_FOLDER}:/opt/models:ro
         depends_on:
             - transcribe-db
         ports:
@@ -94,16 +94,6 @@ services:
             - transcribe-network
             - shared-network
         restart: unless-stopped
-        # Security: limit resources to prevent runaway processes
-        deploy:
-            resources:
-                limits:
-                    memory: 16G
-                    cpus: '4'
-        # Security: read-only root filesystem with only images folder writable
-        read_only: true
-        tmpfs:
-            - /tmp
         environment:
             - APP_PORT=4567
             - DB_CLIENT=pg
@@ -113,6 +103,5 @@ services:
             - QUEUE_DATABASE_PORT=${QUEUE_DATABASE_PORT}
             - QUEUE_DATABASE_HOST=transcribe-db
             - API_KEY=${TRANSCRIBE_API_KEY}
-            - HTR_CLI_IMAGES_FOLDER=/app/packages/transcribe/images
-            - HTR_CLI_MODELS_FOLDER=/opt/models
+            - HTR_CLI_IMAGES_FOLDER=${HTR_CLI_IMAGES_FOLDER}
 

docker-compose.transcribe.yml

@@ -1,44 +0,0 @@
-# Standalone docker-compose for Joplin Transcribe
-#
-# Uses SQLite for the queue (no external database needed).
-# Data is stored in a named volume for proper permissions.
-#
-# Usage:
-#
-#   1. Download models:
-#      mkdir -p ./data/models
-#      wget -O ./data/models/Model-7.6B-Q4_K_M.gguf https://huggingface.co/openbmb/MiniCPM-o-2_6-gguf/resolve/main/Model-7.6B-Q4_K_M.gguf
-#      wget -O ./data/models/mmproj-model-f16.gguf https://huggingface.co/openbmb/MiniCPM-o-2_6-gguf/resolve/main/mmproj-model-f16.gguf
-#
-#   2. Configure:
-#      cp .env-transcribe-sample .env
-#      # Edit .env and set API_KEY
-#
-#   3. Run:
-#      docker compose -f docker-compose.transcribe.yml up
-
-volumes:
-    transcribe-data:
-
-services:
-    transcribe:
-        image: joplin/transcribe:amd64-latest
-        ports:
-            - "4567:4567"
-        volumes:
-            - transcribe-data:/data
-            - ./data/models:/data/models:ro
-        restart: unless-stopped
-        # Security: limit resources to prevent runaway processes
-        deploy:
-            resources:
-                limits:
-                    memory: 16G
-                    cpus: '4'
-        # Security: read-only root filesystem
-        read_only: true
-        tmpfs:
-            - /tmp
-            - /home/transcribe/.cache
-        env_file:
-            - .env

package.json

@@ -60,7 +60,6 @@
     "test": "yarn workspaces foreach --worktree --parallel --verbose --interlaced --jobs 2 run test",
     "tsc": "yarn workspaces foreach --worktree --parallel --verbose --interlaced run tsc",
     "updateIgnored": "node packages/tools/gulp/tasks/updateIgnoredTypeScriptBuildRun.js",
-    "updateCanary": "node ./packages/tools/updateCanary",
     "updateMarkdownDoc": "node ./packages/tools/updateMarkdownDoc",
     "updateNews": "node ./packages/tools/website/updateNews",
     "updatePluginTypes": "./packages/generator-joplin/updateTypes.sh",
@@ -124,7 +123,6 @@
     "depd@npm:~1.1.2": "patch:depd@npm%3A2.0.0#~/.yarn/patches/depd-npm-2.0.0-b6c51a4b43.patch",
     "depd@npm:2.0.0": "patch:depd@npm%3A2.0.0#~/.yarn/patches/depd-npm-2.0.0-b6c51a4b43.patch",
     "depd@npm:^1.1.2": "patch:depd@npm%3A2.0.0#~/.yarn/patches/depd-npm-2.0.0-b6c51a4b43.patch",
-    "depd@npm:^1.1.0": "patch:depd@npm%3A2.0.0#~/.yarn/patches/depd-npm-2.0.0-b6c51a4b43.patch",
-    "formidable@npm:^2.0.1": "patch:formidable@npm%3A2.1.2#~/.yarn/patches/formidable-npm-2.1.2-40ba18d67f.patch"
+    "depd@npm:^1.1.0": "patch:depd@npm%3A2.0.0#~/.yarn/patches/depd-npm-2.0.0-b6c51a4b43.patch"
   }
 }

packages/app-cli/app/cli-utils.js

@@ -31,14 +31,9 @@ cliUtils.printArray = function(logFunction, rows) {
 		const line = [];
 		for (let col = 0; col < colWidths.length; col++) {
 			const item = rows[row][col];
-			const isLastCol = col === colWidths.length - 1;
-			if (isLastCol) {
-				line.push(item ? item.toString() : '');
-			} else {
-				const width = colWidths[col];
-				const dir = colAligns[col] === ALIGN_LEFT ? stringPadding.RIGHT : stringPadding.LEFT;
-				line.push(stringPadding(item, width, ' ', dir));
-			}
+			const width = colWidths[col];
+			const dir = colAligns[col] === ALIGN_LEFT ? stringPadding.RIGHT : stringPadding.LEFT;
+			line.push(stringPadding(item, width, ' ', dir));
 		}
 		logFunction(line.join(' '));
 	}

packages/app-cli/tests/HtmlToMd.ts

@@ -96,34 +96,4 @@ describe('HtmlToMd', () => {
 		expect(htmlToMd.parse('> 1 _2_ 3.pdf', { disableEscapeContent: false })).toBe('\\> 1 \\_2_ 3.pdf');
 	});
 
-	it('should support tightLists option', async () => {
-		const htmlToMd = new HtmlToMd();
-		const html = '<ul><li><p><strong>Item 1</strong></p></li><li><p><strong>Item 2</strong></p></li><li><p><strong>Item 3</strong></p></li></ul>';
-
-		// Without tightLists, paragraphs inside list items produce extra blank lines
-		const looseResult = htmlToMd.parse(html, { tightLists: false });
-		expect(looseResult).toContain('\n    \n');
-
-		// With tightLists, list items are compact without blank lines
-		const tightResult = htmlToMd.parse(html, { tightLists: true });
-		expect(tightResult).toBe('- **Item 1**\n- **Item 2**\n- **Item 3**');
-	});
-
-	it('should support collapseMultipleBlankLines option', async () => {
-		const htmlToMd = new HtmlToMd();
-		const html = '<p>First</p><br><br><br><p>Second</p>';
-
-		// Without collapseMultipleBlankLines, multiple blank lines are preserved
-		const looseResult = htmlToMd.parse(html, { collapseMultipleBlankLines: false });
-		expect(looseResult).toContain('\n\n  \n');
-
-		// With collapseMultipleBlankLines, multiple blank lines are collapsed into one
-		const collapsedResult = htmlToMd.parse(html, { collapseMultipleBlankLines: true });
-		expect(collapsedResult).not.toContain('\n\n\n');
-		expect(collapsedResult).not.toContain('\n\n  \n');
-
-		// Verify that a single blank line is preserved (not fully removed)
-		expect(collapsedResult).toContain('\n\n');
-	});
-
 });

packages/app-cli/tests/services/plugins/PluginService.ts

@@ -10,7 +10,6 @@ import Folder from '@joplin/lib/models/Folder';
 import { expectNotThrow, setupDatabaseAndSynchronizer, switchClient, expectThrow, createTempDir, supportDir, mockMobilePlatform } from '@joplin/lib/testing/test-utils';
 import { newPluginScript } from '../../testUtils';
 import { join } from 'path';
-import { PluginManifest } from '@joplin/lib/services/plugins/utils/types';
 
 const testPluginDir = `${supportDir}/plugins`;
 
@@ -473,18 +472,4 @@ describe('services_PluginService', () => {
 			await fs.remove(testDir);
 		}
 	});
-
-	it('should report a missing app_min_version field specifically', () => {
-		const service = newPluginService();
-		const manifest = {
-			manifest_version: 1,
-			id: 'test.plugin',
-			name: 'Test Plugin',
-			version: '1.0.0',
-			// Missing app_min_version
-		};
-
-		const error = service.describeIncompatibility(manifest as unknown as PluginManifest);
-		expect(error).toContain('Invalid plugin manifest: Missing required field: app_min_version');
-	});
 });

packages/app-desktop/app.ts

@@ -43,7 +43,7 @@ const electronContextMenu = require('./services/electron-context-menu');
 // Commands that are not tied to any particular component.
 // The runtime for these commands can be loaded when the app starts.
 
-import PerFolderSortOrderService from '@joplin/lib/services/sortOrder/PerFolderSortOrderService';
+import PerFolderSortOrderService from './services/sortOrder/PerFolderSortOrderService';
 import ShareService from '@joplin/lib/services/share/ShareService';
 import checkForUpdates from './checkForUpdates';
 import { AppState } from './app.reducer';
@@ -638,23 +638,18 @@ class Application extends BaseApplication {
 
 			if (Setting.value('env') === 'dev') {
 				void AlarmService.updateAllNotifications();
-				RevisionService.instance().runInBackground();
 			} else {
-				setTimeout(() => {
-					// Schedule sync with a delay of 0 and wrap with the desired timeout, as shim.setTimeout may not fire on first run or after an upgrade
-					// eslint-disable-next-line promise/prefer-await-to-then -- Old code before rule was applied
-					void reg.scheduleSync(0).then(() => {
-						// Wait for the first sync before updating the notifications, since synchronisation
-						// might change the notifications.
-						void AlarmService.updateAllNotifications();
+				// eslint-disable-next-line promise/prefer-await-to-then -- Old code before rule was applied
+				void reg.scheduleSync(1000).then(() => {
+					// Wait for the first sync before updating the notifications, since synchronisation
+					// might change the notifications.
+					void AlarmService.updateAllNotifications();
 
-						void DecryptionWorker.instance().scheduleStart();
-
-						RevisionService.instance().runInBackground();
-					});
-				}, 1000);
+					void DecryptionWorker.instance().scheduleStart();
+				});
 			}
 
+			RevisionService.instance().runInBackground();
 			this.startRotatingLogMaintenance(Setting.value('profileDir'));
 		});
 

packages/app-desktop/bridge.ts

@@ -583,11 +583,6 @@ export class Bridge {
 				execPath: process.env.PORTABLE_EXECUTABLE_FILE,
 			};
 			app.relaunch(options);
-		} else if (process.env.APPIMAGE && !this.altInstanceId_) {
-			app.relaunch({
-				execPath: process.env.APPIMAGE,
-				args: ['--appimage-extract-and-run'],
-			});
 		} else if (this.altInstanceId_) {
 			// Couldn't get it to work using relaunch() - it would just "close" the app, but it
 			// would still be open in the tray except unusable. Or maybe it reopens it quickly but

packages/app-desktop/commands/copyDevCommand.ts

@@ -13,8 +13,7 @@ export const runtime = (): CommandRuntime => {
 	return {
 		execute: async () => {
 			const appPath = app.getPath('exe');
-			// Quote the path so it works when it contains spaces (e.g. "C:\Program Files\Joplin\Joplin.exe" on Windows)
-			const cmd = `"${appPath}" --env dev`;
+			const cmd = `${appPath} --env dev`;
 			clipboard.writeText(cmd);
 			await shim.showMessageBox(`The dev mode command has been copied to clipboard:\n\n${cmd}`, { type: MessageBoxType.Info });
 		},

packages/app-desktop/commands/createAccessibleDocument.ts

@@ -1,79 +0,0 @@
-import { CommandRuntime, CommandDeclaration } from '@joplin/lib/services/CommandService';
-import Resource from '@joplin/lib/models/Resource';
-import Setting from '@joplin/lib/models/Setting';
-import shim from '@joplin/lib/shim';
-import { _ } from '@joplin/lib/locale';
-import { ResourceOcrStatus } from '@joplin/lib/services/database/types';
-import bridge from '../services/bridge';
-import Logger from '@joplin/utils/Logger';
-
-const logger = Logger.create('createAccessibleDocument');
-
-export const declaration: CommandDeclaration = {
-	name: 'createAccessibleDocument',
-};
-
-export const runtime = (): CommandRuntime => {
-	return {
-		execute: async (_context: unknown, resourceId: string) => {
-			const resource = await Resource.load(resourceId);
-			if (!resource) {
-				bridge().showErrorMessageBox(_('Resource not found'));
-				return;
-			}
-
-			const resourcePath = Resource.fullPath(resource);
-
-			if (resource.mime !== 'application/pdf') {
-				bridge().showInfoMessageBox(_('This feature is only available for PDF files.'));
-				return;
-			}
-
-			if (resource.ocr_status !== ResourceOcrStatus.Done) {
-				bridge().showInfoMessageBox(_('OCR is not complete. Please wait for OCR to finish before creating an accessible document.'));
-				return;
-			}
-
-			const ocrDetails = resource.ocr_details;
-
-			// If ocr_details is missing (legacy PDF processed before this feature),
-			// automatically re-run OCR to get the coordinate data
-			if (!ocrDetails) {
-				const result = await bridge().showMessageBox(_('OCR needs to run to generate an accessible document. This may take a moment. Would you like to continue?'), {
-					buttons: [_('Run OCR'), _('Cancel')],
-				});
-
-				if (result === 1) return; // User cancelled
-
-				// Trigger OCR re-run with TodoAccessible status to request full OCR details
-				await Resource.save({
-					id: resource.id,
-					ocr_status: ResourceOcrStatus.TodoAccessible,
-					ocr_details: '',
-					ocr_error: '',
-					ocr_text: '',
-				});
-
-				bridge().showInfoMessageBox(_('OCR has been queued. Please wait for it to complete and then try again.'));
-				return;
-			}
-
-			// Show save dialog
-			const defaultFilename = `${(resource.filename || resource.title || resource.id).replace(/\.pdf$/i, '')}_accessible.pdf`;
-			const outputPath = await bridge().showSaveDialog({
-				defaultPath: defaultFilename,
-				filters: [{ name: 'PDF', extensions: ['pdf'] }],
-			});
-
-			if (!outputPath) return;
-
-			try {
-				await shim.createAccessiblePdf(resourcePath, ocrDetails, outputPath, Setting.value('tempDir'));
-				await bridge().openItem(outputPath);
-			} catch (error) {
-				logger.error(error);
-				bridge().showErrorMessageBox(_('Failed to create accessible document: %s', error.message));
-			}
-		},
-	};
-};

packages/app-desktop/commands/index.ts

@@ -1,7 +1,6 @@
 // AUTO-GENERATED using `gulp buildScriptIndexes`
 import * as copyDevCommand from './copyDevCommand';
 import * as copyToClipboard from './copyToClipboard';
-import * as createAccessibleDocument from './createAccessibleDocument';
 import * as editProfileConfig from './editProfileConfig';
 import * as emptyTrash from './emptyTrash';
 import * as exportDeletionLog from './exportDeletionLog';
@@ -28,7 +27,6 @@ import * as toggleTabMovesFocus from './toggleTabMovesFocus';
 const index: any[] = [
 	copyDevCommand,
 	copyToClipboard,
-	createAccessibleDocument,
 	editProfileConfig,
 	emptyTrash,
 	exportDeletionLog,

packages/app-desktop/gui/MasterPasswordDialog/Dialog.tsx

@@ -138,7 +138,6 @@ export default function(props: Props) {
 	}, [currentPassword]);
 
 	function renderPasswordForm() {
-		const passwordsMatch = password1 === password2;
 		const renderCurrentPassword = () => {
 			if (!showCurrentPassword) return null;
 
@@ -177,22 +176,12 @@ export default function(props: Props) {
 							value={password1}
 							onChange={onPasswordChange1}
 						/>
-
 						{needToRepeatPassword && (
-							<>
-								<LabelledPasswordInput
-									labelText={_('Re-enter password')}
-									value={password2}
-									onChange={onPasswordChange2}
-									valid={password2 ? passwordsMatch : undefined}
-								/>
-
-								{password2 && !passwordsMatch && (
-									<p className="error-message">
-										{_('Passwords do not match')}
-									</p>
-								)}
-							</>
+							<LabelledPasswordInput
+								labelText={_('Re-enter password')}
+								value={password2}
+								onChange={onPasswordChange2}
+							/>
 						)}
 					</div>
 					<p className="bold">Please make sure you remember your password. For security reasons, it is not possible to recover it if it is lost.</p>

packages/app-desktop/gui/MenuBar.tsx

@@ -709,7 +709,6 @@ function useMenu(props: Props) {
 						menuItemDic.textCut,
 						menuItemDic.textPaste,
 						menuItemDic.pasteAsText,
-						menuItemDic.pasteAsMarkdown,
 						menuItemDic.textSelectAll,
 						separator(),
 						menuItemDic.globalUndo,

packages/app-desktop/gui/NoteEditor/NoteBody/CodeMirror/utils/useContextMenu.test.ts

@@ -4,38 +4,62 @@ describe('useContextMenu', () => {
 	const resourceId = 'a1b2c3d4e5f6a1b2c3d4e5f6a1b2c3d4';
 	const resourceId2 = 'b2c3d4e5f6a1b2c3d4e5f6a1b2c3d4e5';
 
-	it('should return type=image when cursor is inside markdown image', () => {
+	it('should return resource ID when cursor is inside markdown image', () => {
 		const line = `![alt text](:/${resourceId})`;
-		expect(getResourceIdFromMarkup(line, 15)).toEqual({ resourceId, type: 'image' });
+		expect(getResourceIdFromMarkup(line, 0)).toBe(resourceId);
+		expect(getResourceIdFromMarkup(line, 15)).toBe(resourceId);
+		expect(getResourceIdFromMarkup(line, line.length - 1)).toBe(resourceId);
 	});
 
-	it('should return type=file when cursor is inside markdown link', () => {
-		const line = `[document.pdf](:/${resourceId})`;
-		expect(getResourceIdFromMarkup(line, 15)).toEqual({ resourceId, type: 'file' });
-	});
-
-	it('should return null when cursor is outside markup', () => {
+	it('should return null when cursor is outside markdown image', () => {
 		const line = `Some text ![alt](:/${resourceId}) more text`;
 		expect(getResourceIdFromMarkup(line, 5)).toBeNull();
 		expect(getResourceIdFromMarkup(line, line.length - 5)).toBeNull();
 	});
 
-	it('should correctly distinguish between image and file on same line', () => {
-		const line = `![image](:/${resourceId}) [file](:/${resourceId2})`;
-		expect(getResourceIdFromMarkup(line, 10)).toEqual({ resourceId, type: 'image' });
-		expect(getResourceIdFromMarkup(line, 48)).toEqual({ resourceId: resourceId2, type: 'file' });
+	it('should handle markdown image without alt text', () => {
+		const line = `![](:/${resourceId})`;
+		expect(getResourceIdFromMarkup(line, 5)).toBe(resourceId);
+	});
+
+	it('should return resource ID when cursor is inside HTML img tag', () => {
+		const line = `<img src=":/${resourceId}" />`;
+		expect(getResourceIdFromMarkup(line, 10)).toBe(resourceId);
+	});
+
+	it('should handle HTML img tag with additional attributes', () => {
+		const line = `<img alt="test" src=":/${resourceId}" width="100" />`;
+		expect(getResourceIdFromMarkup(line, 25)).toBe(resourceId);
+	});
+
+	it('should return null when cursor is outside HTML img tag', () => {
+		const line = `text <img src=":/${resourceId}" /> more`;
+		expect(getResourceIdFromMarkup(line, 2)).toBeNull();
+		expect(getResourceIdFromMarkup(line, line.length - 2)).toBeNull();
+	});
+
+	it('should return correct resource ID when multiple images on same line', () => {
+		const line = `![first](:/${resourceId}) ![second](:/${resourceId2})`;
+		expect(getResourceIdFromMarkup(line, 10)).toBe(resourceId);
+		expect(getResourceIdFromMarkup(line, 50)).toBe(resourceId2);
 	});
 
 	it('should return null for empty line', () => {
 		expect(getResourceIdFromMarkup('', 0)).toBeNull();
 	});
 
-	it('should return null for line without resources', () => {
+	it('should return null for line without images', () => {
 		expect(getResourceIdFromMarkup('Just some regular text', 10)).toBeNull();
 	});
 
-	it('should return null for non-resource URLs', () => {
-		expect(getResourceIdFromMarkup('![alt](https://example.com/image.png)', 10)).toBeNull();
-		expect(getResourceIdFromMarkup('[link](https://example.com)', 10)).toBeNull();
+	it('should return null for non-resource links', () => {
+		const line = '![alt](https://example.com/image.png)';
+		expect(getResourceIdFromMarkup(line, 10)).toBeNull();
+	});
+
+	it('should handle cursor at exact boundaries of image markup', () => {
+		const line = `![a](:/${resourceId})`;
+		expect(getResourceIdFromMarkup(line, 0)).toBe(resourceId);
+		expect(getResourceIdFromMarkup(line, line.length)).toBe(resourceId);
 	});
 });

packages/app-desktop/gui/NoteEditor/NoteBody/CodeMirror/utils/useContextMenu.ts

@@ -11,22 +11,15 @@ import type CodeMirrorControl from '@joplin/editor/CodeMirror/CodeMirrorControl'
 import bridge from '../../../../../services/bridge';
 import Setting from '@joplin/lib/models/Setting';
 import Resource from '@joplin/lib/models/Resource';
-import { ContextMenuItemType, ContextMenuOptions, buildMenuItems, handleEditorContextMenuFilter, resolveContextMenuItemType } from '../../../utils/contextMenuUtils';
+import { ContextMenuItemType, ContextMenuOptions, buildMenuItems, handleEditorContextMenuFilter } from '../../../utils/contextMenuUtils';
 import { menuItems } from '../../../utils/contextMenu';
 import isItemId from '@joplin/lib/models/utils/isItemId';
 import { extractResourceUrls } from '@joplin/lib/urlUtils';
 import { WindowIdContext } from '../../../../NewWindowOrIFrame';
 
-export type ResourceMarkupType = 'image' | 'file';
-
-export interface ResourceMarkupInfo {
-	resourceId: string;
-	type: ResourceMarkupType;
-}
-
-// Extract resource ID from resource markup (images or file attachments) at a given cursor position within a line.
-// Returns the resource ID and its type if the cursor is within a resource markup, null otherwise.
-export const getResourceIdFromMarkup = (lineContent: string, cursorPosInLine: number): ResourceMarkupInfo | null => {
+// Extract resource ID from image markup at a given cursor position within a line.
+// Returns the resource ID if the cursor is within an image markup, null otherwise.
+export const getResourceIdFromMarkup = (lineContent: string, cursorPosInLine: number): string | null => {
 	const resourceUrls = extractResourceUrls(lineContent);
 	if (!resourceUrls.length) return null;
 
@@ -34,38 +27,16 @@ export const getResourceIdFromMarkup = (lineContent: string, cursorPosInLine: nu
 		const resourcePattern = new RegExp(`[:](/?${resourceInfo.itemId})`, 'g');
 		let match;
 		while ((match = resourcePattern.exec(lineContent)) !== null) {
-			// Look backwards for ![, [, <img, or <a
-			const imageMarkupStart = lineContent.lastIndexOf('![', match.index);
-			const linkMarkupStart = lineContent.lastIndexOf('[', match.index);
+			// Look backwards for ![ or <img
+			let markupStart = lineContent.lastIndexOf('![', match.index);
 			const imgTagStart = lineContent.lastIndexOf('<img', match.index);
-			const aTagStart = lineContent.lastIndexOf('<a', match.index);
-
-			// Find the closest markup start and determine type
-			let markupStart = -1;
-			let markupType: ResourceMarkupType = 'file';
-
-			if (imageMarkupStart !== -1 && imageMarkupStart > markupStart) {
-				markupStart = imageMarkupStart;
-				markupType = 'image';
-			}
-			if (linkMarkupStart !== -1 && linkMarkupStart > markupStart && lineContent[linkMarkupStart - 1] !== '!') {
-				markupStart = linkMarkupStart;
-				markupType = 'file';
-			}
-			if (imgTagStart !== -1 && imgTagStart > markupStart) {
-				markupStart = imgTagStart;
-				markupType = 'image';
-			}
-			if (aTagStart !== -1 && aTagStart > markupStart) {
-				markupStart = aTagStart;
-				markupType = 'file';
-			}
+			if (imgTagStart > markupStart) markupStart = imgTagStart;
 
 			if (markupStart === -1) continue;
 
 			// Find the end of the markup
 			let markupEnd: number;
-			if (lineContent[markupStart] === '!' || lineContent[markupStart] === '[') {
+			if (lineContent[markupStart] === '!') {
 				markupEnd = lineContent.indexOf(')', match.index);
 				if (markupEnd !== -1) markupEnd += 1;
 			} else {
@@ -74,7 +45,7 @@ export const getResourceIdFromMarkup = (lineContent: string, cursorPosInLine: nu
 			}
 
 			if (markupEnd !== -1 && cursorPosInLine >= markupStart && cursorPosInLine <= markupEnd) {
-				return { resourceId: resourceInfo.itemId, type: markupType };
+				return resourceInfo.itemId;
 			}
 		}
 	}
@@ -161,8 +132,8 @@ const useContextMenu = (props: ContextMenuProps) => {
 			return clickedElement?.closest(`.${imageClassName}`) as HTMLElement | null;
 		};
 
-		// Get resource info from markup at click position (not cursor position)
-		const getResourceInfoAtClickPos = (params: ContextMenuParams): ResourceMarkupInfo | null => {
+		// Get resource ID from image markup at click position (not cursor position)
+		const getResourceIdAtClickPos = (params: ContextMenuParams): string | null => {
 			if (!editorRef.current) return null;
 
 			const editor = editorRef.current.editor;
@@ -181,12 +152,10 @@ const useContextMenu = (props: ContextMenuProps) => {
 
 		const targetWindow = bridge().windowById(windowId);
 
-		const showResourceContextMenu = async (resourceId: string, type: ResourceMarkupType) => {
+		const showImageContextMenu = async (resourceId: string) => {
 			const menu = new Menu();
-			const baseType = type === 'image' ? ContextMenuItemType.Image : ContextMenuItemType.Resource;
-			const itemType = await resolveContextMenuItemType(baseType, resourceId);
 			const contextMenuOptions: ContextMenuOptions = {
-				itemType,
+				itemType: ContextMenuItemType.Image,
 				resourceId,
 				filename: null,
 				mime: null,
@@ -201,8 +170,8 @@ const useContextMenu = (props: ContextMenuProps) => {
 				mdToHtml: null,
 			};
 
-			const resourceMenuItems = await buildMenuItems(menuItems(props.dispatch), contextMenuOptions);
-			for (const item of resourceMenuItems) {
+			const imageMenuItems = await buildMenuItems(menuItems(props.dispatch), contextMenuOptions);
+			for (const item of imageMenuItems) {
 				menu.append(item);
 			}
 
@@ -237,17 +206,17 @@ const useContextMenu = (props: ContextMenuProps) => {
 					if (resourceId) {
 						event.preventDefault();
 						moveCursorToImageLine(imageContainer);
-						await showResourceContextMenu(resourceId, 'image');
+						await showImageContextMenu(resourceId);
 						return;
 					}
 				}
 			}
 
-			// Check if right-clicking on resource markup text (images or file attachments)
-			const markupResourceInfo = getResourceInfoAtClickPos(params);
-			if (markupResourceInfo && pointerInsideEditor(params)) {
+			// Check if right-clicking on image markup text
+			const markupResourceId = getResourceIdAtClickPos(params);
+			if (markupResourceId && pointerInsideEditor(params)) {
 				event.preventDefault();
-				await showResourceContextMenu(markupResourceInfo.resourceId, markupResourceInfo.type);
+				await showImageContextMenu(markupResourceId);
 				return;
 			}
 
@@ -291,16 +260,6 @@ const useContextMenu = (props: ContextMenuProps) => {
 				}),
 			);
 
-			menu.append(
-				new MenuItem({
-					label: _('Paste as Markdown'),
-					enabled: true,
-					click: async () => {
-						await CommandService.instance().execute('pasteAsMarkdown');
-					},
-				}),
-			);
-
 			const spellCheckerMenuItems = SpellCheckerService.instance().contextMenuItems(params.misspelledWord, params.dictionarySuggestions);
 
 			for (const item of spellCheckerMenuItems) {

packages/app-desktop/gui/NoteEditor/NoteBody/CodeMirror/v5/CodeMirror.tsx

@@ -338,7 +338,7 @@ function CodeMirror(props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 	}, [editorPasteText, onEditorPaste]);
 
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Old code before rule was applied
-	const loadScript = async (script: any, document: Document) => {
+	const loadScript = async (script: any) => {
 		return new Promise((resolve) => {
 			// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Old code before rule was applied
 			let element: any = document.createElement('script');
@@ -367,7 +367,6 @@ function CodeMirror(props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 	};
 
 	useEffect(() => {
-		if (!editorRoot) return () => { };
 		let cancelled = false;
 
 		async function loadScripts() {
@@ -394,14 +393,13 @@ function CodeMirror(props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 				});
 			}
 
-			const ownerDoc = editorRoot.ownerDocument;
 			for (const s of scriptsToLoad) {
-				if (ownerDoc.getElementById(s.id)) {
+				if (document.getElementById(s.id)) {
 					s.loaded = true;
 					continue;
 				}
 
-				await loadScript(s, ownerDoc);
+				await loadScript(s);
 				if (cancelled) return;
 
 				s.loaded = true;
@@ -413,7 +411,7 @@ function CodeMirror(props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 		return () => {
 			cancelled = true;
 		};
-	}, [styles.editor.codeMirrorTheme, editorRoot]);
+	}, [styles.editor.codeMirrorTheme]);
 
 	useEffect(() => {
 		if (!editorRoot) return () => {};
@@ -648,7 +646,6 @@ function CodeMirror(props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 				useCustomPdfViewer: props.useCustomPdfViewer,
 				noteId: props.noteId,
 				vendorDir: bridge().vendorDir(),
-				showNoteLinkIcon: props.showNoteLinkIcon,
 			}));
 
 			if (cancelled) return;
@@ -669,7 +666,7 @@ function CodeMirror(props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 			shim.clearTimeout(timeoutId);
 		};
 		// eslint-disable-next-line @seiyab/react-hooks/exhaustive-deps -- Old code before rule was applied
-	}, [props.content, props.contentKey, renderedBodyContentKey, props.contentMarkupLanguage, props.visiblePanes, props.resourceInfos, props.markupToHtml, props.showNoteLinkIcon]);
+	}, [props.content, props.contentKey, renderedBodyContentKey, props.contentMarkupLanguage, props.visiblePanes, props.resourceInfos, props.markupToHtml]);
 
 	useEffect(() => {
 		if (!webviewReady) return;

packages/app-desktop/gui/NoteEditor/NoteBody/CodeMirror/v6/CodeMirror.tsx

@@ -222,7 +222,6 @@ const CodeMirror = (props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 				noteId: props.noteId,
 				vendorDir: bridge().vendorDir(),
 				globalSettings: getGlobalSettings(Setting),
-				showNoteLinkIcon: props.showNoteLinkIcon,
 			}));
 
 			if (cancelled) return;
@@ -245,7 +244,7 @@ const CodeMirror = (props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 	}, [
 		props.content, props.contentKey, renderedBodyContentKey, props.contentMarkupLanguage,
 		props.visiblePanes, props.resourceInfos, props.markupToHtml, props.contentMaxWidth,
-		props.noteId, props.useCustomPdfViewer, props.showNoteLinkIcon,
+		props.noteId, props.useCustomPdfViewer,
 	]);
 
 	useEffect(() => {
@@ -352,7 +351,6 @@ const CodeMirror = (props: NoteBodyEditorProps, ref: ForwardedRef<NoteBodyEditor
 			language: isHTMLNote ? EditorLanguageType.Html : EditorLanguageType.Markdown,
 			readOnly: props.disabled,
 			markdownMarkEnabled: Setting.value('markdown.plugin.mark'),
-			markdownInsertEnabled: Setting.value('markdown.plugin.insert'),
 			katexEnabled: Setting.value('markdown.plugin.katex'),
 			inlineRenderingEnabled: Setting.value('editor.inlineRendering'),
 			imageRenderingEnabled: Setting.value('editor.imageRendering'),

packages/app-desktop/gui/NoteEditor/NoteBody/TinyMCE/TinyMCE.tsx

@@ -705,15 +705,6 @@ const TinyMCE = (props: NoteBodyEditorProps, ref: Ref<NoteBodyEditorRef>) => {
 
 			// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Old code before rule was applied
 			const containerWindow = editorContainerDom.defaultView as any;
-			const isDefaultEnglishLocale = ['en_US', 'en_GB'].includes(language);
-
-			if (!isDefaultEnglishLocale) {
-				await loadScript({
-					id: `tinyMceLang_${language}`,
-					src: `${bridge().vendorDir()}/lib/tinymce/langs/${language}.js`,
-				}, editorContainerDom);
-			}
-
 			const editors = await containerWindow.tinymce.init({
 				selector: `#${editorContainer.id}`,
 
@@ -744,7 +735,7 @@ const TinyMCE = (props: NoteBodyEditorProps, ref: Ref<NoteBodyEditorRef>) => {
 				// Handle the first table row as table header.
 				// https://www.tiny.cloud/docs/plugins/table/#table_header_type
 				table_header_type: 'sectionCells',
-				language: isDefaultEnglishLocale ? undefined : language,
+				language_url: ['en_US', 'en_GB'].includes(language) ? undefined : `${bridge().vendorDir()}/lib/tinymce/langs/${language}`,
 				toolbar: toolbar.join(' '),
 				localization_function: _,
 				// See https://www.tiny.cloud/docs/tinymce/latest/tinymce-and-csp/#content_security_policy

packages/app-desktop/gui/NoteEditor/NoteEditor.tsx

@@ -14,7 +14,7 @@ import useFormNote, { OnLoadEvent, OnSetFormNote } from './utils/useFormNote';
 import useEffectiveNoteId from './utils/useEffectiveNoteId';
 import useFolder from './utils/useFolder';
 import styles_ from './styles';
-import { NoteEditorProps, FormNote, OnChangeEvent, AllAssetsOptions, NoteBodyEditorRef, NoteBodyEditorPropsAndRef, NoteBodyEditorType } from './utils/types';
+import { NoteEditorProps, FormNote, OnChangeEvent, AllAssetsOptions, NoteBodyEditorRef, NoteBodyEditorPropsAndRef } from './utils/types';
 import CommandService from '@joplin/lib/services/CommandService';
 import Button, { ButtonLevel } from '../Button/Button';
 import eventManager, { EventName } from '@joplin/lib/eventManager';
@@ -474,7 +474,6 @@ function NoteEditorContent(props: NoteEditorProps) {
 		noteId: props.noteId,
 		watchedNoteFiles: props.watchedNoteFiles,
 		enableHtmlToMarkdownBanner: props.enableHtmlToMarkdownBanner,
-		showNoteLinkIcon: props.showNoteLinkIcon,
 	};
 
 	let editor = null;
@@ -716,11 +715,11 @@ const mapStateToProps = (state: AppState, ownProps: ConnectProps) => {
 	const windowState = stateUtils.windowStateById(state, ownProps.windowId);
 	const noteId = stateUtils.selectedNoteId(windowState);
 
-	let bodyEditor = windowState.editorCodeView ? NoteBodyEditorType.CodeMirror6 : NoteBodyEditorType.TinyMce;
+	let bodyEditor = windowState.editorCodeView ? 'CodeMirror6' : 'TinyMCE';
 	if (state.settings.isSafeMode) {
-		bodyEditor = NoteBodyEditorType.PlainText;
+		bodyEditor = 'PlainText';
 	} else if (windowState.editorCodeView && state.settings['editor.legacyMarkdown']) {
-		bodyEditor = NoteBodyEditorType.CodeMirror5;
+		bodyEditor = 'CodeMirror5';
 	}
 
 	const mainWindowState = stateUtils.windowStateById(state, defaultWindowId);
@@ -767,8 +766,6 @@ const mapStateToProps = (state: AppState, ownProps: ConnectProps) => {
 		shareCacheSetting: state.settings['sync.shareCache'],
 		searchResults: state.searchResults,
 		enableHtmlToMarkdownBanner: state.settings['editor.enableHtmlToMarkdownBanner'],
-		enableInEditorRendering: state.settings['editor.inlineRendering'],
-		showNoteLinkIcon: state.settings['notes.showNoteLinkIcon'],
 	};
 };
 

packages/app-desktop/gui/NoteEditor/WarningBanner/BannerContent.tsx

@@ -6,7 +6,6 @@ interface Props {
 	acceptMessage: string;
 	onAccept: ()=> void;
 	onDismiss?: ()=> void;
-	dismissMessage?: string;
 	visible: boolean;
 }
 
@@ -18,7 +17,7 @@ const BannerContent: React.FC<Props> = props => {
 	return <div className='warning-banner'>
 		{props.children}
 		&nbsp;&nbsp;<a onClick={props.onAccept} className='warning-banner-link' href="#">[ {props.acceptMessage} ]</a>
-		&nbsp;&nbsp;{ props.onDismiss ? <a onClick={props.onDismiss} className='warning-banner-link' href="#">[ {props.dismissMessage ?? _('Dismiss')} ]</a> : null }
+		&nbsp;&nbsp;{ props.onDismiss ? <a onClick={props.onDismiss} className='warning-banner-link' href="#">[ {_('Dismiss')} ]</a> : null }
 	</div>;
 };
 

packages/app-desktop/gui/NoteEditor/WarningBanner/WarningBanner.tsx

@@ -6,17 +6,13 @@ import BannerContent from './BannerContent';
 import { _ } from '@joplin/lib/locale';
 import onRichTextReadMoreLinkClick from '@joplin/lib/components/shared/NoteEditor/WarningBanner/onRichTextReadMoreLinkClick';
 import onRichTextDismissLinkClick from '@joplin/lib/components/shared/NoteEditor/WarningBanner/onRichTextDismissLinkClick';
-import useEditorTypeMigrationBanner from '@joplin/lib/components/shared/NoteEditor/WarningBanner/useEditorTypeMigrationBanner';
 import { useMemo } from 'react';
 import { PluginStates } from '@joplin/lib/services/plugins/reducer';
 import PluginService from '@joplin/lib/services/plugins/PluginService';
-import { NoteBodyEditorType } from '../utils/types';
 
 interface Props {
-	bodyEditor: NoteBodyEditorType;
-	editorMigrationVersion: number;
+	bodyEditor: string;
 	richTextBannerDismissed: boolean;
-	inEditorRenderingEnabled: boolean;
 	pluginCompatibilityBannerDismissedFor: string[];
 	plugins: PluginStates;
 }
@@ -39,22 +35,6 @@ const incompatiblePluginIds = [
 ];
 
 const WarningBanner: React.FC<Props> = props => {
-
-	const editorMigrationMessage = useEditorTypeMigrationBanner({
-		markdownEditorEnabled: props.bodyEditor === 'CodeMirror6',
-		editorMigrationVersion: props.editorMigrationVersion,
-		inEditorRenderingEnabled: props.inEditorRenderingEnabled,
-	});
-	const editorMigrationBanner = (
-		<BannerContent
-			visible={editorMigrationMessage.enabled}
-			acceptMessage={editorMigrationMessage.keepEnabled.label}
-			onAccept={editorMigrationMessage.keepEnabled.onPress}
-			onDismiss={editorMigrationMessage.disable.onPress}
-			dismissMessage={editorMigrationMessage.disable.label}
-		>{editorMigrationMessage.label}</BannerContent>
-	);
-
 	const wysiwygBanner = (
 		<BannerContent
 			acceptMessage={_('Read more about it')}
@@ -103,7 +83,6 @@ const WarningBanner: React.FC<Props> = props => {
 	return <>
 		{wysiwygBanner}
 		{markdownPluginBanner}
-		{editorMigrationBanner}
 	</>;
 };
 
@@ -112,7 +91,5 @@ export default connect((state: AppState) => {
 		richTextBannerDismissed: state.settings.richTextBannerDismissed,
 		pluginCompatibilityBannerDismissedFor: state.settings['editor.pluginCompatibilityBannerDismissedFor'],
 		plugins: state.pluginService.plugins,
-		editorMigrationVersion: state.settings['editor.migration'],
-		inEditorRenderingEnabled: state.settings['editor.inlineRendering'],
 	};
 })(WarningBanner);

packages/app-desktop/gui/NoteEditor/commands/index.ts

@@ -3,7 +3,6 @@ import * as focusElementNoteBody from './focusElementNoteBody';
 import * as focusElementNoteTitle from './focusElementNoteTitle';
 import * as focusElementNoteViewer from './focusElementNoteViewer';
 import * as focusElementToolbar from './focusElementToolbar';
-import * as pasteAsMarkdown from './pasteAsMarkdown';
 import * as pasteAsText from './pasteAsText';
 import * as showLocalSearch from './showLocalSearch';
 import * as showRevisions from './showRevisions';
@@ -13,7 +12,6 @@ const index: any[] = [
 	focusElementNoteTitle,
 	focusElementNoteViewer,
 	focusElementToolbar,
-	pasteAsMarkdown,
 	pasteAsText,
 	showLocalSearch,
 	showRevisions,

packages/app-desktop/gui/NoteEditor/commands/pasteAsMarkdown.ts

@@ -1,42 +0,0 @@
-import { CommandRuntime, CommandDeclaration } from '@joplin/lib/services/CommandService';
-import { _ } from '@joplin/lib/locale';
-import HtmlToMd from '@joplin/lib/HtmlToMd';
-import { processImagesInPastedHtml } from '../utils/resourceHandling';
-
-const { clipboard } = require('electron');
-
-export const declaration: CommandDeclaration = {
-	name: 'pasteAsMarkdown',
-	label: () => _('Paste as Markdown'),
-};
-
-let htmlToMd_: HtmlToMd | null = null;
-
-const htmlToMd = () => {
-	if (!htmlToMd_) {
-		htmlToMd_ = new HtmlToMd();
-	}
-	return htmlToMd_;
-};
-
-// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Props passed from NoteEditor component
-export const runtime = (comp: any): CommandRuntime => {
-	return {
-		execute: async () => {
-			let html = clipboard.readHTML();
-			if (html) {
-				// Download images and convert them to Joplin resources
-				html = await processImagesInPastedHtml(html, { useInternalUrls: true });
-				const markdown = htmlToMd().parse(html, { tightLists: true, collapseMultipleBlankLines: true });
-				comp.editorRef.current.execCommand({ name: 'insertText', value: markdown });
-			} else {
-				// Fall back to plain text if no HTML is available
-				const text = clipboard.readText();
-				if (text) {
-					comp.editorRef.current.execCommand({ name: 'insertText', value: text });
-				}
-			}
-		},
-		enabledCondition: 'oneNoteSelected && markdownEditorVisible',
-	};
-};

packages/app-desktop/gui/NoteEditor/utils/contextMenu.ts

@@ -103,17 +103,10 @@ export function menuItems(dispatch: Function): ContextMenuItems {
 				}
 			},
 			isActive: (itemType: ContextMenuItemType, options: ContextMenuOptions) => (
-				(!options.textToCopy && (itemType === ContextMenuItemType.Image || itemType === ContextMenuItemType.Resource || itemType === ContextMenuItemType.NoteLink))
+				(!options.textToCopy && (itemType === ContextMenuItemType.Image || itemType === ContextMenuItemType.Resource))
 				|| (!!options.linkToOpen && itemType === ContextMenuItemType.Link)
 			),
 		},
-		openNoteInNewWindow: {
-			label: _('Open in new window'),
-			onAction: async (options: ContextMenuOptions) => {
-				await CommandService.instance().execute('openNoteInNewWindow', options.resourceId);
-			},
-			isActive: (itemType: ContextMenuItemType) => itemType === ContextMenuItemType.NoteLink,
-		},
 		saveAs: {
 			label: _('Save as...'),
 			onAction: async (options: ContextMenuOptions) => {
@@ -214,16 +207,6 @@ export function menuItems(dispatch: Function): ContextMenuItems {
 				return itemType === ContextMenuItemType.Resource || (itemType === ContextMenuItemType.Image && options.resourceId);
 			},
 		},
-		createAccessibleDocument: {
-			label: _('Create accessible document'),
-			onAction: async (options: ContextMenuOptions) => {
-				const { resource } = await resourceInfo(options);
-				await CommandService.instance().execute('createAccessibleDocument', resource.id);
-			},
-			isActive: (itemType: ContextMenuItemType, options: ContextMenuOptions) => {
-				return itemType === ContextMenuItemType.Resource || (itemType === ContextMenuItemType.Image && options.resourceId);
-			},
-		},
 		separator3: makeSeparator(),
 		copyPathToClipboard: {
 			label: _('Copy path to clipboard'),

packages/app-desktop/gui/NoteEditor/utils/contextMenuUtils.ts

@@ -6,8 +6,6 @@ import { ContextMenuItemType, EditContextMenuFilterObject } from '@joplin/lib/se
 import eventManager from '@joplin/lib/eventManager';
 import CommandService from '@joplin/lib/services/CommandService';
 import { type MenuItem as MenuItemType } from 'electron';
-import BaseItem from '@joplin/lib/models/BaseItem';
-import { ModelType } from '@joplin/lib/BaseModel';
 
 const MenuItem = bridge().MenuItem;
 const logger = Logger.create('contextMenuUtils');
@@ -15,19 +13,6 @@ const logger = Logger.create('contextMenuUtils');
 // Re-export for backward compatibility
 export { ContextMenuItemType };
 
-// Resolves whether a resource-type item is actually a note link.
-// Falls back to Resource on error or if the item is not found.
-export const resolveContextMenuItemType = async (itemType: ContextMenuItemType, resourceId: string): Promise<ContextMenuItemType> => {
-	if (itemType !== ContextMenuItemType.Resource || !resourceId) return itemType;
-	try {
-		const item = await BaseItem.loadItemById(resourceId);
-		if (item?.type_ === ModelType.Note) return ContextMenuItemType.NoteLink;
-	} catch (error) {
-		logger.warn('resolveContextMenuItemType: failed to load item, defaulting to Resource', error);
-	}
-	return ContextMenuItemType.Resource;
-};
-
 export interface ContextMenuOptions {
 	itemType: ContextMenuItemType;
 	resourceId: string;

packages/app-desktop/gui/NoteEditor/utils/resourceHandling.test.ts

@@ -63,52 +63,4 @@ describe('resourceHandling', () => {
 		const html = `<img src="file://${encodeURI(Setting.value('resourceDir'))}/resource.png" alt="test"/>`;
 		expect(await processPastedHtml(html, htmlToMd, markupToHtml)).toBe(html);
 	});
-
-	it('should normalize HTML-encoded newlines in image alt attributes', async () => {
-		// Word encodes newlines in alt text as &#10; HTML entities. These must be
-		// normalized to spaces before Turndown processes the HTML, otherwise
-		// node.outerHTML (returned verbatim for images with width/height) embeds
-		// literal newlines that break Markdown raw HTML block parsing.
-		const resourceSrc = `file://${encodeURI(Setting.value('resourceDir'))}/resource.png`;
-		const testCases: [string, string][] = [
-			// HTML entity newlines (Word clipboard format: &#10; = LF)
-			[
-				`<img src="${resourceSrc}" alt="A screenshot&#10;&#10;AI-generated content."/>`,
-				`<img src="${resourceSrc}" alt="A screenshot AI-generated content."/>`,
-			],
-			// Literal newlines in the raw HTML attribute value
-			[
-				`<img src="${resourceSrc}" alt="hello\nworld"/>`,
-				`<img src="${resourceSrc}" alt="hello world"/>`,
-			],
-		];
-
-		for (const [html, expected] of testCases) {
-			expect(await processPastedHtml(html, null, null)).toBe(expected);
-		}
-	});
-
-	it('should render Word-pasted images with newlines in alt as img elements, not broken text', async () => {
-		// When Word pastes an image with width/height attributes and &#10; in the alt,
-		// Turndown returns node.outerHTML verbatim (preserveImageTagsWithSize=true).
-		// Without normalization, literal newlines inside the Markdown raw HTML block
-		// would terminate the block early, causing the <img> to render as plain text.
-		const { markupToHtml, htmlToMd } = createTestMarkupConverters();
-		const resourceSrc = `file://${encodeURI(Setting.value('resourceDir'))}/resource.png`;
-
-		const testCases = [
-			// Word-style: width/height present, alt has &#10; entities
-			`<img width="625" height="284" src="${resourceSrc}" alt="A screenshot&#10;&#10;AI-generated content."/>`,
-			// Multiple consecutive newline entities collapsed to single space
-			`<img width="100" height="100" src="${resourceSrc}" alt="line1&#10;&#13;&#10;line2"/>`,
-		];
-
-		for (const html of testCases) {
-			const result = await processPastedHtml(html, htmlToMd, markupToHtml);
-			// The image must be rendered as an <img> element, not as escaped/broken text
-			expect(result).toContain('<img');
-			// The alt text after normalization must not contain literal newlines
-			expect(result).not.toMatch(/alt="[^"]*\n/);
-		}
-	});
 });

packages/app-desktop/gui/NoteEditor/utils/resourceHandling.ts

@@ -2,7 +2,6 @@ import shim from '@joplin/lib/shim';
 import Setting from '@joplin/lib/models/Setting';
 import Note from '@joplin/lib/models/Note';
 import Resource from '@joplin/lib/models/Resource';
-import { ResourceEntity } from '@joplin/lib/services/database/types';
 import ResourceFetcher from '@joplin/lib/services/ResourceFetcher';
 import htmlUtils from '@joplin/lib/htmlUtils';
 import rendererHtmlUtils, { extractHtmlBody, removeWrappingParagraphAndTrailingEmptyElements } from '@joplin/renderer/htmlUtils';
@@ -121,21 +120,10 @@ export async function getResourcesFromPasteEvent(event: any) {
 }
 
 
-export interface ProcessImagesOptions {
-	// When true, returns Joplin internal URLs (:/resourceId) instead of file:// URLs
-	useInternalUrls?: boolean;
-}
-
-export const processImagesInPastedHtml = async (html: string, options: ProcessImagesOptions = {}) => {
+const processImagesInPastedHtml = async (html: string) => {
 	const allImageUrls: string[] = [];
 	const mappedResources: Record<string, string> = {};
 
-	const resourceUrl = (resource: ResourceEntity) => {
-		return options.useInternalUrls
-			? Resource.internalUrl(resource)
-			: `file://${encodeURI(Resource.fullPath(resource))}`;
-	};
-
 	htmlUtils.replaceImageUrls(html, (src: string) => {
 		allImageUrls.push(src);
 	});
@@ -150,7 +138,7 @@ export const processImagesInPastedHtml = async (html: string, options: ProcessIm
 			await shim.fetchBlob(imageSrc, { path: filePath });
 			const createdResource = await shim.createResourceFromPath(filePath);
 			await shim.fsDriver().remove(filePath);
-			mappedResources[imageSrc] = resourceUrl(createdResource);
+			mappedResources[imageSrc] = `file://${encodeURI(Resource.fullPath(createdResource))}`;
 		} catch (error) {
 			logger.warn(`Error creating a resource for ${imageSrc}.`, error);
 			mappedResources[imageSrc] = imageSrc;
@@ -167,49 +155,14 @@ export const processImagesInPastedHtml = async (html: string, options: ProcessIm
 					const imageFilePath = path.normalize(fileUriToPath(imageSrc));
 					const resourceDirPath = path.normalize(Setting.value('resourceDir'));
 
-					// Use path.relative for robust containment check - startsWith can falsely match sibling paths
-					const rel = path.relative(resourceDirPath, imageFilePath);
-					const isInsideResourceDir = rel && !rel.startsWith('..') && !path.isAbsolute(rel);
-					if (isInsideResourceDir) {
-						if (options.useInternalUrls) {
-							const resourceId = Resource.pathToId(imageFilePath);
-							mappedResources[imageSrc] = `:/${resourceId}`;
-						} else {
-							mappedResources[imageSrc] = imageSrc;
-						}
+					if (imageFilePath.startsWith(resourceDirPath)) {
+						mappedResources[imageSrc] = imageSrc;
 					} else {
 						const createdResource = await shim.createResourceFromPath(imageFilePath);
-						mappedResources[imageSrc] = resourceUrl(createdResource);
+						mappedResources[imageSrc] = `file://${encodeURI(Resource.fullPath(createdResource))}`;
 					}
 				} else if (imageSrc.startsWith('data:')) {
-					// Word encodes base64 with MIME line breaks every ~76 chars.
-					// Strip whitespace before decoding, then save as a Joplin resource
-					// so Turndown's outerHTML (used for images with width/height) gets
-					// a short file:// URL instead of 200KB of base64.
-					const cleanSrc = imageSrc.replace(/\s/g, '');
-					const dataUrlMatch = cleanSrc.match(/^data:([^;]+);base64,(.+)$/);
-					if (dataUrlMatch) {
-						const mimeType = dataUrlMatch[1];
-						const base64Data = dataUrlMatch[2];
-						const fileExt = mimeUtils.toFileExtension(mimeType) || 'bin';
-						const filePath = `${Setting.value('tempDir')}/${md5(Date.now() + Math.random())}.${fileExt}`;
-						try {
-							await shim.fsDriver().writeFile(filePath, base64Data, 'base64');
-							const createdResource = await shim.createResourceFromPath(filePath);
-							mappedResources[imageSrc] = `file://${encodeURI(Resource.fullPath(createdResource))}`;
-						} catch (writeError) {
-							writeError.message = `processPastedHtml: Failed to write or create resource from pasted image: ${writeError.message}`;
-							throw writeError;
-						} finally {
-							try {
-								await shim.fsDriver().remove(filePath);
-							} catch (cleanupError) {
-								logger.warn('processPastedHtml: Error removing temporary file.', cleanupError);
-							}
-						}
-					} else {
-						mappedResources[imageSrc] = imageSrc;
-					}
+					mappedResources[imageSrc] = imageSrc;
 				} else {
 					downloadImages.push(downloadImage(imageSrc));
 				}
@@ -235,27 +188,6 @@ export async function processPastedHtml(html: string, htmlToMd: HtmlToMarkdownHa
 
 	html = await processImagesInPastedHtml(html);
 
-	// Word encodes newlines in alt attributes as HTML entities (&#10; &#13; &#xA; etc.).
-	// These get decoded to literal newline characters by JSDOM when Turndown processes
-	// the HTML. With preserveImageTagsWithSize=true, Turndown returns node.outerHTML
-	// verbatim — embedding literal newlines inside an HTML attribute value, which
-	// breaks the Markdown raw HTML block (a blank line ends the block, making the
-	// parser treat the <img> as plain text). Normalize them to spaces here.
-	html = html.replace(
-		/(\balt\s*=\s*)(["'])([\s\S]*?)\2/gi,
-		(_m, prefix, quote, altText) => {
-			// Replace HTML-encoded newlines/control chars and literal ones with a space
-			const normalized = altText
-				.replace(/&#(?:10|13);|&#x(?:0*[aAdD]);/gi, ' ')
-				// biome-ignore lint/suspicious/noControlCharactersInRegex: intentional sanitisation of control chars
-				// eslint-disable-next-line no-control-regex
-				.replace(/[\r\n\x00-\x08\x0B\x0C\x0E-\x1F\x7F]/g, ' ')
-				.replace(/ {2,}/g, ' ')
-				.trim();
-			return `${prefix}${quote}${normalized}${quote}`;
-		},
-	);
-
 	// TinyMCE can accept any type of HTML, including HTML that may not be preserved once saved as
 	// Markdown. For example the content may have a dark background which would be supported by
 	// TinyMCE, but lost once the note is saved. So here we convert the HTML to Markdown then back

packages/app-desktop/gui/NoteEditor/utils/types.ts

@@ -26,13 +26,6 @@ export interface ToolbarButtonInfos {
 	[key: string]: ToolbarButtonInfo;
 }
 
-export enum NoteBodyEditorType {
-	CodeMirror6 = 'CodeMirror6',
-	CodeMirror5 = 'CodeMirror5',
-	TinyMce = 'TinyMCE',
-	PlainText = 'PlainText',
-}
-
 export interface NoteEditorProps {
 	noteId: string;
 	themeId: number;
@@ -72,10 +65,9 @@ export interface NoteEditorProps {
 	searchResults: ProcessResultsRow[];
 	pluginHtmlContents: PluginHtmlContents;
 	onTitleChange?: (title: string)=> void;
-	bodyEditor: NoteBodyEditorType;
+	bodyEditor: string;
 	startupPluginsLoaded: boolean;
 	enableHtmlToMarkdownBanner: boolean;
-	showNoteLinkIcon: boolean;
 }
 
 export interface NoteBodyEditorRef {
@@ -157,7 +149,6 @@ export interface NoteBodyEditorProps {
 	useCustomPdfViewer: boolean;
 	watchedNoteFiles: string[];
 	enableHtmlToMarkdownBanner: boolean;
-	showNoteLinkIcon: boolean;
 }
 
 export interface NoteBodyEditorPropsAndRef extends NoteBodyEditorProps {

packages/app-desktop/gui/NoteEditor/utils/useConnectToEditorPlugin.ts

@@ -95,18 +95,12 @@ const useConnectToEditorPlugin = ({
 	}, [activeEditorView, editorPluginHandler]);
 
 	const formNoteBody = formNote.body;
-	const formNoteId = formNote.id;
 	useEffect(() => {
-		// Don't emit updates when formNote hasn't loaded the current note yet.
-		// This can happen during note navigation when effectiveNoteId updates
-		// immediately but formNote still contains the previous note's data.
-		if (formNoteId !== effectiveNoteId) return;
-
 		editorPluginHandler.emitUpdate({
 			noteId: effectiveNoteId,
 			newBody: formNoteBody,
 		}, shownEditorViewIds);
-	}, [effectiveNoteId, formNoteId, formNoteBody, editorPluginHandler, shownEditorViewIds]);
+	}, [effectiveNoteId, formNoteBody, editorPluginHandler, shownEditorViewIds]);
 };
 
 export default useConnectToEditorPlugin;

packages/app-desktop/gui/NoteEditor/utils/useMessageHandler.ts

@@ -6,7 +6,6 @@ import PostMessageService from '@joplin/lib/services/PostMessageService';
 import ResourceFetcher from '@joplin/lib/services/ResourceFetcher';
 import { reg } from '@joplin/lib/registry';
 import bridge from '../../../services/bridge';
-import { resolveContextMenuItemType } from './contextMenuUtils';
 
 export default function useMessageHandler(
 	scrollWhenReadyRef: RefObject<ScrollOptions|null>,
@@ -47,11 +46,9 @@ export default function useMessageHandler(
 			if (s.length < 2) throw new Error(`Invalid message: ${msg}`);
 			void ResourceFetcher.instance().markForDownload(s[1]);
 		} else if (msg === 'contextMenu') {
-			const resourceId = arg0.resourceId;
-			const itemType = await resolveContextMenuItemType(arg0 && arg0.type, resourceId);
 			const menu = await contextMenu({
-				itemType,
-				resourceId: resourceId,
+				itemType: arg0 && arg0.type,
+				resourceId: arg0.resourceId,
 				filename: arg0.filename,
 				mime: arg0.mime,
 				linkToOpen: null,

packages/app-desktop/gui/NoteEditor/utils/useWindowCommandHandler.ts

@@ -13,7 +13,6 @@ const commandsWithDependencies = [
 	require('../commands/focusElementNoteViewer'),
 	require('../commands/focusElementToolbar'),
 	require('../commands/pasteAsText'),
-	require('../commands/pasteAsMarkdown'),
 ];
 
 type OnBodyChange = (event: OnChangeEvent)=> void;

packages/app-desktop/gui/NoteList/NoteList2.tsx

@@ -30,7 +30,6 @@ import useFocusVisible from './utils/useFocusVisible';
 import { stateUtils } from '@joplin/lib/reducer';
 import { connect } from 'react-redux';
 import useOnNoteDoubleClick from './utils/useOnNoteDoubleClick';
-import useAutoScroll from './utils/useAutoScroll';
 
 const commands = {
 	focusElementNoteList,
@@ -132,10 +131,6 @@ const NoteList = (props: Props) => {
 		};
 	}, [focusNote]);
 
-	const selectedNoteId = props.selectedNoteIds.length === 1 ? props.selectedNoteIds[0] : '';
-	const targetIndex = props.notes.findIndex(note => note.id === selectedNoteId);
-	useAutoScroll(selectedNoteId, props.selectedFolderId, targetIndex, makeItemIndexVisible);
-
 	const onItemContextMenu = useOnContextMenu(
 		props.selectedNoteIds,
 		props.selectedFolderId,

packages/app-desktop/gui/NoteList/utils/UseAutoScroll.test.ts

@@ -1,106 +0,0 @@
-import useAutoScroll from './useAutoScroll';
-import { renderHook } from '@testing-library/react';
-
-type Props = {
-	selectedNoteId: string;
-	selectedFolderId: string;
-	targetIndex: number;
-	makeItemIndexVisible: (index: number)=> void;
-};
-
-describe('useAutoScroll', () => {
-
-	test('scrolls to the note when a new note is selected', () => {
-		const makeItemIndexVisible = jest.fn();
-
-		renderHook(() => useAutoScroll('note-1', 'folder-1', 5, makeItemIndexVisible));
-
-		expect(makeItemIndexVisible).toHaveBeenCalledTimes(1);
-		expect(makeItemIndexVisible).toHaveBeenCalledWith(5);
-	});
-
-	test('does not scroll when the same note is already selected', () => {
-		const makeItemIndexVisible = jest.fn();
-
-		const { rerender } = renderHook(() =>
-			useAutoScroll('note-1', 'folder-1', 5, makeItemIndexVisible),
-		);
-
-		makeItemIndexVisible.mockClear();
-		rerender();
-
-		expect(makeItemIndexVisible).not.toHaveBeenCalled();
-	});
-
-	test('does not scroll for multi-selection or no selection', () => {
-		const makeItemIndexVisible = jest.fn();
-
-		renderHook(() => useAutoScroll('', 'folder-1', -1, makeItemIndexVisible));
-
-		expect(makeItemIndexVisible).not.toHaveBeenCalled();
-	});
-
-	test('defers scroll until notes load after folder change', () => {
-		const makeItemIndexVisible = jest.fn();
-
-		const { rerender } = renderHook(
-			(props: Props) => useAutoScroll(
-				props.selectedNoteId,
-				props.selectedFolderId,
-				props.targetIndex,
-				props.makeItemIndexVisible,
-			),
-			{ initialProps: { selectedNoteId: 'note-1', selectedFolderId: 'folder-2', targetIndex: -1, makeItemIndexVisible } },
-		);
-
-		expect(makeItemIndexVisible).not.toHaveBeenCalled();
-
-		rerender({ selectedNoteId: 'note-1', selectedFolderId: 'folder-2', targetIndex: 3, makeItemIndexVisible });
-
-		expect(makeItemIndexVisible).toHaveBeenCalledTimes(1);
-		expect(makeItemIndexVisible).toHaveBeenCalledWith(3);
-	});
-
-	test('scrolls again when the folder changes even if note ID is the same', () => {
-		const makeItemIndexVisible = jest.fn();
-
-		const { rerender } = renderHook(
-			(props: Props) => useAutoScroll(
-				props.selectedNoteId,
-				props.selectedFolderId,
-				props.targetIndex,
-				props.makeItemIndexVisible,
-			),
-			{ initialProps: { selectedNoteId: 'note-1', selectedFolderId: 'folder-1', targetIndex: 2, makeItemIndexVisible } },
-		);
-
-		expect(makeItemIndexVisible).toHaveBeenCalledTimes(1);
-
-		rerender({ selectedNoteId: 'note-1', selectedFolderId: 'folder-2', targetIndex: 2, makeItemIndexVisible });
-
-		expect(makeItemIndexVisible).toHaveBeenCalledTimes(2);
-	});
-
-	test('does not scroll again when targetIndex changes after the pending flag is cleared', () => {
-		// Covers the case where a sort or filter changes targetIndex without a new selection.
-		// Without this guard, arrow-key navigation would trigger a spurious second scroll.
-		const makeItemIndexVisible = jest.fn();
-
-		const { rerender } = renderHook(
-			(props: Props) => useAutoScroll(
-				props.selectedNoteId,
-				props.selectedFolderId,
-				props.targetIndex,
-				props.makeItemIndexVisible,
-			),
-			{ initialProps: { selectedNoteId: 'note-1', selectedFolderId: 'folder-1', targetIndex: 5, makeItemIndexVisible } },
-		);
-
-		expect(makeItemIndexVisible).toHaveBeenCalledTimes(1);
-
-		rerender({ selectedNoteId: 'note-1', selectedFolderId: 'folder-1', targetIndex: 7, makeItemIndexVisible });
-
-		expect(makeItemIndexVisible).toHaveBeenCalledTimes(1);
-	});
-
-});

packages/app-desktop/gui/NoteList/utils/useAutoScroll.ts

@@ -1,43 +0,0 @@
-import { useRef, useEffect } from 'react';
-
-// Auto-scrolls the note list to the selected note when selection changes. Uses a pending flag
-// to handle cross-folder navigation where notes may not be loaded on the first render.
-const useAutoScroll = (
-	selectedNoteId: string,
-	selectedFolderId: string,
-	targetIndex: number,
-	makeItemIndexVisible: (index: number)=> void,
-) => {
-	const lastNoteIdRef = useRef('');
-	const lastFolderIdRef = useRef('');
-	const scrollPendingRef = useRef(false); // true when scroll requested but notes not yet loaded
-
-	useEffect(() => {
-		// No selection or multi-selection — reset tracking state.
-		if (!selectedNoteId) {
-			lastNoteIdRef.current = '';
-			lastFolderIdRef.current = selectedFolderId;
-			scrollPendingRef.current = false;
-			return;
-		}
-
-		const isNewNote = selectedNoteId !== lastNoteIdRef.current;
-		const isFolderChange = selectedFolderId !== lastFolderIdRef.current;
-
-		if (isNewNote || isFolderChange) {
-			lastNoteIdRef.current = selectedNoteId;
-			lastFolderIdRef.current = selectedFolderId;
-			scrollPendingRef.current = true;
-		}
-
-		// targetIndex is -1 until the new folder's notes load — re-runs automatically when they do.
-		if (!scrollPendingRef.current || targetIndex === -1) return;
-
-		// makeItemIndexVisible has its own visibility guard and is a no-op when the note is
-		// already visible — this covers arrow-key and click navigation without double-scrolling.
-		makeItemIndexVisible(targetIndex);
-		scrollPendingRef.current = false;
-	}, [selectedNoteId, selectedFolderId, targetIndex, makeItemIndexVisible]);
-};
-
-export default useAutoScroll;

packages/app-desktop/gui/NoteListControls/NoteListControls.tsx

@@ -6,7 +6,7 @@ import Button, { ButtonLevel, ButtonSize, buttonSizePx } from '../Button/Button'
 import CommandService from '@joplin/lib/services/CommandService';
 import { runtime as focusSearchRuntime } from './commands/focusSearch';
 import Note from '@joplin/lib/models/Note';
-import { notesSortOrderNextField } from '@joplin/lib/services/sortOrder/notesSortOrderUtils';
+import { notesSortOrderNextField } from '../../services/sortOrder/notesSortOrderUtils';
 import { _ } from '@joplin/lib/locale';
 import { connect } from 'react-redux';
 import styled from 'styled-components';

packages/app-desktop/gui/NoteRevisionViewer.tsx

@@ -35,7 +35,6 @@ interface Props {
 	customCss: string;
 	scrollbarSize: ScrollbarSize;
 	fontFamily: string;
-	showNoteLinkIcon: boolean;
 }
 
 const useNoteContent = (
@@ -46,7 +45,6 @@ const useNoteContent = (
 	customCss: string,
 	scrollbarSize: ScrollbarSize,
 	fontFamily: string,
-	showNoteLinkIcon: boolean,
 ) => {
 	const [note, setNote] = useState<NoteEntity>(null);
 
@@ -77,18 +75,17 @@ const useNoteContent = (
 			resources: await shared.attachedResources(noteBody),
 			whiteBackgroundNoteRendering: markupLanguage === MarkupLanguage.Html,
 			globalSettings: getGlobalSettings(Setting),
-			showNoteLinkIcon,
 		});
 
 		viewerRef.current.setHtml(result.html, {
 			pluginAssets: result.pluginAssets,
 		});
-	}, [note, viewerRef, markupToHtml, showNoteLinkIcon]);
+	}, [note, viewerRef]);
 
 	return note;
 };
 
-const NoteRevisionViewerComponent: React.FC<Props> = ({ themeId, noteId, onBack, customCss, scrollbarSize, fontFamily, showNoteLinkIcon }) => {
+const NoteRevisionViewerComponent: React.FC<Props> = ({ themeId, noteId, onBack, customCss, scrollbarSize, fontFamily }) => {
 	const helpButton_onClick = useCallback(() => {}, []);
 	const viewerRef = useRef<NoteViewerControl|null>(null);
 	const revisionListRef = useRef<HTMLSelectElement|null>(null);
@@ -99,7 +96,7 @@ const NoteRevisionViewerComponent: React.FC<Props> = ({ themeId, noteId, onBack,
 	const [deleting, setDeleting] = useState(false);
 
 	const note = useNoteContent(
-		viewerRef, currentRevId, revisions, themeId, customCss, scrollbarSize, fontFamily, showNoteLinkIcon,
+		viewerRef, currentRevId, revisions, themeId, customCss, scrollbarSize, fontFamily,
 	);
 
 	const viewer_domReady = useCallback(async () => {
@@ -232,7 +229,6 @@ const mapStateToProps = (state: AppState) => {
 		themeId: state.settings.theme,
 		scrollbarSize: state.settings['style.scrollbarSize'],
 		fontFamily: state.settings['style.viewer.fontFamily'],
-		showNoteLinkIcon: state.settings['notes.showNoteLinkIcon'],
 	};
 };
 

packages/app-desktop/gui/Sidebar/hooks/useFocusHandler.ts

@@ -28,19 +28,12 @@ const useScrollToSelectionHandler = (
 			return lastSelectedItemKey.current;
 		}
 	}, [listItems, selectedIndex]);
+	lastSelectedItemKey.current = selectedItemKey;
 
 	const selectedIndexRef = useRef(selectedIndex);
 	selectedIndexRef.current = selectedIndex;
 
 	useEffect(() => {
-		// Skip scrolling if the selected item hasn't actually changed. When a folder is
-		// expanded or collapsed the selected item's index may shift, but its key stays
-		// the same — in that case we don't want to scroll the view.
-		if (selectedItemKey === lastSelectedItemKey.current) {
-			return;
-		}
-		lastSelectedItemKey.current = selectedItemKey;
-
 		if (!itemListRef.current || !selectedItemKey) return;
 
 		const hasFocus = !!itemListRef.current.container.contains(document.activeElement);

packages/app-desktop/gui/Sidebar/hooks/useOnRenderItem.tsx

@@ -18,7 +18,7 @@ import { FolderEntity } from '@joplin/lib/services/database/types';
 import InteropService from '@joplin/lib/services/interop/InteropService';
 import InteropServiceHelper from '../../../InteropServiceHelper';
 import Setting from '@joplin/lib/models/Setting';
-import PerFolderSortOrderService from '@joplin/lib/services/sortOrder/PerFolderSortOrderService';
+import PerFolderSortOrderService from '../../../services/sortOrder/PerFolderSortOrderService';
 import { getFolderCallbackUrl, getTagCallbackUrl } from '@joplin/lib/callbackUrlUtils';
 import { PluginStates, utils as pluginUtils } from '@joplin/lib/services/plugins/reducer';
 import { MenuItemLocation } from '@joplin/lib/services/plugins/api/types';
@@ -54,42 +54,12 @@ type ItemContextMenuListener = MouseEventHandler<HTMLElement>;
 
 const menuUtils = new MenuUtils(CommandService.instance());
 
-// Checks whether an element is at least partially visible within a scrollable
-// container by comparing their bounding rectangles.
-const isElementVisibleInContainer = (element: HTMLElement, container: HTMLElement) => {
-	const elementRect = element.getBoundingClientRect();
-	const containerRect = container.getBoundingClientRect();
-
-	return elementRect.bottom > containerRect.top && elementRect.top < containerRect.bottom;
-};
-
 const focusListItem = (item: HTMLElement|null) => {
 	if (item) {
-		const activeElement = document.activeElement instanceof HTMLElement ? document.activeElement : null;
-		const itemList = item.closest('.item-list');
-		const activeTreeItem = activeElement?.closest('[role="treeitem"]');
-		const focusWasLost = activeElement === document.body;
-
-		// If the currently focused element is a tree item inside the same list,
-		// the user is navigating with the keyboard — always allow focus to move
-		// to the newly selected item so arrow-key scrolling is not interrupted.
-		const isKeyboardNavigating = !!activeTreeItem && itemList?.contains(activeTreeItem);
-
-		// Avoid disturbing scroll while user is manually scrolling through the list.
-		// However, if focus was lost (activeElement -> <body>), or the user is
-		// navigating with the keyboard, allow re-focusing even when the selected
-		// item is currently out of view.
-		if (itemList instanceof HTMLElement && !isElementVisibleInContainer(item, itemList) && !focusWasLost && !isKeyboardNavigating) {
-			return;
-		}
-
-		// Move focus only if needed: either focus was lost, or selection changed
-		// to a different tree item.
-		if (focusWasLost || activeTreeItem !== item) {
-			// preventScroll: true avoids a secondary scroll caused by the focus() call
-			// itself when the item is near the edge of the visible area.
-			focus('useOnRenderItem', item, { preventScroll: true });
-		}
+		// Avoid scrolling to the selected item when refocusing the note list. Such a refocus
+		// can happen if the note list rerenders and the selection is scrolled out of view and
+		// can cause scroll to change unexpectedly.
+		focus('useOnRenderItem', item, { preventScroll: true });
 	}
 };
 
@@ -393,12 +363,7 @@ const useOnRenderItem = (props: Props) => {
 			multipleItemsSelected: props.selectedIndexes.length > 1,
 		};
 
-		const sidebarContainsFocus = props.containerRef.current?.contains(document.activeElement);
-		// Focus moves to <body> when the previously-focused element is removed
-		// from the DOM (e.g. scrolled out of the virtualized list). We still
-		// want to restore focus to the newly-selected item in that case.
-		const focusLostFromDom = document.activeElement === document.body;
-		const focusInList = document.hasFocus() && (sidebarContainsFocus || focusLostFromDom);
+		const focusInList = document.hasFocus() && props.containerRef.current?.contains(document.activeElement);
 		const anchorRef = (focusInList && primarySelected) ? focusListItem : noFocusListItem;
 
 		if (item.kind === ListItemType.Tag) {

packages/app-desktop/gui/Sidebar/listItemComponents/AllNotesItem.tsx

@@ -6,7 +6,7 @@ import bridge from '../../../services/bridge';
 import Setting from '@joplin/lib/models/Setting';
 import MenuUtils from '@joplin/lib/services/commands/MenuUtils';
 import CommandService from '@joplin/lib/services/CommandService';
-import PerFolderSortOrderService from '@joplin/lib/services/sortOrder/PerFolderSortOrderService';
+import PerFolderSortOrderService from '../../../services/sortOrder/PerFolderSortOrderService';
 import { connect } from 'react-redux';
 import EmptyExpandLink from './EmptyExpandLink';
 import ListItemWrapper, { ItemSelectionState, ListItemRef } from './ListItemWrapper';

packages/app-desktop/gui/WindowCommandsAndDialogs/commands/openItem.ts

@@ -30,27 +30,19 @@ export const runtime = (): CommandRuntime => {
 				} else {
 					void bridge().openExternal(link);
 				}
-			} else if (link.indexOf('file://') === 0) {
-				// When using the file:// protocol, openPath doesn't work (does
-				// nothing) with URL-encoded paths.
-				//
-				// shell.openPath seems to work with file:// urls on Windows,
-				// but doesn't on macOS, so we need to convert it to a path
-				// before passing it to openPath.
-				let decoded = urlDecode(link);
-
-				// On Windows, UNC paths like file://\\server\share have backslashes
-				// right after file:// which makes the URL invalid. Convert them
-				// to forward slashes so fileUriToPath can handle them correctly.
-				// https://github.com/laurent22/joplin/issues/14196
-				if (decoded.startsWith('file://\\')) {
-					decoded = `file://${decoded.substring(7).replace(/\\/g, '/')}`;
-				}
-
-				const decodedPath = fileUriToPath(decoded, shim.platformName());
-				void bridge().openItem(decodedPath);
 			} else if (urlProtocol(link)) {
-				void bridge().openExternal(link);
+				if (link.indexOf('file://') === 0) {
+					// When using the file:// protocol, openPath doesn't work (does
+					// nothing) with URL-encoded paths.
+					//
+					// shell.openPath seems to work with file:// urls on Windows,
+					// but doesn't on macOS, so we need to convert it to a path
+					// before passing it to openPath.
+					const decodedPath = fileUriToPath(urlDecode(link), shim.platformName());
+					void bridge().openItem(decodedPath);
+				} else {
+					void bridge().openExternal(link);
+				}
 			} else {
 				bridge().showErrorMessageBox(_('Unsupported link or message: %s', link));
 			}

packages/app-desktop/gui/WindowCommandsAndDialogs/commands/toggleNotesSortOrderField.ts

@@ -1,5 +1,5 @@
 import { CommandContext, CommandDeclaration, CommandRuntime } from '@joplin/lib/services/CommandService';
-import { setNotesSortOrder } from '@joplin/lib/services/sortOrder/notesSortOrderUtils';
+import { setNotesSortOrder } from '../../../services/sortOrder/notesSortOrderUtils';
 import { _ } from '@joplin/lib/locale';
 
 export const declaration: CommandDeclaration = {

packages/app-desktop/gui/WindowCommandsAndDialogs/commands/toggleNotesSortOrderReverse.ts

@@ -1,7 +1,7 @@
 import { CommandContext, CommandDeclaration, CommandRuntime } from '@joplin/lib/services/CommandService';
 import Setting from '@joplin/lib/models/Setting';
 import { _ } from '@joplin/lib/locale';
-import { setNotesSortOrder } from '@joplin/lib/services/sortOrder/notesSortOrderUtils';
+import { setNotesSortOrder } from '../../../services/sortOrder/notesSortOrderUtils';
 
 export const declaration: CommandDeclaration = {
 	name: 'toggleNotesSortOrderReverse',

packages/app-desktop/gui/WindowCommandsAndDialogs/commands/togglePerFolderSortOrder.ts

@@ -1,6 +1,6 @@
 import { CommandContext, CommandDeclaration, CommandRuntime } from '@joplin/lib/services/CommandService';
 import { _ } from '@joplin/lib/locale';
-import PerFolderSortOrderService from '@joplin/lib/services/sortOrder/PerFolderSortOrderService';
+import PerFolderSortOrderService from '../../../services/sortOrder/PerFolderSortOrderService';
 
 export const declaration: CommandDeclaration = {
 	name: 'togglePerFolderSortOrder',

packages/app-desktop/gui/menuCommandNames.ts

@@ -81,7 +81,6 @@ export default function() {
 		'switchProfile2',
 		'switchProfile3',
 		'pasteAsText',
-		'pasteAsMarkdown',
 		'showNoteProperties',
 		'convertNoteToMarkdown',
 		'toggleEditors',

packages/app-desktop/gui/note-viewer/index.html

@@ -35,10 +35,6 @@
 		ul ul, ul ol, ol ul, ol ol   {
 			margin-bottom: 0px;			
 		}
-		::highlight(search-results) {
-    		background-color: #f7d26e;
-    		color: black;
-		}
 	</style>
 </head>
 
@@ -507,7 +503,7 @@
 			if (!options) options = {};
 
 			// TODO: Add support for scriptType on mobile and CLI
-			CSS.highlights.clear();
+
 			if (!mark_) {
 				mark_ = new Mark(document.getElementById('joplin-container-content'), {
 					exclude: ['img'],
@@ -523,8 +519,7 @@
 
 			let selectedElement = null;
 			let elementIndex = 0;
-			
-			const allRanges = [];
+
 			const markKeywordOptions = {};
 
 			if ('separateWordSearch' in options) markKeywordOptions.separateWordSearch = options.separateWordSearch;
@@ -532,17 +527,9 @@
 			try {
 				for (const keyword of keywords) {
 					markJsUtils.markKeyword(mark_, keyword, {
-                        pregQuote: pregQuote,
-                        replaceRegexDiacritics: replaceRegexDiacritics,
-                    }, {
-                        ...markKeywordOptions,
-                        element: 'mark-ghost',
-                        each: (node) => {
-                            const range = new Range();
-                            range.selectNodeContents(node);
-                            allRanges.push(range);
-                        }
-                    });
+						pregQuote: pregQuote,
+						replaceRegexDiacritics: replaceRegexDiacritics,
+					}, markKeywordOptions);
 				}
 			} catch (error) {
 				if (error.name !== 'SyntaxError') {
@@ -553,10 +540,6 @@
 				// https://github.com/laurent22/joplin/issues/7634
 				console.error('Error while trying to highlight words from search: ', error);
 			}
-			if (allRanges.length > 0) {
-				const searchResultsHighlight = new Highlight(...allRanges);
-            	CSS.highlights.set("search-results", searchResultsHighlight);
-			}
 		}
 
 		let markLoader_ = { state: 'idle', whenDone: null };
@@ -826,37 +809,6 @@
 			);
 		}));
 
-		// By default, Chromium inlines body styles (e.g. theme background color) into the clipboard HTML.
-		// Intercept the copy event and write only the selected content to bypass this behaviour.
-		document.addEventListener('copy', webviewLib.logEnabledEventHandler(e => {
-			const selection = window.getSelection();
-			if (!selection || selection.isCollapsed) return;
-			if (selection.rangeCount === 0 || !e.clipboardData) return;
-
-			const range = selection.getRangeAt(0);
-			const wrapper = document.createElement('div');
-			wrapper.appendChild(range.cloneContents());
-
-			wrapper.querySelectorAll('style').forEach(s => s.remove());
-
-			const inlineTags = new Set(['STRONG', 'EM', 'CODE', 'S', 'DEL', 'INS', 'MARK', 'SUP', 'SUB', 'U', 'SPAN', 'A']);
-			let node = range.commonAncestorContainer;
-			if (node.nodeType === Node.TEXT_NODE) node = node.parentElement;
-
-			while (node && node !== document.body && node.id !== 'rendered-md' && node.id !== 'joplin-container-content') {
-				if (inlineTags.has(node.tagName)) {
-					const el = node.cloneNode(false);
-					while (wrapper.firstChild) el.appendChild(wrapper.firstChild);
-					wrapper.appendChild(el);
-				}
-				node = node.parentElement;
-			}
-
-			e.clipboardData.setData('text/html', wrapper.innerHTML);
-			e.clipboardData.setData('text/plain', selection.toString());
-			e.preventDefault();
-		}));
-
 		let lastClientWidth_ = NaN, lastClientHeight_ = NaN, lastScrollTop_ = NaN;
 
 		window.addEventListener('resize', webviewLib.logEnabledEventHandler(() => {

packages/app-desktop/integration-tests/markdownEditor.spec.ts

@@ -161,7 +161,7 @@ test.describe('markdownEditor', () => {
 		const viewer = noteEditor.getNoteViewerFrameLocator();
 		await expect(viewer.locator('h1')).toHaveText('Testing');
 
-		const matches = viewer.locator('mark-ghost');
+		const matches = viewer.locator('mark');
 		await expect(matches).toHaveCount(0);
 
 		await mainWindow.keyboard.press(process.platform === 'darwin' ? 'Meta+f' : 'Control+f');
@@ -276,12 +276,8 @@ test.describe('markdownEditor', () => {
 		expect(imageSize[1]).toBeGreaterThan(0);
 	});
 
-	test('ctrl-clicking on note links should open the linked note (when the viewer is hidden)', async ({ mainWindow, electronApp }) => {
-		const mainScreen = await new MainScreen(mainWindow);
-		// Workaround: Required for extracting content accurately from the Markdown editor
-		await mainScreen.noteEditor.disableInlineRendering(electronApp);
-
-		await mainScreen.setup();
+	test('ctrl-clicking on note links should open the linked note (when the viewer is hidden)', async ({ mainWindow }) => {
+		const mainScreen = await new MainScreen(mainWindow).setup();
 		await mainScreen.createNewNote('Original');
 		const noteEditor = mainScreen.noteEditor;
 		await noteEditor.hideViewer();
@@ -404,35 +400,5 @@ test.describe('markdownEditor', () => {
 		await activateMainMenuItem(electronApp, 'Redo');
 		await noteEditor.expectToHaveText('A');
 	});
-
-	test('copying from the preview pane should not include theme background color and should preserve bold formatting', async ({ mainWindow, electronApp }) => {
-		// Set dark theme so background-color would be present in clipboard without the fix
-		await setSettingValue(electronApp, mainWindow, 'theme', 2);
-
-		const mainScreen = await new MainScreen(mainWindow).setup();
-		await mainScreen.waitFor();
-
-		await mainScreen.createNewNote('Test copy formatting');
-		const noteEditor = mainScreen.noteEditor;
-		await noteEditor.focusCodeMirrorEditor();
-		await mainWindow.keyboard.type('**hello**');
-
-		const viewerFrame = noteEditor.getNoteViewerFrameLocator();
-		await expect(viewerFrame.locator('strong')).toHaveText('hello');
-
-		// Double-click selects the text node inside <strong>, not <strong> itself.
-		// Without the ancestor re-wrapping fix, <strong> would be dropped.
-		await viewerFrame.locator('strong').dblclick();
-		const modifier = process.platform === 'darwin' ? 'Meta' : 'Control';
-		await mainWindow.keyboard.press(`${modifier}+c`);
-
-		const clipboardHtml = await mainWindow.evaluate(() => {
-			const { clipboard } = require('electron');
-			return clipboard.readHTML();
-		});
-
-		expect(clipboardHtml).toContain('hello');
-		expect(clipboardHtml).not.toMatch(/background-color\s*:/i);
-		expect(clipboardHtml).toContain('<strong>');
-	});
 });
+

packages/app-desktop/integration-tests/models/NoteEditorScreen.ts

@@ -2,7 +2,6 @@ import { ElectronApplication, Locator, Page } from '@playwright/test';
 import { expect } from '../util/test';
 import activateMainMenuItem from '../util/activateMainMenuItem';
 import EditorCodeDialog from './EditorCodeDialog';
-import setSettingValue from '../util/setSettingValue';
 
 export default class NoteEditorPage {
 	public readonly codeMirrorEditor: Locator;
@@ -25,8 +24,8 @@ export default class NoteEditorPage {
 
 	private readonly containerLocator: Locator;
 
-	public constructor(private page_: Page) {
-		this.containerLocator = page_.locator('.rli-editor');
+	public constructor(page: Page) {
+		this.containerLocator = page.locator('.rli-editor');
 		this.codeMirrorEditor = this.containerLocator.locator('.cm-editor');
 		this.richTextEditor = this.containerLocator.locator('iframe[title="Rich Text Area"]');
 		this.editorPluginFrame = this.containerLocator.locator('iframe[id^="plugin-view-"]');
@@ -42,7 +41,7 @@ export default class NoteEditorPage {
 		this.disableTabNavigationButton = this.containerLocator.getByRole('button', { name: 'Tab moves focus' });
 		this.toggleEditorPluginButton = this.containerLocator.getByRole('button', { name: 'Toggle editor plugin' });
 
-		this.richTextCodeEditor = new EditorCodeDialog(page_);
+		this.richTextCodeEditor = new EditorCodeDialog(page);
 	}
 
 	public toolbarButtonLocator(title: string) {
@@ -66,10 +65,6 @@ export default class NoteEditorPage {
 		}
 	}
 
-	public async disableInlineRendering(electronApp: ElectronApplication) {
-		await setSettingValue(electronApp, this.page_, 'editor.inlineRendering', false);
-	}
-
 	public async expectToHaveText(expected: string|RegExp) {
 		// expect(...).toHaveText can fail in the Rich Text Editor (perhaps due to frame locators).
 		// Using expect.poll refreshes the locator on each attempt, which seems to prevent flakiness.

packages/app-desktop/integration-tests/pluginApi.spec.ts

@@ -2,7 +2,6 @@
 import { test, expect } from './util/test';
 import MainScreen from './models/MainScreen';
 import { msleep, Second } from '@joplin/utils/time';
-import setSettingValue from './util/setSettingValue';
 
 test.describe('pluginApi', () => {
 	test('the editor.setText command should update the current note (use RTE: false)', async ({ startAppWithPlugins }) => {
@@ -81,30 +80,6 @@ test.describe('pluginApi', () => {
 		await expectVisible(false);
 	});
 
-	// Regression tests for #13718
-	for (const method of ['Cancel button', 'Escape key'] as const) {
-		test(`should dismiss a plugin dialog via ${method} with isolated iframes`, async ({ startAppWithPlugins }) => {
-			const { app, mainWindow } = await startAppWithPlugins(['resources/test-plugins/dialogs.js']);
-			const mainScreen = await new MainScreen(mainWindow).setup();
-			await mainScreen.createNewNote('Test note');
-
-			// WebView isolation is currently behind a feature flag:
-			await setSettingValue(app, mainWindow, 'featureFlag.plugins.isolatePluginWebViews', true);
-
-			await mainScreen.goToAnything.runCommand(app, 'showTestDialogWithDismiss');
-			const dialogContent = mainScreen.dialog.locator('iframe').contentFrame();
-			await dialogContent.locator('p').waitFor();
-
-			if (method === 'Cancel button') {
-				await mainScreen.dialog.getByRole('button', { name: 'Cancel' }).click();
-			} else {
-				await mainWindow.keyboard.press('Escape');
-			}
-
-			await expect(mainScreen.dialog).toBeHidden();
-		});
-	}
-
 	test('should be possible to create multiple toasts with the same text from a plugin', async ({ startAppWithPlugins }) => {
 		const { app, mainWindow } = await startAppWithPlugins(['resources/test-plugins/showToast.js']);
 		const mainScreen = await new MainScreen(mainWindow).setup();
@@ -206,26 +181,5 @@ test.describe('pluginApi', () => {
 
 		await expect(panelLocator).not.toBeVisible();
 	});
-
-	// Regression test for https://github.com/laurent22/joplin/issues/12793
-	test('a plugin that crashes before register() should not block other plugins', async ({ startAppWithPlugins }) => {
-		// Load a crashing plugin alongside a working plugin. If the fix works,
-		// startAppWithPlugins completes because startup-plugins-loaded fires.
-		// Without the fix, this test would timeout because allPluginsStarted
-		// never becomes true.
-		const { app, mainWindow } = await startAppWithPlugins([
-			'resources/test-plugins/crashBeforeRegister.js',
-			'resources/test-plugins/execCommand.js',
-		]);
-		const mainScreen = await new MainScreen(mainWindow).setup();
-		await mainScreen.createNewNote('Test note');
-
-		// Verify the working plugin is functional
-		const editor = mainScreen.noteEditor;
-		await editor.focusCodeMirrorEditor();
-		await mainWindow.keyboard.type('Should be overwritten.');
-		await mainScreen.goToAnything.runCommand(app, 'testUpdateEditorText');
-		await editor.expectToHaveText('PASS');
-	});
 });
 

packages/app-desktop/integration-tests/resources/test-plugins/crashBeforeRegister.js

@@ -1,22 +0,0 @@
-// Allows referencing the Joplin global:
-/* eslint-disable no-undef */
-
-// Allows the `joplin-manifest` block comment:
-/* eslint-disable multiline-comment-style */
-
-/* joplin-manifest:
-{
-	"id": "org.joplinapp.plugins.example.crashBeforeRegister",
-	"manifest_version": 1,
-	"app_min_version": "3.1",
-	"name": "Crash Before Register",
-	"description": "Plugin that crashes before calling register()",
-	"version": "1.0.0",
-	"author": "",
-	"homepage_url": "https://joplinapp.org"
-}
-*/
-
-// Crash before calling joplin.plugins.register()
-// This tests the fix for https://github.com/laurent22/joplin/issues/12793
-throw new Error('Simulated plugin crash before register()');

packages/app-desktop/integration-tests/resources/test-plugins/dialogs.js

@@ -48,21 +48,6 @@ joplin.plugins.register({
 			},
 		});
 
-		const dismissDialogHandle = await dialogs.create('test-dialog-with-dismiss');
-		await dialogs.setHtml(dismissDialogHandle, '<p>Press Escape to dismiss</p>');
-		await dialogs.setButtons(dismissDialogHandle, [
-			{ id: 'ok', title: 'Okay' },
-			{ id: 'cancel', title: 'Cancel' },
-		]);
-		await joplin.commands.register({
-			name: 'showTestDialogWithDismiss',
-			label: 'showTestDialogWithDismiss',
-			execute: async () => {
-				const result = await joplin.views.dialogs.open(dismissDialogHandle);
-				await joplin.commands.execute('editor.setText', result.id);
-			},
-		});
-
 		await joplin.commands.register({
 			name: 'getTestDialogVisibility',
 			label: 'Returns the dialog visibility state',

packages/app-desktop/integration-tests/util/createStartupArgs.ts

@@ -7,7 +7,7 @@ const createStartupArgs = (profileDirectory: string) => {
 
 	// We need to run with --env dev to disable the single instance check.
 	return [
-		mainPath, '--env', 'dev', '--lang=en-GB', '--log-level', 'debug', '--no-welcome', '--running-tests', '--profile', resolve(profileDirectory),
+		mainPath, '--env', 'dev', '--log-level', 'debug', '--no-welcome', '--running-tests', '--profile', resolve(profileDirectory),
 	];
 };
 

packages/app-desktop/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@joplin/app-desktop",
-  "version": "3.6.4",
+  "version": "3.6.2",
   "description": "Joplin for Desktop",
   "main": "main.bundle.js",
   "private": true,
@@ -140,7 +140,7 @@
   "homepage": "https://github.com/laurent22/joplin#readme",
   "devDependencies": {
     "7zip-bin": "5.2.0",
-    "@axe-core/playwright": "4.11.0",
+    "@axe-core/playwright": "4.10.2",
     "@electron/notarize": "2.5.0",
     "@electron/rebuild": "3.7.2",
     "@fortawesome/fontawesome-free": "5.15.4",

packages/app-desktop/services/plugins/PluginRunner.ts

@@ -168,16 +168,6 @@ export default class PluginRunner extends BasePluginRunner {
 					promise.resolve(message.result);
 				}
 			} else {
-				// Handle notification that a plugin failed to start before
-				// calling register(). Emit 'started' so allPluginsStarted
-				// is not blocked. See: https://github.com/laurent22/joplin/issues/12793
-				if (message.path === '__pluginFailedToStart__') {
-					logger.error(`Plugin "${plugin.id}" failed to start:`, message.args?.[0]);
-					plugin.running = false;
-					plugin.emit('started');
-					return;
-				}
-
 				const mappedArgs = mapEventIdsToHandlers(plugin.id, message.args);
 				const fullPath = `joplin.${message.path}`;
 

packages/app-desktop/services/plugins/UserWebviewDialog.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import { useRef, useCallback, useMemo } from 'react';
+import { useRef, useCallback } from 'react';
 import { ButtonSpec, DialogResult } from '@joplin/lib/services/plugins/api/types';
 import PluginService from '@joplin/lib/services/plugins/PluginService';
 import WebviewController from '@joplin/lib/services/plugins/WebviewController';
@@ -76,16 +76,8 @@ export default function UserWebviewDialog(props: Props) {
 		if (webviewRef.current) focus('UserWebviewDialog', webviewRef.current);
 	}, []);
 
-	// When the iframe is isolated (security setting enabled), keyboard events
-	// like Escape don't reach the iframe content. We let the native <dialog>
-	// handle Escape by passing onCancel, but only when there's a dismiss button.
-	// https://github.com/laurent22/joplin/issues/13718
-	const onCancel = useMemo(() => {
-		return findDismissButton(buttons) ? onDismiss : undefined;
-	}, [buttons, onDismiss]);
-
 	return (
-		<Dialog className={`user-webview-dialog ${props.fitToContent ? '-fit' : ''}`} onCancel={onCancel}>
+		<Dialog className={`user-webview-dialog ${props.fitToContent ? '-fit' : ''}`}>
 			<div className='user-dialog-wrapper'>
 				<UserWebview
 					ref={webviewRef}

packages/app-desktop/services/plugins/plugin_index.js

@@ -130,38 +130,10 @@
 		console.warn('Unhandled plugin message:', message);
 	});
 
-	// Track whether the plugin has called joplin.plugins.register().
-	// If the plugin script throws before register() is called, the 'started'
-	// event will never fire, which blocks other plugins from working.
-	// See: https://github.com/laurent22/joplin/issues/12793
-	let registerCalled = false;
-
-	const originalTarget = target;
-	const wrappedTarget = (path, args) => {
-		if (path === 'plugins.register') {
-			registerCalled = true;
-		}
-		return originalTarget(path, args);
-	};
-
-	// Catch unhandled errors from the plugin script. If the plugin throws
-	// before calling register(), notify the host so it can unblock startup.
-	window.onerror = (message) => {
-		if (!registerCalled) {
-			console.error(`Plugin "${pluginId}" threw an error before registering:`, message);
-			ipcRendererSend('pluginMessage', {
-				target: 'mainWindow',
-				pluginId: pluginId,
-				path: '__pluginFailedToStart__',
-				args: [String(message)],
-			});
-		}
-	};
-
 	const pluginScriptPath = urlParams.get('pluginScript');
 	const script = document.createElement('script');
 	script.src = pluginScriptPath;
 	document.head.appendChild(script);
 
-	globalObject.joplin = sandboxProxy(wrappedTarget);
+	globalObject.joplin = sandboxProxy(target);
 })(window);

packages/app-desktop/services/sortOrder/PerFolderSortOrderService.test.ts

@@ -1,16 +1,16 @@
 import PerFolderSortOrderService from './PerFolderSortOrderService';
 import { setNotesSortOrder } from './notesSortOrderUtils';
-import Setting from '../../models/Setting';
-import { defaultState, serializeNotesParent, State } from '../../reducer';
-import eventManager from '../../eventManager';
-const { shimInit } = require('../../shim-init-node.js');
-const { ALL_NOTES_FILTER_ID } = require('../../reserved-ids');
+import Setting from '@joplin/lib/models/Setting';
+import { AppState, createAppDefaultState } from '../../app.reducer';
+import eventManager from '@joplin/lib/eventManager';
+const { shimInit } = require('@joplin/lib/shim-init-node.js');
+const { ALL_NOTES_FILTER_ID } = require('@joplin/lib/reserved-ids');
 
 const folderId1 = 'aa012345678901234567890123456789';
 const folderId2 = 'bb012345678901234567890123456789';
 
-let appState: State = null;
-const updateAppState = (update: Partial<State>) => {
+let appState: AppState|null = null;
+const updateAppState = (update: Partial<AppState>) => {
 	appState = { ...appState, ...update };
 	eventManager.appStateEmit(appState);
 };
@@ -39,7 +39,7 @@ describe('PerFolderSortOrderService', () => {
 	beforeEach(() => {
 		PerFolderSortOrderService.initialize();
 		Setting.setValue('notes.perFolderSortOrderEnabled', true);
-		updateAppState(defaultState);
+		updateAppState(createAppDefaultState({}));
 		switchToFolder(folderId1);
 	});
 	afterEach(() => {
@@ -137,31 +137,4 @@ describe('PerFolderSortOrderService', () => {
 		expect(Setting.value('notes.sortOrder.field')).toBe('user_updated_time');
 		expect(Setting.value('notes.sortOrder.reverse')).toBe(true);
 	});
-
-	test('should not let All Notes sort bleed into shared sort order on relaunch', () => {
-		switchToFolder(folderId1);
-		setNotesSortOrder('title', false);
-
-		switchToAllNotes();
-		PerFolderSortOrderService.set(ALL_NOTES_FILTER_ID, true);
-		setNotesSortOrder('user_updated_time', true);
-
-		expect(PerFolderSortOrderService.isSet(ALL_NOTES_FILTER_ID)).toBe(true);
-		expect(Setting.value('notes.sortOrder.field')).toBe('user_updated_time');
-
-		// Simulate relaunch: activeFolderId holds the last notebook, notesParent holds All Notes
-		Setting.setValue('activeFolderId', folderId1);
-		Setting.setValue('notesParent', serializeNotesParent({ type: 'SmartFilter', selectedItemId: ALL_NOTES_FILTER_ID }));
-
-		PerFolderSortOrderService.initialize();
-		Setting.setValue('notes.perFolderSortOrderEnabled', true);
-
-		updateAppState(defaultState);
-		switchToAllNotes();
-
-		// Navigating to the notebook must restore shared sort (title), not All Notes' sort
-		switchToFolder(folderId1);
-		expect(Setting.value('notes.sortOrder.field')).toBe('title');
-		expect(Setting.value('notes.sortOrder.reverse')).toBe(false);
-	});
 });

packages/app-desktop/services/sortOrder/PerFolderSortOrderService.ts

@@ -1,7 +1,6 @@
-import Setting from '../../models/Setting';
-import eventManager from '../../eventManager';
+import Setting from '@joplin/lib/models/Setting';
+import eventManager from '@joplin/lib/eventManager';
 import { notesSortOrderFieldArray, setNotesSortOrder } from './notesSortOrderUtils';
-import { parseNotesParent } from '../../reducer';
 
 const SUFFIX_FIELD = '$field';
 const SUFFIX_REVERSE = '$reverse';
@@ -45,14 +44,7 @@ export default class PerFolderSortOrderService {
 		eventManager.appStateOn('notesParentType', this.onFolderSelectionMayChange.bind(this, 'notesParentType'));
 		eventManager.appStateOn('selectedFolderId', this.onFolderSelectionMayChange.bind(this, 'selectedFolderId'));
 		eventManager.appStateOn('selectedSmartFilterId', this.onFolderSelectionMayChange.bind(this, 'selectedSmartFilterId'));
-		// notesParent persists the last view type (including smart filters like All Notes),
-		// unlike activeFolderId which only tracks folder selections.
-		const notesParent = parseNotesParent(Setting.value('notesParent'), Setting.value('activeFolderId'));
-		if (notesParent.type === 'Folder' || notesParent.type === 'SmartFilter') {
-			this.previousFolderId = notesParent.selectedItemId;
-		} else {
-			this.previousFolderId = null;
-		}
+		this.previousFolderId = Setting.value('activeFolderId');
 	}
 
 	public static isSet(folderId: string): boolean {
@@ -96,40 +88,12 @@ export default class PerFolderSortOrderService {
 					reverse = this.sharedSortOrder.reverse;
 				}
 			}
-			this.setSharedSortOrder(field, reverse);
 			PerFolderSortOrderService.setPerFolderSortOrder(targetId, field, reverse);
 		} else {
 			PerFolderSortOrderService.deletePerFolderSortOrder(targetId);
-			if (targetId === selectedId) {
-				this.loadSharedSortOrder();
-
-				const field = this.sharedSortOrder.field;
-				let reverse: boolean;
-				if (Setting.value('notes.perFieldReversalEnabled')) {
-					reverse = this.sharedSortOrder[field] as boolean;
-				} else {
-					reverse = this.sharedSortOrder.reverse;
-				}
-				setNotesSortOrder(field, reverse);
-			}
 		}
 	}
 
-	public static onSortOrderChange(folderId: string) {
-		const field = Setting.value('notes.sortOrder.field');
-		const reverse = Setting.value('notes.sortOrder.reverse');
-		if (!folderId) {
-			this.setSharedSortOrder(field, reverse);
-			return;
-		}
-		if (this.isSet(folderId)) {
-			// Folder has per-folder sort enabled, update its entry
-			this.setPerFolderSortOrder(folderId, field, reverse);
-		} else {
-			// Folder uses shared sort, update the shared sort
-			this.setSharedSortOrder(field, reverse);
-		}
-	}
 
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Old code before rule was applied
 	private static onFolderSelectionMayChange(cause: string, event: any) {

packages/app-desktop/services/sortOrder/notesSortOrderUtils.test.ts

@@ -1,6 +1,6 @@
 import { notesSortOrderNextField, setNotesSortOrder } from './notesSortOrderUtils';
-import Setting from '../../models/Setting';
-const { shimInit } = require('../../shim-init-node.js');
+import Setting from '@joplin/lib/models/Setting';
+const { shimInit } = require('@joplin/lib/shim-init-node.js');
 
 describe('notesSortOrderUtils', () => {
 

packages/app-desktop/services/sortOrder/notesSortOrderUtils.ts

@@ -1,4 +1,4 @@
-import Setting from '../../models/Setting';
+import Setting from '@joplin/lib/models/Setting';
 
 let fields: string[] = null;
 let perFieldReverse: { [field: string]: boolean } = null;

packages/app-mobile/android/app/build.gradle

@@ -83,8 +83,8 @@ android {
         applicationId "net.cozic.joplin"
         minSdkVersion rootProject.ext.minSdkVersion
         targetSdkVersion rootProject.ext.targetSdkVersion
-		versionCode 2097801
-		versionName "3.6.13"
+		versionCode 2097800
+		versionName "3.6.12"
 
         buildConfigField "String", "REACT_NATIVE_RELEASE_LEVEL", "\"${findProperty('reactNativeReleaseLevel') ?: 'stable'}\""
 

packages/app-mobile/components/DismissibleDialog.tsx

@@ -57,7 +57,6 @@ const useStyles = (themeId: number, containerStyle: ViewStyle, size: DialogVaria
 			},
 			closeButton: {
 				margin: 0,
-				marginRight: -8,
 			},
 			// Ensure that the close button is aligned with the center of the header:
 			// Make its container smaller and center it.
@@ -92,8 +91,8 @@ const useStyles = (themeId: number, containerStyle: ViewStyle, size: DialogVaria
 			dialogSurface: {
 				borderRadius: 24,
 				backgroundColor: theme.backgroundColor,
-				paddingHorizontal: theme.margin,
-				paddingVertical: theme.margin,
+				paddingHorizontal: 16,
+				paddingVertical: 24,
 				...dialogSizing,
 			},
 		});

packages/app-mobile/components/FeedbackBanner.test.tsx

@@ -0,0 +1,134 @@
+import * as React from 'react';
+import { Store } from 'redux';
+import { AppState } from '../utils/types';
+import TestProviderStack from './testing/TestProviderStack';
+import { switchClient, setupDatabase, mockMobilePlatform, mockFetch } from '@joplin/lib/testing/test-utils';
+import waitFor from '@joplin/lib/testing/waitFor';
+import createMockReduxStore from '../utils/testing/createMockReduxStore';
+import setupGlobalStore from '../utils/testing/setupGlobalStore';
+import { act, fireEvent, render, screen } from '@testing-library/react-native';
+import FeedbackBanner from './FeedbackBanner';
+import { MobilePlatform } from '@joplin/lib/shim';
+
+interface WrapperProps { }
+
+let store: Store<AppState>;
+const WrappedFeedbackBanner: React.FC<WrapperProps> = () => {
+	return <TestProviderStack store={store}>
+		<FeedbackBanner/>
+	</TestProviderStack>;
+};
+
+const getFeedbackButton = (positive: boolean) => {
+	return screen.getByRole('button', { name: positive ? 'Useful' : 'Not useful' });
+};
+
+const getSurveyLink = () => {
+	return screen.getByRole('button', { name: 'Take survey' });
+};
+
+const mockFeedbackServer = (surveyName = 'web-app-test') => {
+	let helpfulCount = 0;
+	let unhelpfulCount = 0;
+
+	const { reset } = mockFetch((request) => {
+		const surveyBaseUrls = [
+			'https://objects.joplinusercontent.com/',
+			'http://localhost:3430/',
+		];
+		const isSurveyRequest = surveyBaseUrls.some(url => request.url.startsWith(url));
+		if (!isSurveyRequest) {
+			return null;
+		}
+
+		const url = new URL(request.url);
+
+		if (url.pathname === `/r/survey--${surveyName}--helpful`) {
+			helpfulCount ++;
+		} else if (url.pathname === `/r/survey--${surveyName}--unhelpful`) {
+			unhelpfulCount ++;
+		} else {
+			return new Response('Not found', { status: 404 });
+		}
+
+		// The feedback server always redirects to another URL after a
+		// successful request. Mock this by always redirecting to the
+		// same URL.
+		return new Response('', {
+			// See https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Status/302
+			status: 302,
+			statusText: 'Found',
+			headers: [
+				['location', 'https://joplinapp.org'],
+			],
+		});
+	});
+
+	return {
+		reset,
+		get helpfulCount() {
+			return helpfulCount;
+		},
+		get unhelpfulCount() {
+			return unhelpfulCount;
+		},
+	};
+};
+
+describe('FeedbackBanner', () => {
+	const resetMobilePlatform = ()=>{};
+
+	beforeEach(async () => {
+		await setupDatabase(0);
+		await switchClient(0);
+
+		store = createMockReduxStore();
+		setupGlobalStore(store);
+
+		jest.useFakeTimers({ advanceTimers: true });
+		mockMobilePlatform(MobilePlatform.Web);
+	});
+
+	afterEach(() => {
+		screen.unmount();
+		resetMobilePlatform();
+	});
+
+	test.each([
+		{ platform: MobilePlatform.Android, shouldShow: false },
+		{ platform: MobilePlatform.Web, shouldShow: true },
+		{ platform: MobilePlatform.Ios, shouldShow: false },
+	])('should correctly show/hide the feedback banner on %s', ({ platform, shouldShow }) => {
+		mockMobilePlatform(platform);
+
+		render(<WrappedFeedbackBanner />);
+
+		const header = screen.queryByRole('header', { name: 'Feedback' });
+		if (shouldShow) {
+			expect(header).toBeVisible();
+		} else {
+			expect(header).toBeNull();
+		}
+	});
+
+	test('clicking the "Useful" button should submit the response and show the "take survey" link', async () => {
+		const feedbackServerMock = mockFeedbackServer();
+		render(<WrappedFeedbackBanner />);
+
+		try {
+			const usefulButton = getFeedbackButton(true);
+			fireEvent.press(usefulButton);
+
+			await act(() => waitFor(async () => {
+				expect(getSurveyLink()).toBeVisible();
+			}));
+
+			expect(feedbackServerMock).toMatchObject({
+				helpfulCount: 1,
+				unhelpfulCount: 0,
+			});
+		} finally {
+			feedbackServerMock.reset();
+		}
+	});
+});

packages/app-mobile/components/FeedbackBanner.tsx

@@ -0,0 +1,216 @@
+import { _ } from '@joplin/lib/locale';
+import * as React from 'react';
+import { View, StyleSheet, useWindowDimensions, TextStyle, Linking } from 'react-native';
+import { Portal, Text } from 'react-native-paper';
+import IconButton from './IconButton';
+import { useCallback, useMemo } from 'react';
+import shim from '@joplin/lib/shim';
+import { Dispatch } from 'redux';
+import { themeStyle } from './global-style';
+import { AppState } from '../utils/types';
+import { connect } from 'react-redux';
+import Setting from '@joplin/lib/models/Setting';
+import { LinkButton } from './buttons';
+import Logger from '@joplin/utils/Logger';
+import { SurveyProgress } from '@joplin/lib/models/settings/builtInMetadata';
+
+const logger = Logger.create('FeedbackBanner');
+
+interface Props {
+	dispatch: Dispatch;
+	progress: SurveyProgress;
+	surveyKey: string;
+	themeId: number;
+}
+
+const useStyles = (themeId: number, sentFeedback: boolean) => {
+	const { width: windowWidth } = useWindowDimensions();
+	return useMemo(() => {
+		const theme = themeStyle(themeId);
+
+		const iconBaseStyle: TextStyle = {
+			fontSize: 24,
+			color: theme.color3,
+		};
+
+		return StyleSheet.create({
+			container: {
+				backgroundColor: theme.backgroundColor3,
+				borderTopRightRadius: 16,
+				display: 'flex',
+				flexGrow: 1,
+				flexWrap: 'wrap',
+				flexDirection: 'row',
+				position: 'absolute',
+				bottom: 0,
+				left: 0,
+				maxWidth: windowWidth - 50,
+				gap: 18,
+				padding: 12,
+			},
+			contentRight: {
+				display: sentFeedback ? 'none' : 'flex',
+				flexDirection: 'row',
+				alignItems: 'center',
+				gap: 16,
+			},
+			header: {
+				fontWeight: 'bold',
+			},
+			iconUseful: {
+				...iconBaseStyle,
+				color: theme.colorCorrect,
+			},
+			iconNotUseful: {
+				...iconBaseStyle,
+				color: theme.colorWarn,
+			},
+			dismissButtonIcon: {
+				fontSize: 16,
+				color: theme.color2,
+				marginLeft: 'auto',
+				marginRight: 'auto',
+			},
+			dismissButton: {
+				backgroundColor: theme.backgroundColor2,
+				borderColor: theme.backgroundColor,
+				borderWidth: 2,
+				width: 29,
+				height: 29,
+				borderRadius: 14,
+				position: 'absolute',
+				top: -16,
+				right: -16,
+				justifyContent: 'center',
+			},
+			dismissButtonContent: {
+				flexShrink: 1,
+			},
+		});
+	}, [themeId, windowWidth, sentFeedback]);
+};
+
+const useSurveyUrl = (surveyKey: string) => {
+	return useMemo(() => {
+		let baseUrl = 'https://objects.joplinusercontent.com/';
+
+		// For testing with a locally-hosted server:
+		const useLocalServer = false;
+		if (Setting.value('env') === 'dev' && useLocalServer) {
+			baseUrl = 'http://localhost:3430/';
+		}
+
+		return `${baseUrl}r/survey--${encodeURIComponent(surveyKey)}`;
+	}, [surveyKey]);
+};
+
+const setProgress = (progress: SurveyProgress) => {
+	Setting.setValue('survey.webClientEval2025.progress', progress);
+};
+
+const onDismiss = () => {
+	setProgress(SurveyProgress.Dismissed);
+};
+
+const FeedbackBanner: React.FC<Props> = props => {
+	const surveyUrl = useSurveyUrl(props.surveyKey);
+	const sentFeedback = props.progress === SurveyProgress.Started;
+
+	const sendSurveyResponse = useCallback(async (surveyResponse: string) => {
+		const fetchUrl = `${surveyUrl}--${encodeURIComponent(surveyResponse)}`;
+		logger.debug('sending response to', fetchUrl);
+		const showError = (message: string) => {
+			logger.error('Error', message);
+			void shim.showErrorDialog(
+				_('An error occurred while sending the response. This can happen if the app is offline or cannot connect to the server.\nError: %s', message),
+			);
+		};
+
+		try {
+			const response = await shim.fetch(fetchUrl);
+			// The server currently redirects (status 302) in response
+			// to many survey-related requests. This may be returned by
+			// the web app service worker as a 200 OK response, however. Support both:
+			if (response.ok || response.status === 302) {
+				setProgress(SurveyProgress.Started);
+			} else {
+				const body = await response.text();
+				showError(`Server error: ${response.status} ${body}`);
+			}
+		} catch (error) {
+			showError(error);
+		}
+	}, [surveyUrl]);
+
+	const onSurveyLinkClick = useCallback(() => {
+		void Linking.openURL(surveyUrl);
+		onDismiss();
+	}, [surveyUrl]);
+
+	const onNotUsefulClick = useCallback(() => {
+		void sendSurveyResponse('unhelpful');
+	}, [sendSurveyResponse]);
+
+	const onUsefulClick = useCallback(() => {
+		void sendSurveyResponse('helpful');
+	}, [sendSurveyResponse]);
+
+	const styles = useStyles(props.themeId, sentFeedback);
+
+	const renderStatusMessage = () => {
+		if (sentFeedback) {
+			return <View>
+				<Text>{_('Thank you for the feedback!\nDo you have time to complete a short survey?')}</Text>
+				<LinkButton onPress={onSurveyLinkClick}>{_('Take survey')}</LinkButton>
+			</View>;
+		} else {
+			return <Text>{_('Do you find the Joplin web app useful?')}</Text>;
+		}
+	};
+
+	if (shim.mobilePlatform() !== 'web' || props.progress === SurveyProgress.Dismissed) return null;
+
+	return <Portal>
+		<View style={styles.container} role='complementary'>
+			<View>
+				<Text
+					accessibilityRole='header'
+					variant='titleMedium'
+					style={styles.header}
+				>{_('Feedback')}</Text>
+				<Text>{renderStatusMessage()}</Text>
+			</View>
+			<View style={styles.contentRight}>
+				<IconButton
+					iconName='fas times'
+					themeId={props.themeId}
+					onPress={onNotUsefulClick}
+					description={_('Not useful')}
+					iconStyle={styles.iconNotUseful}
+				/>
+				<IconButton
+					iconName='fas check'
+					themeId={props.themeId}
+					onPress={onUsefulClick}
+					description={_('Useful')}
+					iconStyle={styles.iconUseful}
+				/>
+			</View>
+			<IconButton
+				iconName='fas times'
+				themeId={props.themeId}
+				onPress={onDismiss}
+				description={_('Dismiss')}
+				iconStyle={styles.dismissButtonIcon}
+				contentWrapperStyle={styles.dismissButtonContent}
+				containerStyle={styles.dismissButton}
+			/>
+		</View>
+	</Portal>;
+};
+
+export default connect((state: AppState) => ({
+	themeId: state.settings.theme,
+	surveyKey: 'web-app-test',
+	progress: state.settings['survey.webClientEval2025.progress'],
+}))(FeedbackBanner);

packages/app-mobile/components/NoteBodyViewer/NoteBodyViewer.tsx

@@ -34,7 +34,6 @@ interface Props {
 	onScroll: OnScrollCallback;
 	onLoadEnd?: ()=> void;
 	pluginStates: PluginStates;
-	showNoteLinkIcon: boolean;
 }
 
 const onJoplinLinkClick = async (message: string) => {
@@ -85,7 +84,6 @@ function NoteBodyViewer(props: Props) {
 		initialScrollPercent: props.initialScrollPercent,
 
 		paddingBottom: props.paddingBottom,
-		showNoteLinkIcon: props.showNoteLinkIcon,
 	});
 
 	const onLoadEnd = useCallback(() => {
@@ -117,5 +115,4 @@ export default connect((state: AppState) => ({
 	themeId: state.settings.theme,
 	fontSize: state.settings['style.viewer.fontSize'],
 	pluginStates: state.pluginService.plugins,
-	showNoteLinkIcon: state.settings['notes.showNoteLinkIcon'],
 }))(NoteBodyViewer);

packages/app-mobile/components/NoteBodyViewer/hooks/useRerenderHandler.ts

@@ -27,7 +27,6 @@ interface Props {
 	initialScrollPercent: number|undefined;
 
 	paddingBottom: number;
-	showNoteLinkIcon: boolean;
 }
 
 const onlyCheckboxHasChangedHack = (previousBody: string, newBody: string) => {
@@ -101,7 +100,7 @@ const useRerenderHandler = (props: Props) => {
 	const effectDependencies = [
 		props.noteBody, props.noteMarkupLanguage, props.renderer, props.highlightedKeywords,
 		props.noteHash, props.noteResources, props.themeId, props.paddingBottom, resourceDownloadRerenderCounter,
-		props.fontSize, props.showNoteLinkIcon,
+		props.fontSize,
 	];
 	const previousDeps = usePrevious(effectDependencies, []);
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Old code before rule was applied
@@ -139,7 +138,6 @@ const useRerenderHandler = (props: Props) => {
 			// instead.
 			initialScrollPercent: (previousHash && hashChanged) ? undefined : props.initialScrollPercent,
 			noteHash: props.noteHash,
-			showNoteLinkIcon: props.showNoteLinkIcon,
 		};
 
 		try {

packages/app-mobile/components/NoteEditor/NoteEditor.test.tsx

@@ -16,7 +16,7 @@ import setupGlobalStore from '../../utils/testing/setupGlobalStore';
 import { Store } from 'redux';
 import { AppState } from '../../utils/types';
 import { MarkupLanguage } from '@joplin/renderer';
-import { EditorControl, EditorType } from './types';
+import { EditorType } from './types';
 
 let store: Store<AppState>;
 let registeredRuntime: RegisteredRuntime;
@@ -65,27 +65,6 @@ describe('NoteEditor', () => {
 		registeredRuntime.deregister();
 	});
 
-	it('should provide an editor ref', () => {
-		let editorRef: EditorControl;
-		const onSetEditorRef = (ref: EditorControl) => {
-			editorRef = ref;
-		};
-
-		const wrappedNoteEditor = render(
-			<TestProviderStack store={store}>
-				<NoteEditor
-					ref={onSetEditorRef}
-					{...defaultEditorProps}
-					mode={EditorType.RichText}
-				/>
-			</TestProviderStack>,
-		);
-
-		expect(editorRef).toBeTruthy();
-
-		wrappedNoteEditor.unmount();
-	});
-
 	it('should hide the markdown toolbar when the window is small', async () => {
 		const wrappedNoteEditor = render(
 			<TestProviderStack store={store}>

packages/app-mobile/components/NoteEditor/NoteEditor.tsx

@@ -34,8 +34,6 @@ import { MarkupLanguage } from '@joplin/renderer';
 import WarningBanner from './WarningBanner';
 import useIsScreenReaderEnabled from '../../utils/hooks/useIsScreenReaderEnabled';
 import Logger from '@joplin/utils/Logger';
-import { AppState } from '../../utils/types';
-import { connect } from 'react-redux';
 import { Second } from '@joplin/utils/time';
 import useDebounced from '../../utils/hooks/useDebounced';
 
@@ -64,8 +62,6 @@ interface Props {
 	readOnly: boolean;
 	plugins: PluginStates;
 	noteResources: ResourceInfos;
-	editorImageRendering: boolean;
-	editorInlineRendering: boolean;
 
 	onScroll: OnScroll;
 	onChange: OnChange;
@@ -73,7 +69,6 @@ interface Props {
 	onSelectionChange: OnSelectionChange;
 	onUndoRedoDepthChange: OnUndoRedoDepthChange;
 	onAttach: OnAttach;
-	refreshKey?: number;
 }
 
 function fontFamilyFromSettings() {
@@ -261,49 +256,12 @@ const useEditorControl = (
 	}, [webviewRef, editorRef, setLinkDialogVisible, setSearchState]);
 };
 
-const useEditorSettings = (props: Props) => {
+const useHighlightActiveLine = () => {
 	const screenReaderEnabled = useIsScreenReaderEnabled();
 	// Guess whether highlighting the active line can be enabled without triggering
 	// https://github.com/codemirror/dev/issues/1559.
 	const canHighlight = Platform.OS !== 'ios' || !screenReaderEnabled;
-	const highlightActiveLine = canHighlight && Setting.value('editor.highlightActiveLine');
-
-	// Also disable inline rendering. As of January 2026, inline rendering
-	// seems to cause screen readers to behave strangely (e.g. sometimes not announce full
-	// line content, reading "image" when not in an image, etc.)
-	// However, `screenReaderEnabled` is always `true` on web (likely due to the lack of an API
-	// to reliably detect whether the user is using a screen reader), so also allow inline rendering
-	// to be enabled on web:
-	const inlineRenderingEnabled = props.editorInlineRendering && (!screenReaderEnabled || Platform.OS === 'web');
-
-	const editorSettings: EditorSettings = useMemo(() => ({
-		themeData: editorTheme(props.themeId),
-		markdownMarkEnabled: Setting.value('markdown.plugin.mark'),
-		markdownInsertEnabled: Setting.value('markdown.plugin.insert'),
-		katexEnabled: Setting.value('markdown.plugin.katex'),
-		spellcheckEnabled: Setting.value('editor.mobile.spellcheckEnabled'),
-		inlineRenderingEnabled,
-		imageRenderingEnabled: props.editorImageRendering,
-		language: props.markupLanguage === MarkupLanguage.Html ? EditorLanguageType.Html : EditorLanguageType.Markdown,
-		useExternalSearch: true,
-		readOnly: props.readOnly,
-		highlightActiveLine,
-
-		keymap: EditorKeymap.Default,
-		preferMacShortcuts: shim.mobilePlatform() === 'ios',
-
-		automatchBraces: false,
-		ignoreModifiers: false,
-		autocompleteMarkup: Setting.value('editor.autocompleteMarkup'),
-
-		// For now, mobile CodeMirror uses its built-in focus toggle shortcut.
-		tabMovesFocus: false,
-		indentWithTabs: true,
-
-		editorLabel: _('Markdown editor'),
-	}), [props.themeId, props.readOnly, props.markupLanguage, highlightActiveLine, inlineRenderingEnabled, props.editorImageRendering]);
-
-	return editorSettings;
+	return canHighlight && Setting.value('editor.highlightActiveLine');
 };
 
 const useHasSpaceForToolbar = () => {
@@ -322,7 +280,32 @@ const useHasSpaceForToolbar = () => {
 function NoteEditor(props: Props) {
 	const webviewRef = useRef<WebViewControl>(null);
 
-	const editorSettings = useEditorSettings(props);
+	const highlightActiveLine = useHighlightActiveLine();
+	const editorSettings: EditorSettings = useMemo(() => ({
+		themeData: editorTheme(props.themeId),
+		markdownMarkEnabled: Setting.value('markdown.plugin.mark'),
+		katexEnabled: Setting.value('markdown.plugin.katex'),
+		spellcheckEnabled: Setting.value('editor.mobile.spellcheckEnabled'),
+		inlineRenderingEnabled: Setting.value('editor.inlineRendering'),
+		imageRenderingEnabled: Setting.value('editor.imageRendering'),
+		language: props.markupLanguage === MarkupLanguage.Html ? EditorLanguageType.Html : EditorLanguageType.Markdown,
+		useExternalSearch: true,
+		readOnly: props.readOnly,
+		highlightActiveLine,
+
+		keymap: EditorKeymap.Default,
+		preferMacShortcuts: shim.mobilePlatform() === 'ios',
+
+		automatchBraces: false,
+		ignoreModifiers: false,
+		autocompleteMarkup: Setting.value('editor.autocompleteMarkup'),
+
+		// For now, mobile CodeMirror uses its built-in focus toggle shortcut.
+		tabMovesFocus: false,
+		indentWithTabs: true,
+
+		editorLabel: _('Markdown editor'),
+	}), [props.themeId, props.readOnly, props.markupLanguage, highlightActiveLine]);
 
 	const [selectionState, setSelectionState] = useState<SelectionFormatting>(defaultSelectionFormatting);
 	const [linkDialogVisible, setLinkDialogVisible] = useState(false);
@@ -473,7 +456,6 @@ function NoteEditor(props: Props) {
 				minHeight: '30%',
 			}}>
 				<EditorComponent
-					key={props.refreshKey}
 					editorRef={editorRef}
 					webviewRef={webviewRef}
 					themeId={props.themeId}
@@ -491,11 +473,7 @@ function NoteEditor(props: Props) {
 				/>
 			</View>
 
-			<WarningBanner
-				editorType={props.mode}
-				markupLanguage={props.markupLanguage}
-				inEditorRendering={editorSettings.inlineRenderingEnabled}
-			/>
+			<WarningBanner editorType={props.mode}/>
 
 			<SearchPanel
 				editorSettings={editorSettings}
@@ -508,10 +486,4 @@ function NoteEditor(props: Props) {
 	);
 }
 
-export default connect((state: AppState) => {
-	return {
-		themeId: state.settings.theme,
-		editorInlineRendering: state.settings['editor.inlineRendering'],
-		editorImageRendering: state.settings['editor.imageRendering'],
-	};
-}, null, null, { forwardRef: true })(NoteEditor);
+export default NoteEditor;

packages/app-mobile/components/NoteEditor/RichTextEditor.test.tsx

@@ -460,24 +460,6 @@ describe('RichTextEditor', () => {
 		});
 	});
 
-	it.each(['-', '1.'])('should not add extra blank lines around nested lists (marker: %j)', async (marker) => {
-		const nested = marker === '1.' ? '1.' : '-';
-		let body = `${marker} a\n${marker} b\n    ${nested} c\n    ${nested} d\n${marker} e`;
-
-		render(<WrappedEditor
-			noteBody={body}
-			onBodyChange={newBody => { body = newBody; }}
-		/>);
-
-		const window = await getEditorWindow();
-		mockTyping(window, ' testing');
-
-		await waitFor(async () => {
-			// Nested lists should not have extra blank lines above or below
-			expect(body).not.toMatch(/\n\n\s*[-\d]/);
-		});
-	});
-
 	it('should preserve table of contents blocks on edit', async () => {
 		let body = '# Heading\n\n# Heading 2\n\n[toc]\n\nTest.';
 

packages/app-mobile/components/NoteEditor/WarningBanner.tsx

@@ -7,76 +7,35 @@ import { AppState } from '../../utils/types';
 import { EditorType } from './types';
 import { Banner } from 'react-native-paper';
 import { useMemo } from 'react';
-import useEditorTypeMigrationBanner from '@joplin/lib/components/shared/NoteEditor/WarningBanner/useEditorTypeMigrationBanner';
-import { MarkupLanguage } from '@joplin/renderer/types';
 
 interface Props {
 	editorType: EditorType;
 	richTextBannerDismissed: boolean;
-	editorMigrationVersion: number;
-	inEditorRendering: boolean;
-
-	markupLanguage: MarkupLanguage;
 }
 
-const useBanner = ({
-	editorType,
-	richTextBannerDismissed,
-	editorMigrationVersion,
-	inEditorRendering,
-}: Props) => {
-	const editorMigrationBanner = useEditorTypeMigrationBanner({
-		markdownEditorEnabled: editorType === EditorType.Markdown,
-		editorMigrationVersion: editorMigrationVersion,
-		inEditorRenderingEnabled: inEditorRendering,
-	});
-
-	return useMemo(() => {
-		if (editorType === EditorType.RichText && !richTextBannerDismissed) {
-			return {
-				label: _('This Rich Text editor has a number of limitations and it is recommended to be aware of them before using it.'),
-				warning: true,
-				actions: [
-					{
-						label: _('Read more'),
-						onPress: onRichTextReadMoreLinkClick,
-					},
-					{
-						label: _('Dismiss'),
-						accessibilityHint: _('Hides warning'),
-						onPress: onRichTextDismissLinkClick,
-					},
-				],
-			};
-		}
-
-		if (editorMigrationBanner.enabled) {
-			return {
-				label: editorMigrationBanner.label,
-				actions: [
-					editorMigrationBanner.keepEnabled,
-					editorMigrationBanner.disable,
-				],
-			};
-		}
-
-		return null;
-	}, [editorType, richTextBannerDismissed, editorMigrationBanner]);
-};
-
 const WarningBanner: React.FC<Props> = props => {
-	const banner = useBanner(props);
+	const actions = useMemo(() => [
+		{
+			label: _('Read more'),
+			onPress: onRichTextReadMoreLinkClick,
+		},
+		{
+			label: _('Dismiss'),
+			accessibilityHint: _('Hides warning'),
+			onPress: onRichTextDismissLinkClick,
+		},
+	], []);
 
-	if (!banner) return null;
+	if (props.editorType !== EditorType.RichText || props.richTextBannerDismissed) return null;
 	return (
 		<Banner
-			icon={banner.warning ? 'alert-outline' : 'information-outline'}
-			actions={banner.actions}
+			icon='alert-outline'
+			actions={actions}
 			// Avoid hiding with react-native-paper's "visible" prop to avoid potential accessibility issues
 			// related to how react-native-paper hides the banner.
 			visible={true}
 		>
-			{banner.label}
+			{_('This Rich Text editor has a number of limitations and it is recommended to be aware of them before using it.')}
 		</Banner>
 	);
 };
@@ -84,6 +43,5 @@ const WarningBanner: React.FC<Props> = props => {
 export default connect((state: AppState) => {
 	return {
 		richTextBannerDismissed: state.settings.richTextBannerDismissed,
-		editorMigrationVersion: state.settings['editor.migration'],
 	};
 })(WarningBanner);

packages/app-mobile/components/ScreenHeader/WebBetaButton.tsx

@@ -0,0 +1,89 @@
+import * as React from 'react';
+import { Linking, StyleSheet, TextStyle, View, ViewStyle } from 'react-native';
+import { Text } from 'react-native-paper';
+import IconButton from '../IconButton';
+import { _ } from '@joplin/lib/locale';
+import { useCallback, useState } from 'react';
+import DismissibleDialog, { DialogVariant } from '../DismissibleDialog';
+import { LinkButton, PrimaryButton } from '../buttons';
+import makeDiscourseDebugUrl from '@joplin/lib/makeDiscourseDebugUrl';
+import getPackageInfo from '../../utils/getPackageInfo';
+import PluginService from '@joplin/lib/services/plugins/PluginService';
+import Setting from '@joplin/lib/models/Setting';
+
+interface Props {
+	wrapperStyle: ViewStyle;
+	iconStyle: TextStyle;
+	themeId: number;
+}
+
+const onLeaveFeedback = () => {
+	void Linking.openURL('https://forms.gle/B5YGDNzsUYBnoPx19');
+};
+
+const onReportBug = () => {
+	void Linking.openURL(
+		makeDiscourseDebugUrl('', '', [], getPackageInfo(), PluginService.instance(), Setting.value('plugins.states')),
+	);
+};
+
+const styles = StyleSheet.create({
+	feedbackContainer: {
+		flexGrow: 1,
+		flexDirection: 'row',
+		gap: 16,
+		justifyContent: 'flex-end',
+		flexWrap: 'wrap',
+	},
+	paragraph: {
+		paddingBottom: 7,
+	},
+});
+
+const WebBetaButton: React.FC<Props> = props => {
+	const [dialogVisible, setDialogVisible] = useState(false);
+
+	const onShowDialog = useCallback(() => {
+		setDialogVisible(true);
+	}, []);
+
+	const onHideDialog = useCallback(() => {
+		setDialogVisible(false);
+	}, []);
+
+	const renderParagraph = (content: string) => {
+		return <Text variant='bodyLarge' style={styles.paragraph}>{content}</Text>;
+	};
+
+	return (
+		<>
+			<IconButton
+				onPress={onShowDialog}
+				description={_('Beta')}
+				themeId={props.themeId}
+				contentWrapperStyle={props.wrapperStyle}
+
+				iconName="material beta"
+				iconStyle={props.iconStyle}
+			/>
+			<DismissibleDialog
+				heading={_('Beta')}
+				size={DialogVariant.SmallResize}
+				themeId={props.themeId}
+				visible={dialogVisible}
+				onDismiss={onHideDialog}
+			>
+				{renderParagraph('Welcome to the beta version of the Joplin Web App!')}
+				{renderParagraph('Thank you for participating in the beta version of the Joplin Web App.')}
+				{renderParagraph('The Joplin Web App is available for a limited time in open beta and may later join the Joplin Cloud plans.')}
+				{renderParagraph('Feel free to use it and let us know if have any questions or notice any issues!')}
+				<View style={styles.feedbackContainer}>
+					<LinkButton onPress={onReportBug}>{'Report bug'}</LinkButton>
+					<PrimaryButton onPress={onLeaveFeedback}>{'Give feedback'}</PrimaryButton>
+				</View>
+			</DismissibleDialog>
+		</>
+	);
+};
+
+export default WebBetaButton;

packages/app-mobile/components/ScreenHeader/index.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react';
 import { PureComponent, ReactElement } from 'react';
 import { connect } from 'react-redux';
-import { View, Text, StyleSheet, TouchableOpacity, ViewStyle } from 'react-native';
+import { View, Text, StyleSheet, TouchableOpacity, ViewStyle, Platform } from 'react-native';
 import BackButtonService from '../../services/BackButtonService';
 import NavService from '@joplin/lib/services/NavService';
 import { _, _n } from '@joplin/lib/locale';
@@ -20,6 +20,7 @@ import { PluginStates } from '@joplin/lib/services/plugins/reducer';
 import { ContainerType } from '@joplin/lib/services/plugins/WebviewController';
 import { Dispatch } from 'redux';
 import WarningBanner from './WarningBanner';
+import WebBetaButton from './WebBetaButton';
 
 import Menu, { MenuOptionType } from './Menu';
 import shim from '@joplin/lib/shim';
@@ -70,9 +71,6 @@ interface ScreenHeaderProps {
 	showContextMenuButton?: boolean;
 	showPluginEditorButton?: boolean;
 	showBackButton?: boolean;
-	showViewToggleButton?: boolean;
-	onViewTogglePress?: OnPressCallback;
-	viewToggleIconName?: string;
 
 	saveButtonDisabled?: boolean;
 	showSaveButton?: boolean;
@@ -170,7 +168,7 @@ class ScreenHeaderComponent extends PureComponent<ScreenHeaderProps, ScreenHeade
 			},
 			contextMenuTrigger: {
 				fontSize: 30,
-				paddingLeft: 5,
+				paddingLeft: 10,
 				paddingRight: theme.marginRight,
 				color: theme.color2,
 				fontWeight: 'bold',
@@ -376,16 +374,6 @@ class ScreenHeaderComponent extends PureComponent<ScreenHeaderProps, ScreenHeade
 			});
 		};
 
-		const renderViewToggleButton = () => {
-			if (!this.props.showViewToggleButton || !this.props.onViewTogglePress || !this.props.viewToggleIconName) return null;
-			return renderTopButton({
-				iconName: this.props.viewToggleIconName,
-				description: _('Toggle view/edit'),
-				onPress: this.props.onViewTogglePress,
-				visible: true,
-			});
-		};
-
 		// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Old code before rule was applied
 		function selectAllButton(styles: any, onPress: OnPressCallback) {
 			return (
@@ -455,6 +443,18 @@ class ScreenHeaderComponent extends PureComponent<ScreenHeaderProps, ScreenHeade
 			);
 		};
 
+		const betaIconButton = () => {
+			if (Platform.OS !== 'web') return null;
+
+			return (
+				<WebBetaButton
+					themeId={themeId}
+					wrapperStyle={this.styles().iconButton}
+					iconStyle={this.styles().topIcon}
+				/>
+			);
+		};
+
 		// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Old code before rule was applied
 		const renderTogglePluginEditorButton = (styles: any, onPress: OnPressCallback, disabled: boolean) => {
 			if (!this.props.showPluginEditorButton) return null;
@@ -653,6 +653,7 @@ class ScreenHeaderComponent extends PureComponent<ScreenHeaderProps, ScreenHeade
 		const sideMenuComp = !showSideMenuButton ? null : sideMenuButton(this.styles(), () => this.sideMenuButton_press());
 		const backButtonComp = !showBackButton ? null : backButton(this.styles(), () => this.backButton_press(), backButtonDisabled);
 		const pluginPanelsComp = pluginPanelToggleButton(this.styles(), () => this.pluginPanelToggleButton_press());
+		const betaIconComp = betaIconButton();
 		const selectAllButtonComp = !showSelectAllButton ? null : selectAllButton(this.styles(), () => this.selectAllButton_press());
 		const searchButtonComp = !showSearchButton ? null : searchButton(this.styles(), () => this.searchButton_press());
 		const customDeleteButtonComp = this.props.onDeleteButtonPress ? customDeleteButton(this.styles(), this.props.onDeleteButtonPress) : null;
@@ -666,12 +667,12 @@ class ScreenHeaderComponent extends PureComponent<ScreenHeaderProps, ScreenHeade
 		// space while in use, we allow certain buttons to be hidden.
 		const hideableRightComponents = <>
 			{pluginPanelsComp}
+			{betaIconComp}
 			{togglePluginEditorButton}
 			{selectAllButtonComp}
 			{searchButtonComp}
 			{deleteButtonComp}
 			{customDeleteButtonComp}
-			{renderViewToggleButton()}
 		</>;
 
 		const titleComp = createTitleComponent(hideableRightComponents);

packages/app-mobile/components/SyncWizard/SyncWizard.tsx

@@ -8,10 +8,9 @@ import { Icon, Text } from 'react-native-paper';
 import { _ } from '@joplin/lib/locale';
 import JoplinCloudIcon from './JoplinCloudIcon';
 import NavService from '@joplin/lib/services/NavService';
-import { Platform, StyleSheet, View } from 'react-native';
+import { StyleSheet, View } from 'react-native';
 import CardButton from '../buttons/CardButton';
 import Setting from '@joplin/lib/models/Setting';
-import shim from '@joplin/lib/shim';
 
 interface Props {
 	dispatch: Dispatch;
@@ -48,16 +47,6 @@ const styles = StyleSheet.create({
 	},
 });
 
-const isAppJoplinCloud = () => {
-	return Platform.OS === 'web' && location.origin === 'https://app.joplincloud.com';
-};
-
-const useShouldShowOtherButton = () => {
-	// Don't show "other" when hosted on Joplin Cloud (other sync
-	// targets can still be selected from settings).
-	return !isAppJoplinCloud();
-};
-
 interface SyncProviderProps {
 	title: string;
 	icon: ()=> React.ReactNode;
@@ -105,15 +94,7 @@ const SyncWizard: React.FC<Props> = ({ themeId, visible, dispatch }) => {
 
 	const onSelectJoplinCloud = useCallback(async () => {
 		onDismiss();
-		if (Platform.OS === 'web' && !isAppJoplinCloud()) {
-			if (await shim.showConfirmationDialog(
-				_('Self-hosted instances of the Joplin web app cannot sync with Joplin Cloud. Open the official web app?'),
-			)) {
-				await shim.openUrl('https://app.joplincloud.com/');
-			}
-		} else {
-			await NavService.go('JoplinCloudLogin');
-		}
+		await NavService.go('JoplinCloudLogin');
 	}, [onDismiss]);
 
 	const onSelectOtherTarget = useCallback(async () => {
@@ -121,8 +102,6 @@ const SyncWizard: React.FC<Props> = ({ themeId, visible, dispatch }) => {
 		await NavService.go('Config', { sectionName: 'sync' });
 	}, [onDismiss]);
 
-	const showOther = useShouldShowOtherButton();
-
 	return <DismissibleDialog
 		themeId={themeId}
 		visible={visible}
@@ -147,14 +126,14 @@ const SyncWizard: React.FC<Props> = ({ themeId, visible, dispatch }) => {
 				onPress={onSelectJoplinCloud}
 				disabled={false}
 			/>
-			{showOther && <SyncProvider
+			<SyncProvider
 				title={_('Other')}
 				description={_('Select one of the other supported sync targets.')}
 				icon={() => <Icon size={iconSize} source='dots-horizontal-circle'/>}
 				featuresList={[]}
 				onPress={onSelectOtherTarget}
 				disabled={false}
-			/>}
+			/>
 		</View>
 	</DismissibleDialog>;
 };

packages/app-mobile/components/app-nav.tsx

@@ -8,6 +8,7 @@ import { themeStyle } from './global-style';
 import { useSafeAreaInsets } from 'react-native-safe-area-context';
 import useKeyboardState from '../utils/hooks/useKeyboardState';
 import usePrevious from '@joplin/lib/hooks/usePrevious';
+import FeedbackBanner from './FeedbackBanner';
 import { Theme } from '@joplin/lib/themes/type';
 import { useMemo } from 'react';
 import KeyboardAvoidingView from './KeyboardAvoidingView';
@@ -77,6 +78,7 @@ const AppNavComponent: React.FC<Props> = (props) => {
 			<NotesScreen visible={notesScreenVisible} />
 			{searchScreenLoaded && <SearchScreen visible={searchScreenVisible} />}
 			{!notesScreenVisible && !searchScreenVisible && <Screen navigation={{ state: route }} themeId={props.themeId} dispatch={props.dispatch} />}
+			{notesScreenVisible ? <FeedbackBanner/> : null}
 			<View style={{ height: autocompletionBarPadding }} />
 		</KeyboardAvoidingView>
 	);

packages/app-mobile/components/screens/ConfigScreen/plugins/PluginInfoModal.tsx

@@ -39,6 +39,8 @@ const styles = (() => {
 		display: 'flex',
 		flexDirection: 'column',
 		gap: 20,
+		marginLeft: 10,
+		marginRight: 10,
 	};
 	return StyleSheet.create({
 		descriptionText: {
@@ -62,7 +64,7 @@ const styles = (() => {
 			flexDirection: 'row',
 			justifyContent: 'space-between',
 			alignItems: 'center',
-			paddingVertical: 10,
+			padding: 10,
 			marginTop: 12,
 			marginBottom: 14,
 		},
@@ -121,7 +123,7 @@ const PluginInfoModalContent: React.FC<Props> = props => {
 	});
 
 	const aboutPlugin = (
-		<Card mode='outlined' style={{ marginVertical: 8 }} testID='plugin-card'>
+		<Card mode='outlined' style={{ margin: 8 }} testID='plugin-card'>
 			<Card.Content>
 				<PluginTitle manifest={manifest}/>
 				<Text variant='bodyMedium'>{_('by %s', manifest.author)}</Text>

packages/app-mobile/components/screens/Note/Note.test.tsx

@@ -126,17 +126,20 @@ const openNoteActionsMenu = async () => {
 };
 
 const expectToBeEditing = async (editing: boolean) => {
-	if (editing) {
-		await getMarkdownEditorControl();
-	} else {
-		await getNoteViewerDom();
-	}
+	await waitFor(() => {
+		const editButton = screen.queryByLabelText('Edit');
+		if (editing) {
+			expect(editButton).toBeNull();
+		} else {
+			expect(editButton).not.toBeNull();
+		}
+	});
 };
 
 const openEditor = async () => {
-	const editToggle = await screen.findByLabelText('Toggle view/edit');
+	const editButton = await screen.findByLabelText('Edit');
 
-	fireEvent.press(editToggle);
+	fireEvent.press(editButton);
 	await expectToBeEditing(true);
 };
 
@@ -146,18 +149,6 @@ const runEditorCommand = async (commandName: string) => {
 	});
 };
 
-const setupNoteWithPanes = async (panes: string[], noteTitle = 'Test note') => {
-	store.dispatch({
-		type: 'NOTE_VISIBLE_PANES_SET',
-		panes: panes,
-	});
-	await openNewNote({ title: noteTitle, body: 'Test body' });
-	const renderResult = render(<WrappedNoteScreen />);
-	const titleInput = await screen.findByDisplayValue(noteTitle);
-	expect(titleInput).toBeVisible();
-	return renderResult;
-};
-
 describe('screens/Note', () => {
 	beforeEach(async () => {
 		await setupDatabaseAndSynchronizer(1);
@@ -373,133 +364,4 @@ describe('screens/Note', () => {
 
 		unmount();
 	});
-
-	it.each([
-		[['viewer']],
-		[['editor']],
-	])('should initialize in the correct mode when noteVisiblePanes is %j', async (panes) => {
-		const { unmount } = await setupNoteWithPanes(panes);
-		await expectToBeEditing(panes.includes('editor'));
-		unmount();
-	});
-
-	it('should show toggle button', async () => {
-		const { unmount } = await setupNoteWithPanes(['viewer']);
-		const toggleButton = await screen.findByLabelText('Toggle view/edit');
-		expect(toggleButton).toBeVisible();
-		unmount();
-	});
-
-	it.each([
-		[['viewer']],
-		[['editor']],
-	])('should switch modes when toggle button is pressed', async (panes) => {
-		const initialEditing = panes.includes('editor');
-		const expectedEditing = !initialEditing;
-		const { unmount } = await setupNoteWithPanes(panes);
-		await expectToBeEditing(initialEditing);
-		const toggleButton = await screen.findByLabelText('Toggle view/edit');
-		fireEvent.press(toggleButton);
-		await expectToBeEditing(expectedEditing);
-		unmount();
-	});
-
-	it('should always start in edit mode for provisional notes regardless of noteVisiblePanes', async () => {
-		store.dispatch({
-			type: 'NOTE_VISIBLE_PANES_SET',
-			panes: ['viewer'],
-		});
-		const noteId = await openNewNote({ title: 'Provisional note', body: 'Test body' });
-		// Mark note as provisional by dispatching NOTE_UPDATE_ONE with provisional flag
-		const note = await Note.load(noteId);
-		store.dispatch({
-			type: 'NOTE_UPDATE_ONE',
-			note: note,
-			provisional: true,
-		});
-		const { unmount } = render(<WrappedNoteScreen />);
-		const titleInput = await screen.findByDisplayValue('Provisional note');
-		expect(titleInput).toBeVisible();
-		await expectToBeEditing(true);
-		unmount();
-	});
-
-	it.each([
-		[['viewer']],
-		[['editor']],
-	])('should preserve noteVisiblePanes state when leaving and returning to the same note', async (panes) => {
-		const firstRender = await setupNoteWithPanes(panes);
-		await expectToBeEditing(panes.includes('editor'));
-		// Navigate away
-		await act(async () => {
-			store.dispatch({
-				type: 'NAV_GO',
-				routeName: 'Notes',
-			});
-		});
-		firstRender.unmount();
-
-		// Navigate back to the same note
-		const currentState = store.getState();
-		const noteId = currentState.selectedNoteIds[0];
-		await act(async () => {
-			await openExistingNote(noteId);
-		});
-		const { unmount } = render(<WrappedNoteScreen />);
-		const titleInput = await screen.findByDisplayValue('Test note');
-		expect(titleInput).toBeVisible();
-		// Should still be in the same mode
-		await expectToBeEditing(panes.includes('editor'));
-		expect(store.getState().noteVisiblePanes).toEqual(panes);
-		unmount();
-	});
-
-	it.each([
-		[['viewer']],
-		[['editor']],
-	])('should preserve noteVisiblePanes state when navigating from note 1 to note 2', async (panes) => {
-		// Open note 1
-		await act(async () => {
-			store.dispatch({
-				type: 'NOTE_VISIBLE_PANES_SET',
-				panes: panes,
-			});
-		});
-		await act(async () => {
-			return await openNewNote({ title: 'Note 1', body: 'Test body 1' });
-		});
-		const render1 = render(<WrappedNoteScreen />);
-		const titleInput1 = await screen.findByDisplayValue('Note 1');
-		expect(titleInput1).toBeVisible();
-		await expectToBeEditing(panes.includes('editor'));
-		render1.unmount();
-
-		// Open note 2
-		const note2Id = await act(async () => {
-			return await openNewNote({ title: 'Note 2', body: 'Test body 2' });
-		});
-		await act(async () => {
-			await openExistingNote(note2Id);
-		});
-		const { unmount } = render(<WrappedNoteScreen />);
-		const titleInput2 = await screen.findByDisplayValue('Note 2');
-		expect(titleInput2).toBeVisible();
-		// Note 2 should be in the same mode
-		await expectToBeEditing(panes.includes('editor'));
-		expect(store.getState().noteVisiblePanes).toEqual(panes);
-		unmount();
-	});
-
-	it('should set the initial editor cursor location to the specified hash', async () => {
-		await openNewNote({ title: 'To be edited', body: 'a test\n\n# Test\n\n# Test 2\n\n# Test 3' });
-		store.dispatch({ type: 'NAV_GO', noteHash: 'test-2' });
-		const { unmount } = render(<WrappedNoteScreen />);
-
-		await openEditor();
-		const editor = await getMarkdownEditorControl();
-
-		expect(editor.getCursor().line).toBe(4);
-
-		unmount();
-	});
 });

packages/app-mobile/components/screens/Note/Note.tsx

@@ -19,6 +19,7 @@ const md5 = require('md5');
 import BackButtonService from '../../../services/BackButtonService';
 import NavService, { OnNavigateCallback as OnNavigateCallback } from '@joplin/lib/services/NavService';
 import { ModelType } from '@joplin/lib/BaseModel';
+import FloatingActionButton from '../../buttons/FloatingActionButton';
 import { fileExtension, safeFileExtension } from '@joplin/lib/path-utils';
 import * as mimeUtils from '@joplin/lib/mime-utils';
 import ScreenHeader, { MenuOptionType } from '../../ScreenHeader';
@@ -121,7 +122,6 @@ interface Props extends BaseProps {
 	pluginHtmlContents: PluginHtmlContents;
 	editorNoteReloadTimeRequest: number;
 	canPublish: boolean;
-	noteVisiblePanes: string[];
 }
 
 interface ComponentProps extends Props {
@@ -199,7 +199,6 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 	private editorPluginHandler_ = new EditorPluginHandler(PluginService.instance(), saveEvent => {
 		return shared.noteComponent_change(this, 'body', saveEvent.body);
 	});
-	private refreshKey: number | undefined;
 
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Old code before rule was applied
 	public static navigationOptions(): any {
@@ -209,11 +208,9 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 	public constructor(props: ComponentProps) {
 		super(props);
 
-		const initialMode = props.noteVisiblePanes?.includes('editor') ? 'edit' : 'view';
-
 		this.state = {
 			note: Note.new(),
-			mode: initialMode,
+			mode: 'view',
 			readOnly: false,
 			folder: null,
 			lastSavedNote: null,
@@ -245,16 +242,12 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 			titleContainerWidth: 0,
 		};
 
-		const initialScroll = NotePositionService.instance().getScrollPercent(props.noteId, defaultWindowId);
 		const initialCursorLocation = NotePositionService.instance().getCursorPosition(props.noteId, defaultWindowId).markdown;
-		// Ignore the initial scroll and cursor location when there's a note hash. The editor/viewer should jump to
-		// the hash, rather than the last position.
-		if (!props.noteHash) {
-			if (initialCursorLocation) {
-				this.selection = { start: initialCursorLocation, end: initialCursorLocation };
-			}
-			this.lastBodyScroll = initialScroll;
+		if (initialCursorLocation) {
+			this.selection = { start: initialCursorLocation, end: initialCursorLocation };
 		}
+		const initialScroll = NotePositionService.instance().getScrollPercent(props.noteId, defaultWindowId);
+		this.lastBodyScroll = initialScroll;
 
 		this.titleTextFieldRef = React.createRef();
 
@@ -300,7 +293,14 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 
 			if (this.state.mode === 'edit') {
 				Keyboard.dismiss();
+
+				this.setState({
+					mode: 'view',
+				});
+
 				await this.undoRedoService_.reset();
+
+				return true;
 			}
 
 			if (this.state.fromShare) {
@@ -393,7 +393,6 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 				setMode: (mode: 'view'|'edit') => {
 					this.setState({ mode });
 				},
-				dispatch: this.props.dispatch,
 			},
 			commands,
 			true,
@@ -712,27 +711,10 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 			});
 		}
 
-		const editorPluginIdsChanged = this.props.visibleEditorPluginIds !== prevProps.visibleEditorPluginIds;
-		if (editorPluginIdsChanged || this.props.editorNoteReloadTimeRequest !== prevProps.editorNoteReloadTimeRequest) {
+		if (this.props.visibleEditorPluginIds !== prevProps.visibleEditorPluginIds || this.props.editorNoteReloadTimeRequest !== prevProps.editorNoteReloadTimeRequest) {
 			const { editorPlugin } = getShownPluginEditorView(this.props.plugins, this.props.windowId);
-			const explicitReloadRequired = !editorPlugin && this.props.editorNoteReloadTimeRequest > this.state.noteLastLoadTime;
-
-			if (explicitReloadRequired) {
-				void this.reloadNoteAndUpdateRefreshKey();
-			}
-
-			if (explicitReloadRequired || (editorPlugin && editorPluginIdsChanged)) {
-				// Clear the undo / redo state, as undo / redo steps wont be in sync with the current content after the note editor has been refreshed
-				if (!this.useEditorBeta()) {
-					void this.undoRedoService_.reset();
-				}
-
-				this.setState({
-					undoRedoButtonState: {
-						canUndo: false,
-						canRedo: false,
-					},
-				});
+			if (!editorPlugin && this.props.editorNoteReloadTimeRequest > this.state.noteLastLoadTime) {
+				void shared.reloadNote(this);
 			}
 		}
 
@@ -773,11 +755,6 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 		});
 	}
 
-	private async reloadNoteAndUpdateRefreshKey() {
-		await shared.reloadNote(this);
-		this.refreshKey = this.props.editorNoteReloadTimeRequest;
-	}
-
 	private title_changeText(text: string) {
 		let newText = text;
 		newText = text.replace(/(\r\n|\n|\r)/gm, ' ');
@@ -1468,14 +1445,6 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 		await this.saveOneProperty('todo_completed', checked ? time.unixMs() : 0);
 	}
 
-	private toggleVisiblePanes = () => {
-		const isSwitchingToEdit = this.state.mode === 'view';
-		void CommandService.instance().execute('toggleVisiblePanes');
-		if (isSwitchingToEdit) {
-			this.doFocusUpdate_ = true;
-		}
-	};
-
 	public scheduleFocusUpdate() {
 		if (this.focusUpdateIID_) shim.clearInterval(this.focusUpdateIID_);
 
@@ -1736,6 +1705,7 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 					bodyComponent = <NoteEditor
 						ref={this.editorRef}
 						toolbarEnabled={this.props.toolbarEnabled && !increaseSpaceForEditor}
+						themeId={this.props.themeId}
 						noteId={this.props.noteId}
 						noteHash={this.props.noteHash}
 						initialText={note.body}
@@ -1766,12 +1736,32 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 						onScroll={this.props.editorType === EditorType.RichText ? this.onBodyViewerScroll : this.onMarkdownEditorScroll}
 
 						mode={this.props.editorType}
-						refreshKey={this.refreshKey}
 					/>;
 				}
 			}
 		}
 
+		const voiceTypingDialogShown = this.state.showSpeechToTextDialog || this.state.showAudioRecorder;
+		const renderActionButton = () => {
+			if (voiceTypingDialogShown) return null;
+			if (editorView) return null;
+			if (!this.state.note || !!this.state.note.deleted_time) return null;
+
+			const editButton = {
+				label: _('Edit'),
+				icon: 'create',
+				onPress: () => {
+					this.setState({ mode: 'edit' });
+
+					this.doFocusUpdate_ = true;
+				},
+			};
+
+			if (this.state.mode === 'edit') return null;
+
+			return <FloatingActionButton mainButton={editButton} />;
+		};
+
 		// Save button is not really needed anymore with the improved save logic
 		const showSaveButton = false; // this.state.mode === 'edit' || this.isModified() || this.saveButtonHasBeenShown_;
 		const saveButtonDisabled = true;// !this.isModified();
@@ -1810,7 +1800,6 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 					multiline={this.state.multiline}
 					text={note.title}
 					updateState={textWrapCalculator_updateState}
-					readOnly={false}
 				/>
 				{isTodo && <Checkbox style={this.styles().checkbox} checked={!!Number(note.todo_completed)} onChange={this.todoCheckbox_change} />}
 				<TextInput
@@ -1871,9 +1860,6 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 			undoButtonDisabled={!this.state.undoRedoButtonState.canUndo && this.state.undoRedoButtonState.canRedo}
 			onUndoButtonPress={this.screenHeader_undoButtonPress}
 			onRedoButtonPress={this.screenHeader_redoButtonPress}
-			showViewToggleButton={!!this.state.note && !this.state.note.deleted_time && !editorView}
-			viewToggleIconName={this.state.mode === 'edit' ? 'ionicon book' : 'ionicon pencil'}
-			onViewTogglePress={this.toggleVisiblePanes}
 			title={getDisplayParentTitle(this.state.note, this.state.folder)}
 		/>;
 
@@ -1883,6 +1869,7 @@ class NoteScreenComponent extends BaseScreenComponent<ComponentProps, State> imp
 				{!increaseSpaceForEditor && titleComp}
 				{bodyComponent}
 				{renderVoiceTypingDialogs()}
+				{renderActionButton()}
 
 				<SelectDateTimeDialog themeId={this.props.themeId} shown={this.state.alarmDialogShown} date={dueDate} onAccept={this.onAlarmDialogAccept} onReject={this.onAlarmDialogReject} />
 
@@ -1950,7 +1937,6 @@ const NoteScreen = connect((state: AppState) => {
 		plugins: state.pluginService.plugins,
 		pluginHtmlContents: state.pluginService.pluginHtmlContents,
 		editorNoteReloadTimeRequest: state.editorNoteReloadTimeRequest,
-		noteVisiblePanes: state.noteVisiblePanes,
 
 		editorType: state.settings['editor.codeView'] ? EditorType.Markdown : EditorType.RichText,
 

packages/app-mobile/components/screens/Note/commands/toggleVisiblePanes.ts

@@ -1,6 +1,5 @@
 import { CommandContext, CommandDeclaration, CommandRuntime } from '@joplin/lib/services/CommandService';
 import { CommandRuntimeProps } from '../types';
-import Setting from '@joplin/lib/models/Setting';
 
 export const declaration: CommandDeclaration = {
 	// For compatibility with the desktop app, this command is called "toggleVisiblePanes".
@@ -11,13 +10,8 @@ export const declaration: CommandDeclaration = {
 export const runtime = (props: CommandRuntimeProps): CommandRuntime => {
 	return {
 		execute: async (_context: CommandContext) => {
-			const panes = Setting.value('noteVisiblePanes') || ['viewer'];
-			props.dispatch({
-				type: 'NOTE_VISIBLE_PANES_SET',
-				panes: panes.includes('editor') ? ['viewer'] : ['editor'],
-			});
-			const currentMode = props.getMode();
-			const newMode = currentMode === 'edit' ? 'view' : 'edit';
+			// For now, the only two "panes" on mobile are view and edit.
+			const newMode = props.getMode() === 'edit' ? 'view' : 'edit';
 			props.setMode(newMode);
 		},
 	};