Skip to content

Dev.ap/tip tap editor#585

Closed
roedoejet wants to merge 5 commits into
mainfrom
dev.ap/tip-tap-editor
Closed

Dev.ap/tip tap editor#585
roedoejet wants to merge 5 commits into
mainfrom
dev.ap/tip-tap-editor

Conversation

@roedoejet

Copy link
Copy Markdown
Collaborator

PR Goal?

This is a re-implemntation of #583 using TipTap rich text editor instead of text overlay.

The goal of this PR is to see how far we can get with a rich text editor in solving some outstanding issues with the studio.

Namely:

  1. Localized g2p error handling
  2. Better paragraph/page boundary highlighting
  3. Ability to add do-not-align spans

Fixes?

Feedback sought?

Functionality mostly - I've only implemented the first 2 (and haven't done DNA spans yet). It's more code, but that's mostly because instead of sending plain text, we're explicitly mapping to XML. A good 500 lines are just defining the TipTap Schema and methods for converting to/from that format and XML/plain text.

Priority?

medium

Tests added?

Lots of spec tests for various scenarios

How to test?

npx nx test:once studio-web

Confidence?

medium. I got a lot of help from Claude Sonnet ( after planning it all out with @joanise)

TODO

  • We'll have to update the tour accordingly
  • Add do-not-align segment UX
  • Change and/or translate new toast message asking user to look at underlined g2p errored words
  • i18n for the page insert button

Version change?

roedoejet and others added 5 commits July 2, 2026 15:45
@semanticdiff-com

semanticdiff-com Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review changes with  SemanticDiff

Changed Files
File Status
  packages/studio-web/src/app/shared/shared.module.ts  48% smaller
  packages/studio-web/src/app/upload/upload.component.ts  19% smaller
  packages/studio-web/src/test.ts  9% smaller
  packages/studio-web/src/app/text-format-dialog/text-format-dialog.component.html  6% smaller
  .github/workflows/deploy.yml  0% smaller
  .github/workflows/dev-preview.yml  0% smaller
  .github/workflows/end-to-end-tests.yml  0% smaller
  .github/workflows/windows-tests.yml  0% smaller
  package-lock.json Unsupported file format
  package.json  0% smaller
  packages/studio-web/angular.json  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/g2p/document-words.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/g2p/document-words.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/g2p/error-highlight-extension.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/g2p/error-highlight-extension.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/g2p/g2p-error-mapping.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/g2p/g2p-error-mapping.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/popover/error-popover.component.html  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/popover/error-popover.component.sass Unsupported file format
  packages/studio-web/src/app/shared/tiptap-editor/popover/error-popover.component.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/insert-page-break-command.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/insert-page-break-command.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/keymap-extension.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/keymap-extension.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/nodes.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/nodes.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/paste-extension.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/paste-extension.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/split-into-new-paragraph.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/schema/split-into-new-paragraph.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/serialization/doc-to-xml.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/serialization/doc-to-xml.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/serialization/text-to-doc.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/serialization/text-to-doc.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/serialization/xml-to-doc.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/serialization/xml-to-doc.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/tiptap-editor.component.html  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/tiptap-editor.component.sass Unsupported file format
  packages/studio-web/src/app/shared/tiptap-editor/tiptap-editor.component.spec.ts  0% smaller
  packages/studio-web/src/app/shared/tiptap-editor/tiptap-editor.component.ts  0% smaller
  packages/studio-web/src/app/studio/studio.component.spec.ts  0% smaller
  packages/studio-web/src/app/studio/studio.component.ts  0% smaller
  packages/studio-web/src/app/studio/studio.service.ts  0% smaller
  packages/studio-web/src/app/upload/upload.component.html  0% smaller
  packages/studio-web/src/app/upload/upload.component.spec.ts  0% smaller
  packages/studio-web/tsconfig.spec.json  0% smaller

@roedoejet roedoejet closed this Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2026-07-03 18:25 UTC

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants