Skip to content

Commit 57a9c63

Browse files
Merge pull request Expensify#70105 from lorretheboy/fix/69213
Trip Summary - Copy to clipboard not working on Flight/Record locator
2 parents 98e1b02 + f96db60 commit 57a9c63

1 file changed

Lines changed: 16 additions & 1 deletion

File tree

src/pages/Travel/FlightTripDetails.tsx

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,11 +66,15 @@ function FlightTripDetails({reservation, prevReservation, personalDetails}: Flig
6666
description={`${translate('travel.flight')} ${CONST.DOT_SEPARATOR} ${flightDuration}`}
6767
title={`${reservation.company?.longName} ${CONST.DOT_SEPARATOR} ${reservation.route?.airlineCode}`}
6868
copyValue={`${reservation.company?.longName} ${CONST.DOT_SEPARATOR} ${reservation.route?.airlineCode}`}
69+
copyable
70+
interactive={false}
6971
/>
7072
<MenuItemWithTopDescription
7173
description={translate('common.date')}
7274
title={startDate.date}
7375
interactive={false}
76+
copyValue={startDate.date}
77+
copyable
7478
/>
7579

7680
<MenuItemWithTopDescription
@@ -80,6 +84,8 @@ function FlightTripDetails({reservation, prevReservation, personalDetails}: Flig
8084
helperText={`${reservation.start.longName} (${reservation.start.shortName})${reservation.arrivalGate?.terminal ? `, ${reservation.arrivalGate?.terminal}` : ''}`}
8185
helperTextStyle={[styles.pb3, styles.mtn2]}
8286
interactive={false}
87+
copyValue={`${startDate.hour} ${reservation.start.longName} (${reservation.start.shortName})${reservation.arrivalGate?.terminal ? `, ${reservation.arrivalGate?.terminal}` : ''}`}
88+
copyable
8389
/>
8490
<MenuItemWithTopDescription
8591
description={translate('travel.flightDetails.landing')}
@@ -88,6 +94,8 @@ function FlightTripDetails({reservation, prevReservation, personalDetails}: Flig
8894
helperText={`${reservation.end.longName} (${reservation.end.shortName})`}
8995
helperTextStyle={[styles.pb3, styles.mtn2]}
9096
interactive={false}
97+
copyValue={`${endDate.hour} ${reservation.end.longName} (${reservation.end.shortName})`}
98+
copyable
9199
/>
92100

93101
<View style={[styles.flexRow, styles.flexWrap]}>
@@ -97,6 +105,8 @@ function FlightTripDetails({reservation, prevReservation, personalDetails}: Flig
97105
description={translate('travel.flightDetails.seat')}
98106
title={reservation.route?.number}
99107
interactive={false}
108+
copyValue={reservation.route?.number}
109+
copyable={!!reservation.route?.number?.length}
100110
/>
101111
</View>
102112
)}
@@ -106,15 +116,19 @@ function FlightTripDetails({reservation, prevReservation, personalDetails}: Flig
106116
description={translate('travel.flightDetails.class')}
107117
title={cabinClassMapping[reservation.route.class] || reservation.route.class}
108118
interactive={false}
119+
copyValue={cabinClassMapping[reservation.route.class] || reservation.route.class}
120+
copyable
109121
/>
110122
</View>
111123
)}
112124
{!!reservation.confirmations?.at(0)?.value && (
113-
<View style={styles.w50}>
125+
<View style={styles.w100}>
114126
<MenuItemWithTopDescription
115127
description={translate('travel.flightDetails.recordLocator')}
116128
title={reservation.confirmations?.at(0)?.value}
117129
copyValue={reservation.confirmations?.at(0)?.value}
130+
copyable={!!reservation.confirmations?.at(0)?.value?.length}
131+
interactive={false}
118132
/>
119133
</View>
120134
)}
@@ -128,6 +142,7 @@ function FlightTripDetails({reservation, prevReservation, personalDetails}: Flig
128142
description={personalDetails?.login ?? reservation.travelerPersonalInfo?.email}
129143
interactive={false}
130144
wrapperStyle={styles.pb3}
145+
labelStyle={styles.mb2}
131146
/>
132147
)}
133148
</>

0 commit comments

Comments
 (0)