Skip to content

Conversation

@pinarol
Copy link
Contributor

@pinarol pinarol commented Mar 13, 2019

Fix: #671

WPiOS PR: wordpress-mobile/WordPress-iOS#11258

This PR fixes the issue about heading placeholder not adjusting itself due to heading level.

After the fix:
heading-placeholder-size

To Test

Checkout branch on WPiOS PR: wordpress-mobile/WordPress-iOS#11258

  • Add heading block
  • Change heading level on toolbar
  • Verify that placeholder size adjusts accordingly

Copy link
Contributor

@etoledom etoledom left a comment

Choose a reason for hiding this comment

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

Looks and works great! 🎉
Thank you @pinarol for this fix.

@pinarol pinarol merged commit 7cc5710 into develop Mar 13, 2019
@pinarol pinarol deleted the issue/671-heading-placeholder-size branch March 13, 2019 15:24
daniloercoli added a commit that referenced this pull request Mar 14, 2019
…rg-mobile into issue/707-Placeholder-is-missing-on-Heading-block

* 'develop' of https://github.com/wordpress-mobile/gutenberg-mobile: (22 commits)
  Update gutenberg ref
  Update gutenberg ref
  Update gutenberg ref
  Update heading placeholder size on heading level change (#742)
  Fix - HTML Editor - Keyboard can not be closed (#743)
  Update gutenberg ref
  Update gutenberg ref
  Fix lint errors
  Make sure onSafeAreaInsetsUpdate isn't called when BlockManager is unmounted
  Remove extra semicolon
  Remove unused editor settings prop
  Unmount the app after each test to prevent dangling subscribers
  Fix switching to html when initialHtmlModeEnabled is true
  Auto import native versions of variables and colors
  Fix lint errors
  Use @wordpress/edit-post package to bootstrap the editor and refactor components
  Add symlink for @wordpress/token-list
  Minor syntax fixes
  Update tests to use the un deprecated version of the store methods
  Update gutenberg ref
  ...

# Conflicts:
#	gutenberg
daniloercoli added a commit that referenced this pull request Mar 15, 2019
…rg-mobile into issue/345-fix-for-press-delete-no-merge-move-focus

* 'develop' of https://github.com/wordpress-mobile/gutenberg-mobile: (39 commits)
  Update gb hash after merging the companion PR
  Update GB ref
  Update gutenberg ref
  Update gutenberg ref
  Update gutenberg ref
  Tests: Add mock for the newly added CSS class
  Update GB ref
  Raise the minHeight value for heading block to 60 (it was 50)
  Introduce a constant for the string `tag` and use it instead of the hard coded string
  Make `tagName` private
  Update heading placeholder size on heading level change (#742)
  Fix - HTML Editor - Keyboard can not be closed (#743)
  Update gutenberg ref
  Update gutenberg ref
  Update GB ref
  Add comments
  Update caret position after inserting link (#716)
  Update GB hash
  Reuse `blockType` instead of adding a new property
  Update GB ref
  ...

# Conflicts:
#	src/block-management/block-holder.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[OS] iOS [Type] Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[iOS] Heading placeholder won't adjust to heading level.

3 participants