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

Port Quote block step 1 #214

Closed
wants to merge 5 commits into from
Closed

Conversation

daniloercoli
Copy link
Contributor

@daniloercoli daniloercoli commented Oct 31, 2018

Add a very basic initial support for quote blocks.

GB side PR:

How has this been tested?

  • Make sure the (web) GB build and webapp work normally
  • Make sure the GB mobile demo page has a quote block added initially

There are still things need to be addressed, like adding the ability to create a block from toolbar.
Also the ability to add/remove the cite part is missing, as well as basic interactions "within" the block: move the cursor up/down between those 2 editors.

Gutenberg Side PR: WordPress/gutenberg#11311

…g-mobile into port-quote-block-step-1

* 'master' of https://github.com/wordpress-mobile/gutenberg-mobile: (27 commits)
  Point to latest from RecyclerViewList
  Point to latest react-native-aztec
  Adding yarn.lock field for `turbo-combine-reducers`.
  Added turbo-combine-reducers dependency to fix failed builds
  Update subrepo ref to point to master again
  Update subrepo ref
  No need for change of default Jest platform
  Update subrepo ref
  Update subrepo ref
  Fix lint warnings
  Mocking `react-native-gutenberg-bridge` to make iOS tests pass. From: jestjs/jest#2208 (comment)
  rn-gutenberg-bridge: Adding dumb iOS implementation to avoid errors.
  Revert "Update gutenberg subproject ref"
  Add react-native-hr library to add styled line+text elements easily
  Update gutenberg subproject ref
  Match wpandroid's Android API levels
  Remove plugin from Babel to make wpandroid release build work
  Fix wpandroid checkstyle issues
  Remove dep, Matro will find the package automagically
  Fix indentation
  ...

# Conflicts:
#	gutenberg
@koke koke changed the base branch from master to develop December 21, 2018 09:22
@koke
Copy link
Member

koke commented Dec 21, 2018

I've changed the target branch to develop, which is right now the same commit as master, so there shouldn't be any issue

@daniloercoli
Copy link
Contributor Author

This PR is very old, and we've recently agreed to re-use much of the web code as possible when porting new blocks to mobile. Experimenting this new way in the List block here: WordPress/gutenberg#14249

Also the GB side of this PR was already closed.

@SergioEstevao SergioEstevao deleted the port-quote-block-step-1 branch November 8, 2019 11:49
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