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-15515: Android: Add support for shadowOffset, shadowColor and shadowRadius on Label & Button #4811

Merged
merged 2 commits into from Oct 22, 2013

Conversation

pingwang2011
Copy link
Contributor

@ghost ghost assigned hieupham007 Oct 19, 2013
- name: shadowRadius
summary: Radius of the shadow.
type: Number
default: 0
Copy link
Contributor

Choose a reason for hiding this comment

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

The default isn't 0 with this PR. The default is 'undefined'. To make default 0, you need to add it to defaultValues map, either in proxy's constructor, or in handleCreationArgs().

@hieupham007
Copy link
Contributor

Code reviewed. Please address comments.

/**
* @module.api
*/
public static final String PROPERTY_SHADOW_COLOR = "shadowColor";
Copy link
Contributor

Choose a reason for hiding this comment

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

this should place before shadow_offset :)

@bhatfield
Copy link
Contributor

Reviewed. Ran validate and docgen. Left two comments.

@pingwang2011
Copy link
Contributor Author

Updated. Ready for review.

@hieupham007
Copy link
Contributor

Code reviewed and functionally tested. Request accepted. Waiting for doc's approval

@bhatfield
Copy link
Contributor

DR approved.

hieupham007 added a commit that referenced this pull request Oct 22, 2013
timob-15515: Android: Add support for shadowOffset, shadowColor and shadowRadius on Label & Button
@hieupham007 hieupham007 merged commit 35140c1 into tidev:master Oct 22, 2013
@pingwang2011 pingwang2011 deleted the timob-15515 branch December 3, 2013 18:33
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