Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Fix #1326 Safari can't focus with draggable parent #1356

Closed
wants to merge 6 commits into from

Conversation

mattkrick
Copy link
Contributor

Summary

DraftEditors that lived inside draggable parents were useless in Safari. Now they work.
Solution found here: react-dnd/react-dnd#178 (comment)

Test Plan

Use the fiddle in #1326 with the updated version.
For a quick n dirty, just add this to the stylesheet:
div[contenteditable="true"] { user-select: text;}

@jordanh
Copy link

jordanh commented Sep 1, 2017

👏 👏 👏

@flarnie flarnie self-requested a review September 12, 2017 00:50
@flarnie
Copy link
Contributor

flarnie commented Sep 22, 2017

Thanks for submitting this!

The bug is easy to reproduce, and I'm still testing to verify that your change fixes it.

Copy link
Contributor

@flarnie flarnie left a comment

Choose a reason for hiding this comment

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

Awesome detective work!
happyjello
Adding this to the queue for merging~

@facebook-github-bot
Copy link

@flarnie has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

3 similar comments
@facebook-github-bot
Copy link

@flarnie has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link

@flarnie has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link

@flarnie has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@flarnie flarnie added this to the v0.10.4 milestone Sep 28, 2017
@facebook-github-bot
Copy link

@flarnie has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

1 similar comment
@facebook-github-bot
Copy link

@flarnie has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

midas19910709 added a commit to midas19910709/draft-js that referenced this pull request Mar 30, 2022
Summary:
**Summary**

DraftEditors that lived inside draggable parents were useless in Safari. Now they work.
Solution found here: react-dnd/react-dnd#178 (comment)

**Test Plan**

Use the fiddle in #1326 with the updated version.
For a quick n dirty, just add this to the stylesheet:
`div[contenteditable="true"] { user-select: text;}`
Closes facebookarchive/draft-js#1356

Differential Revision: D5901625

fbshipit-source-id: 7d3797286e4040575b9b5503c776bff1e29b67a9
alicayan008 pushed a commit to alicayan008/draft-js that referenced this pull request Jul 4, 2023
Summary:
**Summary**

DraftEditors that lived inside draggable parents were useless in Safari. Now they work.
Solution found here: react-dnd/react-dnd#178 (comment)

**Test Plan**

Use the fiddle in #1326 with the updated version.
For a quick n dirty, just add this to the stylesheet:
`div[contenteditable="true"] { user-select: text;}`
Closes facebookarchive/draft-js#1356

Differential Revision: D5901625

fbshipit-source-id: 7d3797286e4040575b9b5503c776bff1e29b67a9
aforismesen added a commit to aforismesen/draft-js that referenced this pull request Jul 12, 2024
Summary:
**Summary**

DraftEditors that lived inside draggable parents were useless in Safari. Now they work.
Solution found here: react-dnd/react-dnd#178 (comment)

**Test Plan**

Use the fiddle in #1326 with the updated version.
For a quick n dirty, just add this to the stylesheet:
`div[contenteditable="true"] { user-select: text;}`
Closes facebookarchive/draft-js#1356

Differential Revision: D5901625

fbshipit-source-id: 7d3797286e4040575b9b5503c776bff1e29b67a9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants