Skip to content
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

fix(TabViewItemDirective ): textTransform property added #1315

Merged
merged 6 commits into from
May 21, 2018

Conversation

bhavincb
Copy link
Contributor

@bhavincb bhavincb commented May 7, 2018

PR Checklist

What is the current behavior?

currently TabViewItemDirective only accepts title and iconSource as input not textTransform

What is the new behavior?

added textTransform property to TabViewItemDirective

Fixes/Implements/Closes #[Issue Number].
#1312

textTransform property added to TabViewItemDirective so that in angular tabItem textTransformation can be changed.
textTransform property to TabViewItemDirective
@ghost ghost added the ♥ community PR label May 7, 2018
removed tailing whitespace  which was causing build failed.
@bhavincb bhavincb changed the title Adding textTransform property to TabViewItemDirective fix(TabViewItemDirective ): textTransform property added May 8, 2018
@ghost ghost assigned MartoYankov May 17, 2018
@ghost ghost added in progress and removed ♥ community PR labels May 17, 2018
@bhavincb
Copy link
Contributor Author

unable to see details of failed check.

@MartoYankov
Copy link
Contributor

@bhavincb Some tests failed that don't seem to have anything to do with this PR. We are taking a look.

@dtopuzov
Copy link
Contributor

test

@dtopuzov dtopuzov added this to the 6.0 milestone May 21, 2018
@ghost ghost assigned SvetoslavTsenov May 21, 2018
@SvetoslavTsenov
Copy link
Contributor

test

@SvetoslavTsenov SvetoslavTsenov merged commit 11d01f9 into NativeScript:master May 21, 2018
@ghost ghost removed the in progress label May 21, 2018
sis0k0 pushed a commit that referenced this pull request May 23, 2018
An issue came out of #1315. TabViewItem textTransform has a default value for Android that kick in only if no value (even a null value) is set.

Note: Will need to change an image in the ui tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants