Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow browser's default paste behavior when pasting from Office Android #2863

Merged

Conversation

Rain-Zheng
Copy link
Contributor

Android users are unable to paste content from Word to Outlook after recent Chromium update. The root cause is that the clipboard data from Word contains only a File object, without any HTML or plain text. And we can't use FileReader to extract paste content from it.

image
image

To workaround this issue, we are adding a check to CopyPastePlugin: if the clipboard data can't be processed by roosterjs (e.g., it doesn't have any plain text, HTML and image), preventDefault() shouldn't be called, allowing the browser to handle the paste event instead.

@JiuqingSong JiuqingSong merged commit f753bec into microsoft:master Nov 7, 2024
4 checks passed
BryanValverdeU added a commit that referenced this pull request Nov 15, 2024
* fix test

* test

* Merge text node and segments (#2846)

* Merge text segments

* Fix test

* merge node

* fix build and test

* add test

* Add test

* fix test

* Remove tablePreProcessor (#2849)

* Add change data and apiName to ContentChangedEvent when handle keyboard input (#2854)

* Fix #2857 (#2858)

* Revert Shortcut command for Clear Format on mac (#2834)

* revert Shortcut command for Clear Format on mac

* fix ctrl and meta can press in the same time

* update comment

* Allow browser's default paste behavior when pasting from Office Android (#2863)

* Implement shouldPreventDefaultPaste function

* fix build

* fix build

* list-bugs

* fix image floating

* Customize the behavior of merging format values (#2865)

* define types

* add callbacks

* use param instead

* fix test

---------

Co-authored-by: Jiuqing Song <[email protected]>

* updateVersion.json

* Fix merge segment issue (#2871)

---------

Co-authored-by: Julia Roldi (from Dev Box) <[email protected]>
Co-authored-by: Julia Roldi <[email protected]>
Co-authored-by: Jiuqing Song <[email protected]>
Co-authored-by: 庄黛淳华 <[email protected]>
Co-authored-by: Rain-Zheng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants