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-3408] iOS: Support justified text-align in Labels, Buttons and TextFields #8562

Merged
merged 2 commits into from Nov 7, 2016

Conversation

hansemannn
Copy link
Collaborator

@hansemannn hansemannn added this to the 6.1.0 milestone Oct 30, 2016
Copy link
Contributor

@sgtcoolguy sgtcoolguy left a comment

Choose a reason for hiding this comment

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

Also, it'd be good to add/edit a unit test to validate that the new constant is a valid value to assign to these properties. (i.e. assign it and check the getter? Validate the constant lists and is actually a constant (not writable)/has the value we expect?)

@@ -412,26 +412,9 @@ -(void)setVerticalAlign_:(id)align
-(void)setTextAlign_:(id)align
{
button = [self button];
NSTextAlignment alignment = NSTextAlignmentNatural;
Copy link
Contributor

Choose a reason for hiding this comment

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

Note that this actually does change behavior here. If the user specified null/undefined/ or a bad int value, we'd default to NSTextAlignmentNatural, while we can no longer specify any value to get that setting now. We may want to expose a "natural" constant value as well?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Agree. I'd rather change this constant to NSTextAlignmentNatural though. They look the same, but NSTextAlignmentNatural is the native way to go and default since iOS 6.

Copy link
Contributor

@vijaysingh-axway vijaysingh-axway left a comment

Choose a reason for hiding this comment

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

FT and CR passed .

@sgtcoolguy sgtcoolguy merged commit 5a824ed into tidev:master Nov 7, 2016
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