Skip to content

[lexical-playground] Bug Fix: Change list, strikethrough and quoteblock shortcuts to match Google Docs for Windows compatibility#7556

Merged
etrepum merged 1 commit intofacebook:mainfrom
harshmetkel24:7543
May 22, 2025
Merged

[lexical-playground] Bug Fix: Change list, strikethrough and quoteblock shortcuts to match Google Docs for Windows compatibility#7556
etrepum merged 1 commit intofacebook:mainfrom
harshmetkel24:7543

Conversation

@harshmetkel24
Copy link
Copy Markdown
Contributor

Description

I found that keyboard shortcuts for Bullet list, Strikethrough and Quote block formatting were conflicting with some existing shortcut combinations on Windows machine. So while updating to these new shortcuts I've tried to mimic what google docs does along with aim to have somewhat similar key combinations for both Mac and Windows devices

Closes #7535

Test plan

updated relevant tests

Before

Action Mac Windows
Numbered List ⌘+Opt+5 Ctrl+Alt+5
Bullet List ⌘+Opt+4 Ctrl+Alt+4
Check List ⌘+Opt+6 Ctrl+Alt+6
Strikethrough ⌘+Shift+S Ctrl+Shift+S
Quote Block ⌘+Opt+Q Ctrl+Alt+Q

After

Action Mac Windows
Numbered List ⌘+Shift+7 Ctrl+Shift+7
Bullet List ⌘+Shift+8 Ctrl+Shift+8
Check List ⌘+Shift+9 Ctrl+Shift+9
Strikethrough ⌘+Shift+X Ctrl+Shift+X
Quote Block ⌃+Shift+Q Ctrl+Shift+Q

…rking in windows fix by unifying and avoiding existing shortcuts
@vercel
Copy link
Copy Markdown

vercel bot commented May 21, 2025

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 May 21, 2025 11:48am
lexical-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 11:48am

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 21, 2025
@etrepum etrepum added the extended-tests Run extended e2e tests on a PR label May 21, 2025
@etrepum etrepum changed the title [7535][BUG-FIX] bullet, strikethrough and quoteblock shortcuts not wo… [lexical-playground] Bug Fix: Change bullet, strikethrough and quoteblock shortcuts to match Google Docs for Windows compatibility May 21, 2025
@etrepum etrepum changed the title [lexical-playground] Bug Fix: Change bullet, strikethrough and quoteblock shortcuts to match Google Docs for Windows compatibility [lexical-playground] Bug Fix: Change list, strikethrough and quoteblock shortcuts to match Google Docs for Windows compatibility May 21, 2025
@etrepum etrepum added this pull request to the merge queue May 22, 2025
Merged via the queue into facebook:main with commit 01f967e May 22, 2025
95 of 98 checks passed
fantactuka pushed a commit that referenced this pull request Aug 11, 2025
…ck shortcuts to match Google Docs for Windows compatibility (#7556)
GermanJablo added a commit to payloadcms/payload that referenced this pull request Sep 3, 2025
Fixes #13386

Below I write a clarification to copy and paste into the release note,
based on our latest upgrade of Lexical [in
v3.29.0](https://github.com/payloadcms/payload/releases/tag/v3.29.0).

## Important
This release upgrades the lexical dependency from 0.28.0 to 0.34.0.

If you installed lexical manually, update it to 0.34.0. Installing
lexical manually is not recommended, as it may break between updates,
and our re-exported versions should be used. See the [yellow banner
box](https://payloadcms.com/docs/rich-text/custom-features) for details.

If you still encounter richtext-lexical errors, do the following, in
this order:

- Delete node_modules
- Delete your lockfile (e.g. pnpm-lock.json)
- Reinstall your dependencies (e.g. pnpm install)

### Lexical Breaking Changes

The following Lexical releases describe breaking changes. We recommend
reading them if you're using Lexical APIs directly
(`@payloadcms/richtext-lexical/lexical/*`).

- [v.0.33.0](https://github.com/facebook/lexical/releases/tag/v0.33.0)
- [v.0.30.0](https://github.com/facebook/lexical/releases/tag/v0.30.0)
- [v.0.29.0](https://github.com/facebook/lexical/releases/tag/v0.29.0)

___

TODO:
- [x] facebook/lexical#7719
- [x] facebook/lexical#7362
- [x] facebook/lexical#7707
- [x] facebook/lexical#7388
- [x] facebook/lexical#7357
- [x] facebook/lexical#7352
- [x] facebook/lexical#7472
- [x] facebook/lexical#7556
- [x] facebook/lexical#7417
- [x] facebook/lexical#1036
- [x] facebook/lexical#7509
- [x] facebook/lexical#7693
- [x] facebook/lexical#7408
- [x] facebook/lexical#7450
- [x] facebook/lexical#7415
- [x] facebook/lexical#7368
- [x] facebook/lexical#7372
- [x] facebook/lexical#7572
- [x] facebook/lexical#7558
- [x] facebook/lexical#7613
- [x] facebook/lexical#7405
- [x] facebook/lexical#7420
- [x] facebook/lexical#7662

---
- To see the specific tasks where the Asana app for GitHub is being
used, see below:
  - https://app.asana.com/0/0/1211202581885926
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: strikethrough, bullet formatting and quote block formatting not working through keyboard shortcuts on windows

3 participants