Skip to content

1264 - Activate Preformatted block#1442

Merged
SergioEstevao merged 68 commits intorelease/1.17from
issue/1264_activate_pre_format_block
Nov 13, 2019
Merged

1264 - Activate Preformatted block#1442
SergioEstevao merged 68 commits intorelease/1.17from
issue/1264_activate_pre_format_block

Conversation

@SergioEstevao
Copy link
Copy Markdown
Contributor

@SergioEstevao SergioEstevao commented Oct 11, 2019

Fixes #1264

GB PR: WordPress/gutenberg#17906
AztecEditor-Android PR: wordpress-mobile/AztecEditor-Android#869
WP-Android PR: wordpress-mobile/WordPress-Android#10764
WP-iOS PR: wordpress-mobile/WordPress-iOS#12934

This PR add the preformated block to the list of available blocks.

Pre-light-dark

To make this work some changes were needed in Aztec and on the refreshFont code to be able to handle scenarios were the active font is not the default font, because pre elements need to use a monospace font.

At the moment this PR also needs an Android Dev to make sure that the pre element is formatted correctly on Android.

To test:

  • Open the demo app
  • Add an PreFormatted block to the content
  • Write around on it
  • Try to merge it with other blocks

Update release notes:

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

marecar3 and others added 30 commits September 30, 2019 13:14
Increment gb-mobile version to 1.14.0
Update bundle files
# Conflicts:
#	package.json
…pens_safari

Don't allow interactions with URL in Aztec text blocks.
# Conflicts:
#	gutenberg
…Content-is-missing-on-long-posts-in-html-mode

Fix/1268 content is missing on long posts in html mode
# Conflicts:
#	bundle/android/App.js
#	bundle/android/App.js.map
#	bundle/ios/App.js
#	bundle/ios/App.js.map
#	gutenberg
#	react-native-aztec/ios/Cartfile
#	react-native-aztec/ios/Cartfile.resolved
#	react-native-gutenberg-bridge/android/src/main/java/org/wordpress/mobile/WPAndroidGlue/WPAndroidGlueCode.java
@SergioEstevao
Copy link
Copy Markdown
Contributor Author

@daniloercoli and @etoledom do you mind review this PR?

@SergioEstevao
Copy link
Copy Markdown
Contributor Author

@daniloercoli and @etoledom do you mind to give it another look, now that all the issues are sorted.

# Conflicts:
#	gutenberg
#	react-native-aztec/ios/RNTAztecView/RCTAztecView.swift
Copy link
Copy Markdown
Member

@koke koke left a comment

Choose a reason for hiding this comment

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

Working well on iOS

@SergioEstevao SergioEstevao changed the base branch from develop to release/1.17 November 11, 2019 22:55
Copy link
Copy Markdown
Contributor

@daniloercoli daniloercoli left a comment

Choose a reason for hiding this comment

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

LGTM!

@SergioEstevao SergioEstevao merged commit 3c5051c into release/1.17 Nov 13, 2019
@SergioEstevao SergioEstevao deleted the issue/1264_activate_pre_format_block branch November 13, 2019 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Aztec Parity Feature exists in Aztec but not yet in Gutenberg mobile

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants