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-14007] set font property #5198

Conversation

bijupmb
Copy link
Contributor

@bijupmb bijupmb commented Jan 9, 2014

@@ -147,6 +150,10 @@ public void processProperties(KrollDict d) {
numericMonths = TiConvert.toBoolean(d, "numericMonths");
}

if (d.containsKeyStartingWith("font")) {
setFontProperties();
Copy link
Contributor

Choose a reason for hiding this comment

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

indentation

@hieupham007
Copy link
Contributor

Code reviewed. Please address comments.

@bijupmb
Copy link
Contributor Author

bijupmb commented Feb 4, 2014

will check it

String fontWeight = null;
Typeface typeface = null;
KrollDict d = proxy.getProperties();
// KrollDict d = new KrollDict();
Copy link
Contributor

Choose a reason for hiding this comment

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

remove unused code

@hieupham007
Copy link
Contributor

Code reviewed. Please address comments.

@negupta
Copy link
Contributor

negupta commented Apr 17, 2014

@hieupham007 Your review comments were addressed over 2 weeks ago. Please review the PR and bring it to a closure.

@bijupmb
Copy link
Contributor Author

bijupmb commented Apr 18, 2014

Already modified
document and code changed. Please see just bellow the comment code reviewed .

@negupta
Copy link
Contributor

negupta commented Apr 28, 2014

@hieupham007 - Please review this PR.

@hieupham007
Copy link
Contributor

Code reviewed and functionally tested. Request accepted

hieupham007 added a commit that referenced this pull request Apr 29, 2014
…tyForPickerAndPickerColmn

[TIMOB-14007] set font property
@hieupham007 hieupham007 merged commit 6d75d2b into tidev:master Apr 29, 2014
farfromrefug pushed a commit to Akylas/titanium_mobile that referenced this pull request Aug 15, 2014
…ropertyForPickerAndPickerColmn

[TIMOB-14007] set font property
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

3 participants