Skip to content

Allow setting setContenteditableValue to plaintext-only for tests#29999

Merged
mmalerba merged 1 commit into
angular:mainfrom
tatp22:main
Nov 13, 2024
Merged

Allow setting setContenteditableValue to plaintext-only for tests#29999
mmalerba merged 1 commit into
angular:mainfrom
tatp22:main

Conversation

@tatp22

@tatp22 tatp22 commented Nov 12, 2024

Copy link
Copy Markdown
Contributor

plaintext-only is a valid value for contenteditable that specifies that the raw text is editable but rich formatting is disabled. This is supported now for every browser except for Firefox, where it is now supported in a nightly version. See more specifications here.

@tatp22 tatp22 requested a review from a team as a code owner November 12, 2024 16:31
@tatp22 tatp22 requested review from amysorto and crisbeto and removed request for a team November 12, 2024 16:31

@mmalerba mmalerba left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mmalerba mmalerba added the target: patch This PR is targeted for the next patch release label Nov 12, 2024
@mmalerba mmalerba self-assigned this Nov 12, 2024
@mmalerba mmalerba added the action: merge The PR is ready for merge by the caretaker label Nov 12, 2024
@mmalerba mmalerba merged commit b3c93c5 into angular:main Nov 13, 2024
mmalerba pushed a commit that referenced this pull request Nov 13, 2024
mmalerba pushed a commit that referenced this pull request Nov 13, 2024
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Dec 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants