Skip to content

/ gutenberg-mobile

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

Jump to bottom

[iOS] Fix empty rich-text with no height on RTL layout #2544

Merged
merged 2 commits into from Aug 14, 2020
Merged

[iOS] Fix empty rich-text with no height on RTL layout #2544

merged 2 commits into from Aug 14, 2020

Conversation

@etoledom
Copy link
Contributor

@etoledom etoledom commented Aug 12, 2020 •
edited

Fixes #

PR to test WordPress/gutenberg#24510
WPiOS PR: wordpress-mobile/WordPress-iOS#14630

RTL

To test:

  • Run the project from wordpress-mobile/WordPress-iOS#14630.
  • Set the Application Language to a RTL language (Hebrew - Arabic) on the Run Options on Xcode.
  • Open a new empty post.
    • Check that the Title and inserter placeholders are visible.
    • Check that the fields are selectable and you can type on them.
    • Check that longer text makes the text area increase in height as expected.
  • Make the same checks from ⬆️ on a LTR language.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.
@etoledom etoledom mentioned this pull request Aug 12, 2020
Closed
3 of 3 tasks complete
@etoledom etoledom requested a review from guarani Aug 12, 2020
@etoledom etoledom added this to the 1.35 milestone Aug 12, 2020
@etoledom etoledom mentioned this pull request Aug 12, 2020
Merged
6 of 6 tasks complete
@peril-wordpress-mobile
Copy link

@peril-wordpress-mobile peril-wordpress-mobile bot commented Aug 12, 2020 •
edited

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!
@etoledom etoledom marked this pull request as ready for review Aug 12, 2020
@guarani
guarani approved these changes Aug 12, 2020
View changes
Copy link
Contributor

@guarani guarani left a comment

Approved via wordpress-mobile/WordPress-iOS#14630 (review)
@etoledom
Update gutenberg ref
Loading status checks…
f393db0
@etoledom etoledom force-pushed the fix-RTL-lang-empty-rich-text branch from f1bc780 to f393db0 Aug 14, 2020
@ceyhun
Copy link
Member

@ceyhun ceyhun commented Aug 14, 2020

@etoledom heads up we'll be cutting the 1.35 release branch beginning of next week. Please try to get this merged by then or bump the milestone 🙇
@etoledom
Update gutenberg ref
Loading status checks…
04d3509
@etoledom etoledom self-assigned this Aug 14, 2020
@etoledom etoledom merged commit b7fb979 into develop Aug 14, 2020
7 checks passed
7 checks passed
ci/circleci: Android Native Unit Tests Your tests passed on CircleCI!
Details
ci/circleci: Check Correctness Your tests passed on CircleCI!
Details
ci/circleci: Test Android Your tests passed on CircleCI!
Details
ci/circleci: Test Android on Device - Canaries Your tests passed on CircleCI!
Details
ci/circleci: Test iOS Your tests passed on CircleCI!
Details
ci/circleci: Test iOS on Device - Canaries Your tests passed on CircleCI!
Details
ci/circleci: gutenberg-mobile/Optional UI Tests Your job is on hold on CircleCI!
Details
@etoledom etoledom deleted the fix-RTL-lang-empty-rich-text branch Aug 14, 2020
@cameronvoell cameronvoell mentioned this pull request Aug 18, 2020
Merged
5 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guarani guarani

Assignees

@etoledom etoledom

Labels
RTL [OS] iOS
Projects
None yet
Milestone
1.35
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@etoledom @ceyhun @guarani
You can’t perform that action at this time.