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

Update Aztec-Android for alignment handling #2006

Merged
merged 1 commit into from
Apr 24, 2020

Conversation

mchowning
Copy link
Contributor

@mchowning mchowning commented Mar 12, 2020

Description

Enables alignment on Android for non <p>-based blocks. In particular,

  1. Enables alignment for Heading blocks on Android (was already working on iOS);
  2. Applies center alignment for the Pullquote block on Android (was already working on iOS) Pullquote block #1854; and
  3. Enables alignment for the Verse block (the option was there before, but it didn't work) Enable alignment toolbar for verse block #2037;
  4. Enables alignment for the quote block on both platforms (alignment button was present, and it would update the HTML, but visually the block did not display the selected alignment) Quote Block display does not respect alignment #2161

⚠️ Once the AztecEditor-Android PR is merged, the aztecVersion in this PR must be updated before this is merged.

Related PRs

wordpress-mobile/AztecEditor-Android#899
WordPress/gutenberg#21803
wordpress-mobile/WordPress-Android#11752

Testing

These tests should all be performed on both iOS and Android.

Heading, Verse, and Quote Blocks

[Heading, Verse, Quote].forEach { block ->

  1. Select a ^block^
  2. Change the alignment for the block
  3. Observe the text visually updates to reflect the new alignment
  4. Verify that the HTML is updated with the new alignment

}

Pullquote Block

  1. Verify that the text in the Pullquote block has center alignment

Regressions

Verify that:

  • Alignment still works on Paragraph blocks
  • Image and video captions are still centrally aligned

PR submission checklist

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@marecar3
Copy link
Contributor

We're freezing 1.24.0 today. If you think it is ready and if you'd like to ship it with 1.24.0please merge it into develop branch. Otherwise, we can bump it to 1.25.0. @mchowning

@mchowning mchowning modified the milestones: 1.24, 1.25 Mar 17, 2020
@SergioEstevao SergioEstevao self-assigned this Mar 25, 2020
@SergioEstevao
Copy link
Contributor

@mchowning I still have hopes that you can have this available on 1.25, but if not please move it to the 1.26 milestone.

@mchowning mchowning modified the milestones: 1.25, 1.26 Mar 31, 2020
@pinarol pinarol modified the milestones: 1.26, 1.27 Apr 13, 2020
@pinarol
Copy link
Contributor

pinarol commented Apr 13, 2020

I am changing the milestone of this PR as 1.27 because the 1.26 release branch will be cut quite soon. Let me know if you instead wanted to include this to 1.26.

@mchowning mchowning force-pushed the update_aztec_for_alignment branch 4 times, most recently from 719b019 to 6ac77c6 Compare April 23, 2020 18:47
@cameronvoell cameronvoell requested review from mkevins and removed request for mkevins April 24, 2020 02:31
@mchowning mchowning force-pushed the update_aztec_for_alignment branch from 6ac77c6 to 174dac2 Compare April 24, 2020 13:04
Copy link
Contributor

@hypest hypest left a comment

Choose a reason for hiding this comment

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

Looking good code-wise! I still haven't made a pass to test the behavior but I'll LGTM this and someone (or me) can do the testing pass.

@mchowning mchowning force-pushed the update_aztec_for_alignment branch 2 times, most recently from aabf1ce to 90dd372 Compare April 24, 2020 17:55
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.

5 participants