Skip to content

Disable lists.#14844

Merged
SergioEstevao merged 1 commit intornmobile/release-v1.2from
rnmobile/disable_lists_for_v1.2
Apr 5, 2019
Merged

Disable lists.#14844
SergioEstevao merged 1 commit intornmobile/release-v1.2from
rnmobile/disable_lists_for_v1.2

Conversation

@SergioEstevao
Copy link
Copy Markdown
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
Copy Markdown
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