Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Android] Fixes HorizontalTextAlignment not being applied to TextView Elements #2743

Merged
merged 1 commit into from
May 17, 2018

Conversation

PureWeen
Copy link
Contributor

@PureWeen PureWeen commented May 16, 2018

Description of Change

Still set the TextView.Gravity property based on the Horizontal Alignment

In my testing just setting TextAlignment was unreliable. For example in the ControlGallery project it works fine but if you create a new XF project and try to set the HorizontalTextAlignment there on an Entry field it wouldn't work.

In the latter case whenever the View.TextAlignment was set to Center android was just reverting it back to Gravity.

Caused by #1222.

Bugs Fixed

fixes #2167

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of the target branch at time of PR
  • Changes adhere to coding standard
  • Consolidate commits as makes sense

@PureWeen
Copy link
Contributor Author

build --uitests

1 similar comment
@PureWeen
Copy link
Contributor Author

build --uitests

@samhouts samhouts added this to In Review in v3.6.0 May 16, 2018
@samhouts samhouts changed the title Fixes HorizontalTextAlignment not being applied to TextView Elements [Android] Fixes HorizontalTextAlignment not being applied to TextView Elements May 16, 2018
@PureWeen
Copy link
Contributor Author

build --uitests

@PureWeen
Copy link
Contributor Author

The tests all passed in app center even though above is not reporting it as such

@samhouts samhouts added the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label May 16, 2018
Copy link
Member

@samhouts samhouts left a comment

Choose a reason for hiding this comment

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

Excellent. Needs rebase to 3.0.0. 👍

@PureWeen PureWeen changed the base branch from master to 3.0.0 May 17, 2018 00:24
@PureWeen PureWeen added 3.0.0 p/Android and removed retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. labels May 17, 2018
@samhouts samhouts removed this from In Review in v3.6.0 May 17, 2018
@samhouts samhouts added this to In Review in v3.0.0 May 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
v3.0.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants