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 paragraph splits on the Paragraph block #104

Merged
merged 5 commits into from
Aug 13, 2018

Conversation

SergioEstevao
Copy link
Contributor

This PR fixes a regression where inserting multiple lines on a paragraph block was crashing the example app.

@SergioEstevao SergioEstevao added the [Type] Bug Something isn't working label Aug 13, 2018
@SergioEstevao SergioEstevao requested a review from koke August 13, 2018 10:49
package.json Outdated
@@ -45,7 +45,7 @@
"pre-build": "yarn check-gb-unused-babelrc && yarn bypass-gb-babel",
"test:pre-build": "yarn check-gb-unused-babelrc && yarn test:bypass-gb-babel",
"start": "yarn pre-build && yarn react-native start",
"start:reset": "yarn clean:runtime && yarn start --reset-cache",
"start:reset": "yarn clean:metro && yarn start --reset-cache",
Copy link
Member

Choose a reason for hiding this comment

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

There's #103 which looks like it's also taking care of this. Can you maybe review that one and get it merged? Otherwise, I think there will be a conflict

@SergioEstevao
Copy link
Contributor Author

@koke thanks for the heads up! I just reviewed the other PR and reverted the changes on this PR.

Copy link
Contributor

@hypest hypest left a comment

Choose a reason for hiding this comment

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

Works nicely on my end @SergioEstevao 👍

Feel free to update to the GB hash when the GB side PR is merged and merge here.

@SergioEstevao SergioEstevao merged commit 949dc53 into master Aug 13, 2018
@SergioEstevao SergioEstevao deleted the fix/paragraph_splits branch August 13, 2018 18:30
hypest pushed a commit that referenced this pull request Jan 3, 2019
Avoid testing for focus if there is nothing focused
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants