Skip to content

Conversation

@gjulivan
Copy link
Collaborator

Pull request type


Description

@gjulivan gjulivan changed the title fix: tab keypress and softbreak on rich text [WC-3149][WC-3109]: tab keypress and softbreak on rich text Nov 24, 2025
@gjulivan gjulivan marked this pull request as ready for review November 24, 2025 12:18
@gjulivan gjulivan requested a review from a team as a code owner November 24, 2025 12:18
@gjulivan gjulivan force-pushed the rte-update branch 3 times, most recently from e3be62e to b67ccca Compare December 3, 2025 13:04
@gjulivan gjulivan closed this Dec 3, 2025
@gjulivan gjulivan reopened this Dec 3, 2025
@gjulivan gjulivan force-pushed the rte-update branch 2 times, most recently from 93d4856 to c1a6401 Compare December 8, 2025 10:24
Copy link
Contributor

@samuelreichert samuelreichert left a comment

Choose a reason for hiding this comment

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

LGTM

import Quill from "quill";
import Module from "quill/core/module";
import QuillClipboard from "quill/modules/clipboard";
// import QuillClipboard from "quill/modules/clipboard";
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// import QuillClipboard from "quill/modules/clipboard";

Maybe remove if the plan is to not use it anymore in the future

Comment on lines +32 to +67
return [
"address",
"article",
"blockquote",
"canvas",
"dd",
"div",
"dl",
"dt",
"fieldset",
"figcaption",
"figure",
"footer",
"form",
"h1",
"h2",
"h3",
"h4",
"h5",
"h6",
"header",
"iframe",
"li",
"main",
"nav",
"ol",
"output",
"p",
"pre",
"section",
"table",
"td",
"tr",
"ul",
"video"
].includes(node.tagName.toLowerCase());
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we have maybe a const here? like const TAG_NAMES = [...]

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants