Skip to content

Commit

Permalink
Merge pull request #47704 from Krishna2323/krishna2323/issue/47445
Browse files Browse the repository at this point in the history
fix: Workspace - ESC key does not dismiss popup in Distance rates.
  • Loading branch information
bondydaa authored Aug 20, 2024
2 parents b2eb922 + d48a513 commit 72a7593
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions src/pages/settings/AboutPage/ConsolePage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -215,6 +215,7 @@ function ConsolePage({capturedLogs, shouldStoreLogs}: ConsolePageProps) {
title={translate('initialSettingsPage.debugConsole.shareLog')}
isVisible={isLimitModalVisible}
onConfirm={() => setIsLimitModalVisible(false)}
onCancel={() => setIsLimitModalVisible(false)}
prompt={translate('initialSettingsPage.debugConsole.logSizeTooLarge', {
size: CONST.API_ATTACHMENT_VALIDATIONS.MAX_SIZE / 1024 / 1024,
})}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -183,6 +183,7 @@ function PolicyDistanceRateDetailsPage({policy, route}: PolicyDistanceRateDetail
/>
<ConfirmModal
onConfirm={() => setIsWarningModalVisible(false)}
onCancel={() => setIsWarningModalVisible(false)}
isVisible={isWarningModalVisible}
title={translate('workspace.distanceRates.oopsNotSoFast')}
prompt={translate('workspace.distanceRates.workspaceNeeds')}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -338,6 +338,7 @@ function PolicyDistanceRatesPage({
)}
<ConfirmModal
onConfirm={() => setIsWarningModalVisible(false)}
onCancel={() => setIsWarningModalVisible(false)}
isVisible={isWarningModalVisible}
title={translate('workspace.distanceRates.oopsNotSoFast')}
prompt={translate('workspace.distanceRates.workspaceNeeds')}
Expand Down

0 comments on commit 72a7593

Please sign in to comment.