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

[TIMOB-23140] Android: Add lineSpacing to Label #7909

Merged
merged 4 commits into from Apr 6, 2016
Merged

[TIMOB-23140] Android: Add lineSpacing to Label #7909

merged 4 commits into from Apr 6, 2016

Conversation

m1ga
Copy link
Contributor

@m1ga m1ga commented Apr 2, 2016

@hansemannn hansemannn changed the title [AC-3449] Android: Add lineSpacing to Label (work in progress) [TIMOB-23140] Android: Add lineSpacing to Label (work in progress) Apr 2, 2016
@m1ga m1ga changed the title [TIMOB-23140] Android: Add lineSpacing to Label (work in progress) [TIMOB-23140] Android: Add lineSpacing to Label Apr 2, 2016
@@ -707,6 +707,21 @@
/**
* @module.api
*/
public static final String PROPERTY_LINE_SPACING = "lineSpacing";
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add the Strings in alphabetical order?

@ashcoding
Copy link
Contributor

Everything looks good, just would like the TiC properties to be arranged alphabetically. :) Thank you @m1ga

@m1ga
Copy link
Contributor Author

m1ga commented Apr 5, 2016

sure, its below property_lines now

@hansemannn
Copy link
Collaborator

Approved! Thank you @m1ga, great work! 🚀

@hansemannn hansemannn merged commit fe46c60 into tidev:master Apr 6, 2016
@m1ga m1ga deleted the line_spacing branch May 27, 2016 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants