Skip to content

Commit

Permalink
Adjust row height by font size in PickerIOS
Browse files Browse the repository at this point in the history
Summary:
- [x] Explain the **motivation** for making this change.
- [x] Provide a **test plan** demonstrating that the code is solid.
- [x] Match the **code formatting** of the rest of the codebase.
- [x] Target the `master` branch, NOT a "stable" branch.

There is a problem where setting a bigger fontSize in PickerItem style
clips the top and bottom of the text.
This solves that problem by computing the row height using the font
size.

Create a PickerIOS component and set a larger font size (e.g. 50). The row height will grow accordingly.

Example with `fontSize=50`: [Screenshot](http://i.imgur.com/YwK5fOc.png)
Closes #13513

Differential Revision: D5692124

Pulled By: shergin

fbshipit-source-id: 4629403e37ad68cdbc0b17b48ba924a77e133078
  • Loading branch information
alin23 authored and facebook-github-bot committed Aug 24, 2017
1 parent 22d283b commit 28c1c88
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions React/Views/RCTPicker.m
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,10 @@ - (NSString *)pickerView:(__unused UIPickerView *)pickerView
return [RCTConvert NSString:_items[row][@"label"]];
}

- (CGFloat)pickerView:(UIPickerView *)pickerView rowHeightForComponent:(NSInteger)component {
return _font.pointSize + 19;
}

- (UIView *)pickerView:(UIPickerView *)pickerView
viewForRow:(NSInteger)row
forComponent:(NSInteger)component
Expand Down

0 comments on commit 28c1c88

Please sign in to comment.