Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add tiptap's subscript and superscript extansion to tiptap wysiwyg #2732

Open
wants to merge 3 commits into
base: 3.x
Choose a base branch
from

Conversation

tommycornilleau
Copy link

Add subscript or superscript content in TipTap wysiwyg

Subscript uses @tiptap/extension-subscript and wrap wysiwyg text within a <sub /> tag.
Superscript uses @tiptap/extension-superscript and wrap wysiwyg text within a <sup /> tag.

Capture d’écran 2025-02-24 à 12 07 04

@CLAassistant
Copy link

CLAassistant commented Feb 24, 2025

CLA assistant check
All committers have signed the CLA.

Comment on lines 384 to 385
subscript: true,
superscript: true,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will enable them by default on all Tiptap fields where the developer passes an empty array of options, which I don't think should be the case in a minor release.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, this was probably too much

Comment on lines 29 to 30
'subscript',
'superscript',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here, if the developer does not specify their options in a field, they will end up with new buttons after updating Twill, which I consider as a "breaking" change. We should remove it from this default configuration for now. We might decide to add them back for the next major release.

Comment on lines +28 to +29
"@tiptap/extension-subscript": "^2.11.5",
"@tiptap/extension-superscript": "^2.11.5",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When new npm dependencies are added, the package.lock file needs to be pushed to the repository.

@tommycornilleau tommycornilleau requested a review from ifox March 13, 2025 07:15
@tommycornilleau
Copy link
Author

Changes request has been addressed.
Subscript and superscript are no longer default options and package-lock has been added

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.

3 participants