Fix block appender placeholder not matching the paragraph placeholder issue#732
Merged
Fix block appender placeholder not matching the paragraph placeholder issue#732
Conversation
5 tasks
etoledom
approved these changes
Mar 12, 2019
Contributor
etoledom
left a comment
etoledom
left a comment
There was a problem hiding this comment.
Working great! 🎉
Tested via:
WordPress/gutenberg#14386 (review)
pinarol
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes: #637
gutenberg PR WordPress/gutenberg#14386
^^^ The issue contains more than 1 issues.
the border makes the text go up/down 1px when it gets and loses focus -> This was already solved with Add border on Title when focused #622
When we create a new post block appender's placeholder shows at a slightly different position than the paragraph placeholder so it moves the first time you tap on it. -> This is the problem we are solving in this PR
Before:
iOS:
Android:
After:
iOS:
Android:
TEST
For WPiOS
Checkout the PRs branch to any arbitrary folder and cd .. to it
run yarn install, yarn start
Open XCode WPiOS on the latest develop
Clean build folder on Xcode, and then run the app
For WPAndroid
open grade.properties at WordPress-Android folder
add wp.BUILD_GUTENBERG_FROM_SOURCE = true to grade.properties
checkout the PRs branch in the subrepo of WordPress-Android repo
cd to WordPress-Android/libs/gutenberg-mobile
run yarn install, yarn start
yarn wpandroid on a separate terminal in the same directory
Test Steps