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

Fix for node.rel crash #630

Merged
merged 4 commits into from
Feb 21, 2019
Merged

Conversation

hypest
Copy link
Contributor

@hypest hypest commented Feb 20, 2019

Addresses at least one case of "Pasting from some sources causes a red screen (related comment)" from #624

To test:

  1. On the mobile, open the browser and copy the "Gravity Forms" text (it's a link) from https://designsbytierney.com/2010/01/how-to-add-edit-format-text-in-a-wordpress-post-or-page/
  2. Run the demo app
  3. In a paragraph block, paste
  4. It should work and the link be available in the block

@hypest hypest added this to the Beta milestone Feb 20, 2019
Copy link
Contributor

@diegoreymendez diegoreymendez left a comment

Choose a reason for hiding this comment

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

I couldn't get the mentioned red screen to pop-up.

@hypest
Copy link
Contributor Author

hypest commented Feb 21, 2019

Thanks for the pass @diegoreymendez !

I will actually need to update this PR to host a different solution as, I closed the Gutenberg side PR as invalid. Will report here again soon.

@hypest
Copy link
Contributor Author

hypest commented Feb 21, 2019

Ready for another pass!

This time, the solution uses this PR opened against the jsdom-jscore library repo we're using.

@mkevins
Copy link
Contributor

mkevins commented Feb 21, 2019

This time, the solution uses this PR opened against the jsdom-jscore library repo we're using.

I think this will make a lot of things easier. Does this mean we can move some (maybe all?) of the jsdom-patches?

@hypest hypest changed the title Update GB ref with fix for node.rel Fix for node.rel crash Feb 21, 2019
@hypest
Copy link
Contributor Author

hypest commented Feb 21, 2019

Does this mean we can move some (maybe all?) of the jsdom-patches?

Most probably, but let's wait until after our release this coming days so we don't shake things up too much.

@hypest
Copy link
Contributor Author

hypest commented Feb 21, 2019

jscore-jsdom-rn upstream accepted our PR so, will update this branch to use the upstream.

@diegoreymendez diegoreymendez self-requested a review February 21, 2019 14:03
Copy link
Contributor

@diegoreymendez diegoreymendez left a comment

Choose a reason for hiding this comment

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

Again, I can't manage to reproduce the original issue. Nice work!

@hypest hypest merged commit 8ee7064 into develop Feb 21, 2019
@hypest hypest deleted the fix/red-screen-pasting-from-particular-site branch February 21, 2019 14:54
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.

3 participants