FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(android-label): Match iOS default vertical alignment by Logikgate · Pull Request #8216 · NativeScript/NativeScript · GitHub

fix(android-label): Match iOS default vertical alignment - #8216

Closed
Logikgate wants to merge 1 commit into
NativeScript:masterfrom
flypapertech:fix-android-label-valignment
Closed

Logikgate wants to merge 1 commit into
NativeScript:masterfrom
flypapertech:fix-android-label-valignment

Conversation

Copy link
Copy Markdown

Set gravity to CENTER_VERTICAL on android to match iOS labels which default to vertically centered

Fixes issue #3829

PR Checklist

Playground

This playground displays the issue

What is the current behavior?

Labels on iOS are by default vertically aligned center in all layouts. On Android they are aligned top. Reference issue #3829.

What is the new behavior?

Labels on Android are now vertically aligned center by default

Fixes #3829.

Set gravity to CENTER_VERTICAL on android to match iOS labels which default to vertically centered

Fixes issue NativeScript#3829
cla-bot Bot added the cla: yes label Dec 27, 2019

vakrilov commented Jan 6, 2020

Copy link
Copy Markdown
Contributor

Hey @Logikgate
Thanks for the PR. However, changing the default behavior will probably break the layout and appearance of many apps that are relying on the existing behavior. Such change should be treated as a breaking change and so we cannot safely merge it now.
I think there are other options - let's discuss these in the issue.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vertically center label on android by default

3 participants


Back | FazBrowse Home | New Git URL