From c50de27763679d524d7b82bbd5668bf0d266c89e Mon Sep 17 00:00:00 2001
From: Chris Bobbe <cbobbe@zulip.com>
Date: Fri, 29 Jan 2021 17:20:33 -0500
Subject: [PATCH] lightbox header/footer: Use `size` prop.

Greg points out [1] that it doesn't really make sense to set the
size with `fontSize`.

[1] https://github.com/zulip/zulip-mobile/pull/4442#discussion_r567110710
---
 src/lightbox/LightboxFooter.js | 3 +--
 src/lightbox/LightboxHeader.js | 5 +----
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/src/lightbox/LightboxFooter.js b/src/lightbox/LightboxFooter.js
index 5551ae5becb..45af3da655c 100644
--- a/src/lightbox/LightboxFooter.js
+++ b/src/lightbox/LightboxFooter.js
@@ -21,7 +21,6 @@ const styles = createStyleSheet({
     alignSelf: 'center',
   },
   icon: {
-    fontSize: 28,
     alignSelf: 'center',
   },
 });
@@ -39,7 +38,7 @@ export default class LightboxFooter extends PureComponent<Props> {
       <SafeAreaView mode="padding" edges={['right', 'bottom', 'left']}>
         <View style={[styles.wrapper, style]}>
           <Text style={styles.text}>{displayMessage}</Text>
-          <Icon style={styles.icon} color="white" name="more-vertical" onPress={onOptionsPress} />
+          <Icon size={28} color="white" name="more-vertical" onPress={onOptionsPress} />
         </View>
       </SafeAreaView>
     );
diff --git a/src/lightbox/LightboxHeader.js b/src/lightbox/LightboxHeader.js
index 96720beb0a4..1a75fe2b4e6 100644
--- a/src/lightbox/LightboxHeader.js
+++ b/src/lightbox/LightboxHeader.js
@@ -27,9 +27,6 @@ const styles = createStyleSheet({
   rightIconTouchTarget: {
     alignSelf: 'center',
   },
-  rightIcon: {
-    fontSize: 28,
-  },
   wrapper: {
     flex: 1,
     flexDirection: 'row',
@@ -74,7 +71,7 @@ export default class LightboxHeader extends PureComponent<Props> {
             </Text>
           </View>
           <Touchable style={styles.rightIconTouchTarget} onPress={onPressBack}>
-            <Icon style={styles.rightIcon} color="white" name="x" />
+            <Icon size={28} color="white" name="x" />
           </Touchable>
         </View>
       </SafeAreaView>