facebook / facebook/lexical

Bug: Unable to move between consecutive decorator nodes on mobile device.

Open
#4,859 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
23.9k
Forks
2.2k
Avg merge
1d 16h
Merged PRs (30d)
61

Description

In the lexical playground project, when I insert two images one after the other, I want the user to be able to move between photos by moving the cursor position on mobile devices, but it can't because `ImageNode` is given this style `user-select: none` in `.editor-image` class. I removed this style and I was able to move between images, but I wasn't able to delete images by pressing backspace on my phone keyboard.

Lexical version:
0.11.3
browser:
chrome mobile 114

## Steps To Reproduce

1. In the lexical playground project, go to index.css and remove the `user-select:none` style for the `.editor-image` class.
2. you are able to move between continuous images, but you are unable to delete them.

Link to code example:

## The current behavior
https://github.com/facebook/lexical/assets/24582053/0cf2f7b2-375f-4723-beee-f58b56b74d52

## The expected behavior
the user must be able to move between images with the cursor on mobile devices and he must be able to delete the one before the cursor.

Contributor guide

Open the contributing guide

Research direction

Start in the lexical playground project at index.css and inspect the ImageNode styling, especially the .editor-image user-select rule. Reproduce the consecutive-image case in Chrome mobile and verify that the cursor can move between images and that backspace deletes the image before the cursor.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend, mobile-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.