Skip to content

[lexical] Fix: ctrl+a doesn't work well in plain text mode on Firefox - #7579

Merged
etrepum merged 3 commits into
facebook:mainfrom
wataru-chocola:fix_select_all_on_firefox
Jun 4, 2025
Merged

etrepum merged 3 commits into
facebook:mainfrom
wataru-chocola:fix_select_all_on_firefox

Conversation

@wataru-chocola

@wataru-chocola wataru-chocola commented May 29, 2025 •

Copy link
Copy Markdown
Contributor

Description

Enable override of selectall introduced by #4671 for Firefox.

Closes #7578

Test plan

Before

before

After

after

@vercel

vercel Bot commented May 29, 2025 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lexical ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 3, 2025 10:40pm
lexical-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 3, 2025 10:40pm

@wataru-chocola

Copy link
Copy Markdown
Contributor Author

Fixed test cases.

I skipped selectAll related tests if os is linux and browser is firefox, because selectAll in keyboardShortcuts/index.mjs does emulation on the condition and tests won't work.

@etrepum etrepum left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't think skipping the tests is the best approach, especially because it means that the tests don't cover what you're trying to fix here

@wataru-chocola

Copy link
Copy Markdown
Contributor Author

I don't think skipping the tests is the best approach, especially because it means that the tests don't cover what you're trying to fix here

You are right.
I fixed test cases again.

@etrepum etrepum left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This seems reasonable to me, do you have any opinion on it @zurfyx?

@etrepum
etrepum added this pull request to the merge queue Jun 4, 2025
Merged via the queue into facebook:main with commit bb9fb20 Jun 4, 2025
@etrepum etrepum mentioned this pull request Jun 4, 2025
fantactuka pushed a commit that referenced this pull request Aug 11, 2025

This branch was successfully deployed

2 active deployments
Preview – lexical-playground — f69c790a Deployed Jun 3, 2025 by vercel[bot]
Preview – lexical — f69c790a Deployed Jun 3, 2025 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. extended-tests Run extended e2e tests on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Ctrl+A doesn't work well in PlainTextPlugin on Firefox

3 participants