Skip to content

Commit b10d793

Browse files
committed
fix: align standalone pill right-padding with UserPills container
1 parent daaf3d0 commit b10d793

3 files changed

Lines changed: 25 additions & 19 deletions

File tree

src/components/ApprovalWorkflowSection.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ function ApprovalWorkflowSection({approvalWorkflow, onPress, currency = CONST.CU
121121
onPress={onPress}
122122
shouldRemoveBackground
123123
titleComponent={
124-
<View style={styles.ml3}>
124+
<View style={[styles.ml3, styles.pr3]}>
125125
<UserPill
126126
avatar={approver.avatar}
127127
displayName={approver.displayName}

src/pages/workspace/workflows/approvals/ApprovalWorkflowEditor.tsx

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -190,12 +190,14 @@ function ApprovalWorkflowEditor({approvalWorkflow, removeApprovalWorkflow, polic
190190
descriptionTextStyle={!!approver?.displayName && styles.textLabelSupportingNormal}
191191
titleComponent={
192192
approver ? (
193-
<UserPill
194-
avatar={approver.avatar}
195-
displayName={approver.displayName}
196-
email={approver.email}
197-
style={styles.userPillStandalone}
198-
/>
193+
<View style={styles.pr3}>
194+
<UserPill
195+
avatar={approver.avatar}
196+
displayName={approver.displayName}
197+
email={approver.email}
198+
style={styles.userPillStandalone}
199+
/>
200+
</View>
199201
) : undefined
200202
}
201203
onPress={() => editApprover(approverIndex)}

src/pages/workspace/workflows/approvals/WorkspaceWorkflowsApprovalsApprovalLimitPage.tsx

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -207,12 +207,14 @@ function WorkspaceWorkflowsApprovalsApprovalLimitPage({policy, isLoadingReportDa
207207
descriptionTextStyle={approverDisplayName ? styles.textLabelSupportingNormal : undefined}
208208
titleComponent={
209209
currentApprover ? (
210-
<UserPill
211-
avatar={currentApprover.avatar}
212-
displayName={currentApprover.displayName}
213-
email={currentApprover.email}
214-
style={styles.userPillStandalone}
215-
/>
210+
<View style={styles.pr3}>
211+
<UserPill
212+
avatar={currentApprover.avatar}
213+
displayName={currentApprover.displayName}
214+
email={currentApprover.email}
215+
style={styles.userPillStandalone}
216+
/>
217+
</View>
216218
) : undefined
217219
}
218220
onPress={navigateToApproverChange}
@@ -254,12 +256,14 @@ function WorkspaceWorkflowsApprovalsApprovalLimitPage({policy, isLoadingReportDa
254256
descriptionTextStyle={selectedApproverDisplayName ? styles.textLabelSupportingNormal : undefined}
255257
titleComponent={
256258
selectedApproverEmail ? (
257-
<UserPill
258-
avatar={selectedApproverPersonalDetails?.avatar}
259-
displayName={selectedApproverPersonalDetails?.displayName ?? selectedApproverEmail}
260-
email={selectedApproverEmail}
261-
style={styles.userPillStandalone}
262-
/>
259+
<View style={styles.pr3}>
260+
<UserPill
261+
avatar={selectedApproverPersonalDetails?.avatar}
262+
displayName={selectedApproverPersonalDetails?.displayName ?? selectedApproverEmail}
263+
email={selectedApproverEmail}
264+
style={styles.userPillStandalone}
265+
/>
266+
</View>
263267
) : undefined
264268
}
265269
onPress={navigateToApproverSelector}

0 commit comments

Comments
 (0)