Skip to content

Commit 5379c9c

Browse files
committed
LightboxHeader: Center "x" button vertically.
That is, center it in the content area of the header, not including the space in the top inset. Like what the three-dots icon in the footer does. The `alignSelf` property was effectively being ignored; now, it's not.
1 parent d6c3be2 commit 5379c9c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/lightbox/LightboxHeader.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,11 @@ const styles = createStyleSheet({
2424
color: 'white',
2525
fontSize: 12,
2626
},
27+
rightIconTouchTarget: {
28+
alignSelf: 'center',
29+
},
2730
rightIcon: {
2831
fontSize: 28,
29-
alignSelf: 'center',
3032
},
3133
wrapper: {
3234
flex: 1,
@@ -71,7 +73,7 @@ export default class LightboxHeader extends PureComponent<Props> {
7173
{subheader}
7274
</Text>
7375
</View>
74-
<Touchable onPress={onPressBack}>
76+
<Touchable style={styles.rightIconTouchTarget} onPress={onPressBack}>
7577
<Icon style={styles.rightIcon} color="white" name="x" />
7678
</Touchable>
7779
</View>

0 commit comments

Comments
 (0)