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

Release v1.10.0 #1246

Merged
merged 193 commits into from
Jul 26, 2019
Merged

Release v1.10.0 #1246

merged 193 commits into from
Jul 26, 2019

Conversation

Tug
Copy link
Contributor

@Tug Tug commented Jul 26, 2019

This merges release/1.10.0 to master for the next releases of the WPAndroid/WPiOS apps.

Gutenberg branch: WordPress/gutenberg#16775
WPAndroid PR: wordpress-mobile/WordPress-Android#10286
WPiOS PR: wordpress-mobile/WordPress-iOS#12210

JavonDavis and others added 30 commits June 3, 2019 21:24
mchowning and others added 9 commits July 19, 2019 19:41
Address fix for issue where inserting a block from the post title would not
replace an empty paragraph block if that block was the first block in the
post. Instead it would add a second empty paragraph block at the beginning
of the post.
…ial-empty-paragraph-block-from-title

When inserting block from title replace block if appropriate
…on-config

Update vscode extension config to prevent VSCode to show a warning on project opening
Node 9.x seems no longer relevant since we are now directing devs to use
`nvm install --latest-npm` (currently v.12.6.0)
…re-default-tagname

Fix loss of center alignment in image captions on Android
------
* Adding a block from the post title now shows the add block here indicator.
* Deselect post title any time a block is added
* Fix merging of empty paragraph blocks when added from a post's title
Copy link
Contributor

Choose a reason for hiding this comment

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

@Tug In light of your comment (WordPress/gutenberg#16677 (comment)) that we may not want this change
and since we're likely going to lose it with your imminent PR merge, we might want to pull this line out of the release notes. Alternatively, I could revert the entire change from #1227. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If you think we can easily revert it, let's do this. Otherwise keeping it shouldn't impact the user experience too much I think

Copy link
Contributor

@mchowning mchowning Jul 26, 2019

Choose a reason for hiding this comment

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

Reverted

* Adding a block from the post title now shows the add block here indicator.
* Deselect post title any time a block is added
* Fix merging of empty paragraph blocks when added from a post's title
* Fix loss of center alignment in image captions
Copy link
Contributor

Choose a reason for hiding this comment

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

Looking back at this now, seems like I perhaps should have called this out as being an Android-specific issue. I'm happy to add that change if you agree.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -1,3 +1,10 @@
1.10.0
------
* Adding a block from the post title now shows the add block here indicator.
Copy link
Contributor

Choose a reason for hiding this comment

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

There is one final edge case for this (adding from the post title on a new post when there is just the placeholder text and no paragraph block yet) is still not fixed. Should we pull this from the release notes until that is fixed?

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 this is fine as it's included in this release

Copy link
Contributor

Choose a reason for hiding this comment

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

Aww, but that screws up my plan of changing all the release notes. 😉

Copy link
Contributor

@marecar3 marecar3 left a comment

Choose a reason for hiding this comment

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

LGTM! Nice work here :)

@marecar3 marecar3 merged commit 853684a into master Jul 26, 2019
@marecar3 marecar3 deleted the release/1.10.0 branch July 26, 2019 20:00
@mchowning mchowning mentioned this pull request Aug 24, 2019
11 tasks
mchowning added a commit that referenced this pull request Aug 24, 2019
This was previously removed, but somehow snuck back in.

Previous discussion: #1246 (comment)
Previous revert commit: d6a551a
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.