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

Disable lists. #14844

Merged
Merged

Conversation

SergioEstevao
Copy link
Contributor

Description

Disable lists for release v1.2 until we fix some bug we found.

How has this been tested?

Screenshots

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.

@SergioEstevao SergioEstevao added the Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) label Apr 5, 2019
@SergioEstevao SergioEstevao added this to the 5.5 (Gutenberg) milestone Apr 5, 2019
@SergioEstevao SergioEstevao requested review from hypest and mzorz April 5, 2019 16:19
@hypest hypest removed their request for review April 5, 2019 16:28
Copy link
Contributor

@mzorz mzorz left a comment

Choose a reason for hiding this comment

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

LGTM

@SergioEstevao SergioEstevao merged commit 7b89592 into rnmobile/release-v1.2 Apr 5, 2019
@SergioEstevao SergioEstevao deleted the rnmobile/disable_lists_for_v1.2 branch April 5, 2019 16:35
hypest added a commit that referenced this pull request Apr 9, 2019
* Disable lists. (#14844)

* Enable the list block support on native mobile
etoledom added a commit that referenced this pull request Apr 23, 2019
* Adding accessibility properties to MediaPlaceholder and ImageEdit

* Fix lint issues

* Fix typo

* Better accessibility labels

* Disable lists. (#14844)

* Fix scroll with VoiceOver ON.

For some reason, VoiceOver wouldn't scroll down the block list.
The breaking change was located at the removal of this css property `font-family: $default-regular-font;` on the nextpage block.

Addinb back that property to be able to scroll the blocks list again using VoiceOver.

* Setting accessibilityStates properly to avoid Android crash

* Remove vscode settings file (added by mistake)

* Fix lint issues

* Update accessibility labels to use sprintf

* Fix lint issues
hypest pushed a commit that referenced this pull request May 2, 2019
* Adding accessibility properties to MediaPlaceholder and ImageEdit

* Fix lint issues

* Fix typo

* Better accessibility labels

* Disable lists. (#14844)

* Fix scroll with VoiceOver ON.

For some reason, VoiceOver wouldn't scroll down the block list.
The breaking change was located at the removal of this css property `font-family: $default-regular-font;` on the nextpage block.

Addinb back that property to be able to scroll the blocks list again using VoiceOver.

* Setting accessibilityStates properly to avoid Android crash

* Remove vscode settings file (added by mistake)

* Fix lint issues

* Update accessibility labels to use sprintf

* Fix lint issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants