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

Refactor Caption Component #1678

Closed
wants to merge 9 commits into from
Closed

Refactor Caption Component #1678

wants to merge 9 commits into from

Conversation

geriux
Copy link
Contributor

@geriux geriux commented Dec 13, 2019

Fixes #1639

Gutenberg PR -> WordPress/gutenberg#19118

To test:

Check the Gutenberg PR

Update release notes:

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

@geriux geriux changed the title Refactor Caption Block Refactor Caption component Dec 13, 2019
@geriux geriux changed the title Refactor Caption component Refactor Caption Component Dec 13, 2019
@geriux geriux requested a review from mkevins December 16, 2019 11:06
@geriux geriux marked this pull request as ready for review December 16, 2019 11:07
Gerardo Pacheco added 3 commits December 18, 2019 11:17
…e into refactor/caption-block

# Conflicts:
#	gutenberg
…e into refactor/caption-block

# Conflicts:
#	gutenberg
Copy link
Contributor

@mkevins mkevins left a comment

Choose a reason for hiding this comment

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

Tested via steps described here: WordPress/gutenberg#19118

I had to locally merge with latest (gutenberg:master and gutenberg-mobile:develop) to the respective branches for these PRs due to an intermittent dependency issue, but after that, everything is working nicely. Nice work @geriux 👍 . After updating to latest, LGTM!

@geriux
Copy link
Contributor Author

geriux commented Jan 15, 2020

Closing since this merged PR #1771 updates the same Gutenberg reference

@geriux geriux closed this Jan 15, 2020
@geriux geriux deleted the refactor/caption-block branch January 15, 2020 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor Caption component to make it available for Gallery tiles
2 participants