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

Ports the Paragraph Block to React Native #8149

Closed
wants to merge 8 commits into from

Conversation

SergioEstevao
Copy link
Contributor

Description

This PR implements the Paragraph Block using RN. This is still a stubbed version using the TextInput component.

How has this been tested?

No tests implemented.

Types of changes

Implements a native version of the Paragraph Block using a native version of the RichText component.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.

@SergioEstevao SergioEstevao added the Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) label Jul 23, 2018
@SergioEstevao SergioEstevao added this to the 3.4 milestone Jul 23, 2018
@SergioEstevao SergioEstevao requested a review from hypest July 23, 2018 20:48
*
* @return {Function} Returns the new composite function.
*/
export { flowRight as compose };
Copy link
Member

@gziolo gziolo Jul 24, 2018

Choose a reason for hiding this comment

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

compose and createHigherOrderComponent were moved to compose package.

See: https://github.com/WordPress/gutenberg/blob/master/packages/compose/src/index.js

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think we are still some commits behind in terms of a branch that is compatible with RN.

return (
<TextInput
multiline={ true }
value={ content[0] }
Copy link
Member

Choose a reason for hiding this comment

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

This file could use some whitespace formatting.

},
];

class ParagraphBlock extends Component {
Copy link
Member

Choose a reason for hiding this comment

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

We should move ParagraphBlock to its own file edit.js and override it rather than all paragraph settings.

@SergioEstevao SergioEstevao changed the title Ports the Paragraph Block to Read Native Ports the Paragraph Block to React Native Jul 24, 2018
@SergioEstevao
Copy link
Contributor Author

Closing in favor of #8231

@SergioEstevao SergioEstevao deleted the rnmobile/sergio_paragraph_block_port branch July 27, 2018 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants