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

Add methods to custom background color ... #195

Merged
merged 1 commit into from
Jul 21, 2015

Conversation

ainopara
Copy link

and custom the attributes of text in string picker as an example.
Use them like this:

    ActionSheetStringPicker *picker = [[ActionSheetStringPicker alloc] initWithTitle:@"" rows:array initialSelection:1 doneBlock:nil cancelBlock:nil origin:self.pageLabel];
    picker.pickerBackgroundColor = [UIColor blackColor];
    picker.pickerTextAttributes = @{NSForegroundColorAttributeName: [UIColor whiteColor],
                                    NSFontAttributeName:[UIFont systemFontOfSize:21.0],};
    [picker showActionSheetPicker];

This PR will close #58, fix #102

@skywinder
Copy link
Owner

@ainopara Wow, awesome! Can you also add some example usage of new method to Example project?

@skywinder
Copy link
Owner

This PR will close #58

This was referenced Jul 21, 2015
@ainopara
Copy link
Author

Example updated. It seems custom font attribute will be reset by system.

@ainopara
Copy link
Author

I don't know how to fix the CI build failure this time.

@skywinder
Copy link
Owner

@ainopara yes it's common issue with frameworks. Try to merge my PR. It should help.

@skywinder
Copy link
Owner

@ainopara ok, don't worry, I will handle this failure.

@ainopara
Copy link
Author

^_^
Thanks.

@skywinder
Copy link
Owner

@ainopara Thank you! BDW: fixed according this: http://stackoverflow.com/a/27614914/1698467

skywinder added a commit that referenced this pull request Jul 21, 2015
Add methods to custom background color ...
@skywinder skywinder merged commit cc55b7d into skywinder:master Jul 21, 2015
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.

2 participants