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

Don't overwrite or fetch font when already loaded #700

Draft
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

cameronvoell
Copy link

@cameronvoell cameronvoell commented Jun 1, 2021

Fixes #676

Related WPAndroid PR: wordpress-mobile/WordPress-Android#14774

Steps to test:

  1. use WPAndroid and create a Story from the My Site screen -> FAB -> Add new Story post
  2. select one back ground image to add to your Story slide
  3. once in the composer, add a Text by tapping the "A" button
  4. in the Text Editor, choose a font other than the default one. For example, use STRONG.
  5. add some text, optionally resize it if you'd like to make it more manageable. Finally tap on the checkmark on the top-right corner of the screen.
  6. Now tap on the -> arrow on the top-right of the screen. The pre-publishing bottom sheet will appear. Optionally give your Story post a title, and tap on PUBLISH NOW
  7. After your Story is published, go to Posts list (Published) and find it first in the list
  8. tap to open
  9. once Gutenberg is loaded and shows the Story block, tap on it twice (one to select the block, one to trigger the Story composer)
  10. observe the added text does not respect the font you chose in step 4 above.
  11. tapping the text to edit will however show the right font selection, and tapping on the checkmark will now correctly update the added view with the originally selected font.

@peril-automattic
Copy link

You can test the changes on this Pull Request by downloading the APK here.

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.

Text Editing: loading from a saved Story in prefs does not initialize the added views correctly
1 participant