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

[RNMobile] Fix jumping toolbar #23684

Merged
merged 6 commits into from
Jul 21, 2020
Merged

Conversation

lukewalczak
Copy link
Member

@lukewalczak lukewalczak commented Jul 3, 2020

Description

Fix jumping toolbar after closing the LinkSettings when Button/Paragraph is focused

Fixes: wordpress-mobile/gutenberg-mobile#2301

How has this been tested?

  1. Add a Button
  2. Click on Button block settings
  3. Focus on URL field
  4. Close the Bottom sheet by tapping outside of it
  5. Expect: toolbar mustn't jump

Test as well Buttons-1-2-3

Screenshots

iphone 8 iphone 11
jump_toolbar_ok keyboard_iX_2

Types of changes

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR.

@lukewalczak lukewalczak force-pushed the rnmobile/fix-jumping-toolbar branch from 9f314af to b01f218 Compare July 6, 2020 07:52
@github-actions
Copy link

github-actions bot commented Jul 6, 2020

Size Change: 0 B

Total Size: 1.15 MB

ℹ️ View Unchanged
Filename Size Change
build/a11y/index.min.js 1.14 kB 0 B
build/annotations/index.min.js 3.67 kB 0 B
build/api-fetch/index.min.js 3.43 kB 0 B
build/autop/index.min.js 2.82 kB 0 B
build/blob/index.min.js 620 B 0 B
build/block-directory/index.min.js 7.63 kB 0 B
build/block-directory/style-rtl.css 944 B 0 B
build/block-directory/style.css 945 B 0 B
build/block-editor/index.min.js 124 kB 0 B
build/block-editor/style-rtl.css 10.8 kB 0 B
build/block-editor/style.css 10.8 kB 0 B
build/block-library/editor-rtl.css 7.58 kB 0 B
build/block-library/editor.css 7.57 kB 0 B
build/block-library/index.min.js 132 kB 0 B
build/block-library/style-rtl.css 7.77 kB 0 B
build/block-library/style.css 7.77 kB 0 B
build/block-library/theme-rtl.css 728 B 0 B
build/block-library/theme.css 729 B 0 B
build/block-serialization-default-parser/index.min.js 1.88 kB 0 B
build/block-serialization-spec-parser/index.min.js 3.1 kB 0 B
build/blocks/index.min.js 48.3 kB 0 B
build/components/index.min.js 200 kB 0 B
build/components/style-rtl.css 15.6 kB 0 B
build/components/style.css 15.6 kB 0 B
build/compose/index.min.js 9.67 kB 0 B
build/core-data/index.min.js 11.5 kB 0 B
build/data-controls/index.min.js 1.29 kB 0 B
build/data/index.min.js 8.45 kB 0 B
build/date/index.min.js 5.38 kB 0 B
build/deprecated/index.min.js 772 B 0 B
build/dom-ready/index.min.js 568 B 0 B
build/dom/index.min.js 3.23 kB 0 B
build/edit-navigation/index.min.js 10.8 kB 0 B
build/edit-navigation/style-rtl.css 1.08 kB 0 B
build/edit-navigation/style.css 1.08 kB 0 B
build/edit-post/index.min.js 304 kB 0 B
build/edit-post/style-rtl.css 5.61 kB 0 B
build/edit-post/style.css 5.61 kB 0 B
build/edit-site/index.min.js 16.8 kB 0 B
build/edit-site/style-rtl.css 3.06 kB 0 B
build/edit-site/style.css 3.06 kB 0 B
build/edit-widgets/index.min.js 9.35 kB 0 B
build/edit-widgets/style-rtl.css 2.45 kB 0 B
build/edit-widgets/style.css 2.45 kB 0 B
build/editor/editor-styles-rtl.css 537 B 0 B
build/editor/editor-styles.css 539 B 0 B
build/editor/index.min.js 45.1 kB 0 B
build/editor/style-rtl.css 3.78 kB 0 B
build/editor/style.css 3.78 kB 0 B
build/element/index.min.js 4.65 kB 0 B
build/escape-html/index.min.js 733 B 0 B
build/format-library/index.min.js 7.72 kB 0 B
build/format-library/style-rtl.css 547 B 0 B
build/format-library/style.css 548 B 0 B
build/hooks/index.min.js 2.13 kB 0 B
build/html-entities/index.min.js 621 B 0 B
build/i18n/index.min.js 3.56 kB 0 B
build/is-shallow-equal/index.min.js 711 B 0 B
build/keyboard-shortcuts/index.min.js 2.51 kB 0 B
build/keycodes/index.min.js 1.94 kB 0 B
build/list-reusable-blocks/index.min.js 3.12 kB 0 B
build/list-reusable-blocks/style-rtl.css 476 B 0 B
build/list-reusable-blocks/style.css 476 B 0 B
build/media-utils/index.min.js 5.32 kB 0 B
build/notices/index.min.js 1.79 kB 0 B
build/nux/index.min.js 3.4 kB 0 B
build/nux/style-rtl.css 671 B 0 B
build/nux/style.css 668 B 0 B
build/plugins/index.min.js 2.56 kB 0 B
build/primitives/index.min.js 1.4 kB 0 B
build/priority-queue/index.min.js 789 B 0 B
build/redux-routine/index.min.js 2.85 kB 0 B
build/rich-text/index.min.js 13.9 kB 0 B
build/server-side-render/index.min.js 2.71 kB 0 B
build/shortcode/index.min.js 1.7 kB 0 B
build/token-list/index.min.js 1.27 kB 0 B
build/url/index.min.js 4.06 kB 0 B
build/viewport/index.min.js 1.85 kB 0 B
build/warning/index.min.js 1.14 kB 0 B
build/wordcount/index.min.js 1.17 kB 0 B

compressed-size-action

@lukewalczak lukewalczak marked this pull request as ready for review July 6, 2020 08:54
@lukewalczak lukewalczak requested review from dratwas and geriux July 13, 2020 13:25
Copy link
Member

@geriux geriux left a comment

Choose a reason for hiding this comment

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

LGTM! Tested it on both Android and iOS. Nice work!

Copy link
Contributor

@dratwas dratwas left a comment

Choose a reason for hiding this comment

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

GOOD job Luke!

@lukewalczak lukewalczak force-pushed the rnmobile/fix-jumping-toolbar branch 2 times, most recently from 22cf42d to 3eed636 Compare July 16, 2020 14:57
@pinarol
Copy link
Contributor

pinarol commented Jul 21, 2020

hey Luke 👋 is this the same issue? wordpress-mobile/gutenberg-mobile#2301
if yes please add it to the description so the automation can pick up when this is merged.

@lukewalczak lukewalczak merged commit d86cd5f into master Jul 21, 2020
@lukewalczak lukewalczak deleted the rnmobile/fix-jumping-toolbar branch July 21, 2020 16:16
@github-actions github-actions bot added this to the Gutenberg 8.7 milestone Jul 21, 2020
chipsnyder added a commit that referenced this pull request Aug 5, 2020
chipsnyder added a commit that referenced this pull request Aug 5, 2020
ceyhun pushed a commit that referenced this pull request Aug 6, 2020
cameronvoell added a commit that referenced this pull request Aug 11, 2020
* Release script: Update react-native-editor version to 1.34.0

* [RNMobile] E2E Android - Use swipe gesture to scroll inserter menu (#24338)

* Set autosaveInterval to 1 on mobile (#24353)

* Revert "[RNMobile] Fix jumping toolbar (#23684)" (#24388)

This reverts commit d86cd5f.

* Turn off autosave interval for mobile (#24415)

* [RNMobile] Merge release 1.33.1 to master (#24448)

* Release script: Update react-native-editor version to 1.33.0

* Update release notes.

* Update version numbers for packages.

* Revert "Reduce spacing between label and slider control (#23580)" (#24109)

This reverts commit 2a67de0.

* Release script: Update react-native-* versions to 1.33.1

* Release script: Update with changes from 'npm run core preios'

* Allow @ char in posts in mobile editor (#24428)

This fixes a bug with the @-mention feature on mobile that made entering the standalone @ character difficult because of the @-mention UI getting in the way.
The fix here is to allow the @ character to be intercepted without being consumed by the @ key event logic.

* Remove duplicate entry from changelog

Co-authored-by: Sergio Estevao <[email protected]>
Co-authored-by: Antonis Lilis <[email protected]>
Co-authored-by: Ceyhun Ozugur <[email protected]>
Co-authored-by: Paul Von Schrottky <[email protected]>

* Update version numbers

* Ran pod install to update podfile.lock

Co-authored-by: Drapich Piotr <[email protected]>
Co-authored-by: Adam Zielinski <[email protected]>
Co-authored-by: Chip <[email protected]>
Co-authored-by: Cameron Voell <[email protected]>
Co-authored-by: Sergio Estevao <[email protected]>
Co-authored-by: Antonis Lilis <[email protected]>
Co-authored-by: Paul Von Schrottky <[email protected]>
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.

Button: Toolbar goes up and down for a split second when dismissing settings
4 participants