Skip to content

Add mobile Spacer component#1436

Merged
lukewalczak merged 29 commits intodevelopfrom
callstack/rnmobile/spacer
Nov 8, 2019
Merged

Add mobile Spacer component#1436
lukewalczak merged 29 commits intodevelopfrom
callstack/rnmobile/spacer

Conversation

@lukewalczak
Copy link
Copy Markdown
Contributor

@lukewalczak lukewalczak commented Oct 11, 2019

Fixes #1428
Ref to gutenberg PR: WordPress/gutenberg#17896

NOTE: Spacer block is currently behind the DEV flag so, it's not visible to the production user

To test: Instruction attached in gutenberg PR.

Update release notes:

  • If there are user facing changes, I have added an item to RELEASE-NOTES.txt.

@pinarol pinarol requested a review from etoledom October 21, 2019 13:07
@pinarol pinarol added the [Type] Enhancement Improves a current area of the editor label Oct 21, 2019
@lukewalczak lukewalczak force-pushed the callstack/rnmobile/spacer branch from 2c6d095 to 1d46e96 Compare November 5, 2019 08:39
@pinarol
Copy link
Copy Markdown
Contributor

pinarol commented Nov 7, 2019

Could you add a sample spacer block into the demo content?(initial-html.js)

@etoledom
Copy link
Copy Markdown
Contributor

etoledom commented Nov 7, 2019

Could you add a sample spacer block into the demo content?(initial-html.js)

@pinarol - There are two already. I guess they used to be hidden as “unsupported”.

@pinarol
Copy link
Copy Markdown
Contributor

pinarol commented Nov 7, 2019

No need then, thanks for the heads up @etoledom :) let’s merge this anyway. gutenberg ref is prolly updated by another pr already but if we close this without merging then it can be misleading for the weekly reports.

@lukewalczak lukewalczak merged commit 1995c3a into develop Nov 8, 2019
@lukewalczak lukewalczak deleted the callstack/rnmobile/spacer branch November 8, 2019 09:22
@hypest hypest mentioned this pull request Nov 11, 2019
4 tasks
@hypest
Copy link
Copy Markdown
Contributor

hypest commented Nov 11, 2019

👋, my impression is that the Spacer block is currently behind the DEV flag so, not visible to the end/production user. Can you confirm @lukewalczak and if that's the case, let's also update the PR description to note that fact, thanks!

@lukewalczak
Copy link
Copy Markdown
Contributor Author

@hypest You're right, Spacer component is behind the DEV flag. I've updated the description above 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Enhancement Improves a current area of the editor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Spacer block - 1st iteration

4 participants