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

[Mobile] Reduce spacing between label and slider control #23580

Merged
merged 1 commit into from
Jul 13, 2020

Conversation

antonis
Copy link
Member

@antonis antonis commented Jun 30, 2020

Fixes wordpress-mobile/gutenberg-mobile#2126

Related PRs:

gutenberg-mobile wordpress-mobile/gutenberg-mobile#2451

Description

Reduces spacing between label and slider control by:

  • 14px on iOS
  • 24px on Android

How has this been tested?

  • Add a block that has as slider control in the settings (e.g. spacer)
  • Press on the settings button
  • Verify that the space between the label and the slider control is appropriate

Screenshots

Android iOS
Screenshot_1593521798 Simulator Screen Shot - iPhone 11 - 2020-06-30 at 15 52 30

Types of changes

Enhancement

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.

@antonis antonis changed the title Reduce spacing between label and slider control [Mobile] Reduce spacing between label and slider control Jul 1, 2020
@pinarol pinarol requested a review from chipsnyder July 8, 2020 09:14
@antonis antonis marked this pull request as ready for review July 8, 2020 09:17
Copy link
Contributor

@chipsnyder chipsnyder left a comment

Choose a reason for hiding this comment

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

This looks good to me from a code perspective. Just pending the design thumbs up

@chipsnyder chipsnyder merged commit 2a67de0 into WordPress:master Jul 13, 2020
@github-actions github-actions bot added this to the Gutenberg 8.6 milestone Jul 13, 2020
antonis added a commit that referenced this pull request Jul 22, 2020
antonis added a commit that referenced this pull request Jul 22, 2020
mchowning pushed a commit that referenced this pull request Jul 23, 2020
* 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.

Co-authored-by: Antonis Lilis <[email protected]>
cameronvoell added a commit that referenced this pull request Aug 7, 2020
* 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]>
cameronvoell added a commit that referenced this pull request Aug 7, 2020
* 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]>
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.

2 participants