From be06a141feea3fba53f9b62e3c23c424da9e18b4 Mon Sep 17 00:00:00 2001
From: linonetwo <linonetwo012@gmail.com>
Date: Tue, 19 Mar 2024 18:21:56 +0800
Subject: [PATCH] fix: button style on ios

---
 src/components/NavigationButtons.tsx | 15 ++++++++++-----
 src/pages/Importer/Index.tsx         | 21 ++++++++++++---------
 2 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/src/components/NavigationButtons.tsx b/src/components/NavigationButtons.tsx
index 0c1498c..9040cf5 100644
--- a/src/components/NavigationButtons.tsx
+++ b/src/components/NavigationButtons.tsx
@@ -1,15 +1,18 @@
+/* eslint-disable react-native/no-inline-styles */
 import { useNavigation } from '@react-navigation/native';
 import { StackScreenProps } from '@react-navigation/stack';
 import { useTranslation } from 'react-i18next';
-import { Button } from 'react-native-paper';
+import { Button, Text } from 'react-native-paper';
 import { styled } from 'styled-components/native';
 import { RootStackParameterList } from '../App';
 
 const MainFeatureButton = styled(Button)`
   margin: 10px;
-  padding: 20px;
-  height: 3em;
+  /* Use height: 3em will cause label to disappear on iOS */
+  min-height: 3em;
 `;
+/** Can't reach the label from button's style-component. Need to defined using `labelStyle`. Can't set padding on button, otherwise padding can't trigger click. */
+const ButtonLabelPadding = 15;
 
 export function ImporterButton() {
   const { t } = useTranslation();
@@ -21,8 +24,9 @@ export function ImporterButton() {
       onPress={() => {
         navigation.navigate('Importer', {});
       }}
+      labelStyle={{ padding: ButtonLabelPadding }}
     >
-      {t('Menu.ScanQRToSync')}
+      <Text>{t('Menu.ScanQRToSync')}</Text>
     </MainFeatureButton>
   );
 }
@@ -37,8 +41,9 @@ export function CreateWorkspaceButton() {
       onPress={() => {
         navigation.navigate('CreateWorkspace');
       }}
+      labelStyle={{ padding: ButtonLabelPadding }}
     >
-      {t('AddWorkspace.AddWorkspace')}
+      <Text>{t('AddWorkspace.AddWorkspace')}</Text>
     </MainFeatureButton>
   );
 }
diff --git a/src/pages/Importer/Index.tsx b/src/pages/Importer/Index.tsx
index 0a5c4eb..85d0040 100644
--- a/src/pages/Importer/Index.tsx
+++ b/src/pages/Importer/Index.tsx
@@ -1,7 +1,7 @@
 /* eslint-disable unicorn/no-nested-ternary */
 /* eslint-disable unicorn/no-useless-undefined */
 import { StackScreenProps } from '@react-navigation/stack';
-import { BarCodeScannedCallback, BarCodeScanner } from 'expo-barcode-scanner';
+import { BarCodeScannedCallback, BarCodeScanner, PermissionStatus } from 'expo-barcode-scanner';
 import React, { FC, useCallback, useEffect, useState } from 'react';
 import { useTranslation } from 'react-i18next';
 import Collapsible from 'react-native-collapsible';
@@ -29,19 +29,19 @@ const LargeBarCodeScanner = styled(BarCodeScanner)`
 `;
 const ImportWikiButton = styled(Button)`
   margin-top: 20px;
-  height: 100px;
+  min-height: 100px;
   display: flex;
   flex-direction: column;
   justify-content: center;
 `;
 const ScanQRButton = styled(Button)`
-  margin: 10px;
-  padding: 20px;
-  height: 3em;
+  margin: 10px 0;
+  min-height: 3em;
 `;
+/** Can't reach the label from button's style-component. Need to defined using `labelStyle`. Can't set padding on button, otherwise padding can't trigger click. */
+const ButtonLabelPadding = 30;
 const OpenWikiButton = styled(Button)`
-  padding: 20px;
-  height: 5em;
+  min-height: 5em;
   margin-top: 5px;
 `;
 const DoneImportActionsTitleText = styled(Text)`
@@ -76,12 +76,12 @@ export const Importer: FC<StackScreenProps<RootStackParameterList, 'Importer'>>
   const [wikiUrl, setWikiUrl] = useState<undefined | URL>(route.params.uri === undefined ? undefined : new URL(new URL(route.params.uri).origin));
   const [serverUriToUseString, setServerUriToUseString] = useState(wikiUrl?.toString() ?? '');
   const [wikiName, setWikiName] = useState('wiki');
-  const [addServer, updateServer] = useServerStore(state => [state.add, state.update]);
+  const [addServer] = useServerStore(state => [state.add, state.update]);
   const addAsServer = route.params.addAsServer ?? true;
   useEffect(() => {
     const getBarCodeScannerPermissions = async () => {
       const { status } = await BarCodeScanner.requestPermissionsAsync();
-      setHasPermission(status === 'granted');
+      setHasPermission(status === PermissionStatus.GRANTED);
     };
 
     void getBarCodeScannerPermissions();
@@ -155,6 +155,7 @@ export const Importer: FC<StackScreenProps<RootStackParameterList, 'Importer'>>
       <ScanQRButton
         mode={importStatus === 'idle' ? 'elevated' : 'outlined'}
         disabled={importStatus !== 'idle'}
+        labelStyle={{ padding: ButtonLabelPadding }}
         onPress={() => {
           setQrScannerOpen(!qrScannerOpen);
         }}
@@ -202,6 +203,7 @@ export const Importer: FC<StackScreenProps<RootStackParameterList, 'Importer'>>
             mode='elevated'
             disabled={importStatus !== 'idle'}
             onPress={addServerAndImport}
+            labelStyle={{ padding: ButtonLabelPadding }}
           >
             <ButtonText>
               {t('Import.ImportWiki')}
@@ -269,6 +271,7 @@ export const Importer: FC<StackScreenProps<RootStackParameterList, 'Importer'>>
               navigation.navigate('MainMenu', { fromWikiID: createdWikiWorkspace.id });
               navigation.navigate('WikiWebView', { id: createdWikiWorkspace.id });
             }}
+            labelStyle={{ padding: ButtonLabelPadding }}
           >
             <Text>{`${t('Open')} ${createdWikiWorkspace.name}`}</Text>
           </OpenWikiButton>