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

fix(uilabel): line height setter should not break line break mode by manoldonev · Pull Request #5544 · NativeScript/NativeScript · GitHub

fix(uilabel): line height setter should not break line break mode - #5544

Merged
manoldonev merged 1 commit into
masterfrom
hettiger/uilabel-line-break-mode-fix
Mar 15, 2018
Merged

manoldonev merged 1 commit into
masterfrom
hettiger/uilabel-line-break-mode-fix

Conversation

Copy link
Copy Markdown
Contributor

NOTE: Cherry-picked from #5251 due to CI infrastructure issues with the original PR.

When setting line-height using a stylesheet, line break mode will stop working.

Working line break mode:
https://play.nativescript.org/?template=play-ng&id=3HPeu8

Now adding a style to home.component.css:

Label {
    line-height: 1;
}

Results in broken line break mode:
https://play.nativescript.org/?template=play-ng&id=3HPeu8&v=5

My Pull Request fixes this behavior.

Copy link
Copy Markdown
Contributor Author

👍 for PR approval.

manoldonev changed the title fix(line Line-height property breaks line break mode (iOS) fix(uilabel): line height setter should not break line break mode Mar 15, 2018
manoldonev merged commit 75bd1d2 into master Mar 15, 2018
ghost removed the in progress label Mar 15, 2018
manoldonev deleted the hettiger/uilabel-line-break-mode-fix branch March 15, 2018 14:21

lock Bot commented Aug 26, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock Bot locked and limited conversation to collaborators Aug 26, 2019
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL