Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Fix #7695 updated text for the restore tabs page #7703

Closed
wants to merge 1 commit into from
Closed

Fix #7695 updated text for the restore tabs page #7703

wants to merge 1 commit into from

Conversation

hkarmoush
Copy link
Contributor

Summary of Changes

This pull request fixes #7695

Submitter Checklist:

  • Unit Tests are updated to cover new or changed functionality
  • User-facing strings use NSLocalizableString()
  • New or updated UI has been tested across:
    • Light & dark mode
    • Different size classes (iPhone, landscape, iPad)
    • Different dynamic type sizes

Test Plan:

Screenshots:

Reviewer Checklist:

  • Issues include necessary QA labels:
    • QA/(Yes|No)
    • bug / enhancement
  • Necessary security reviews have taken place.
  • Adequate unit test coverage exists to prevent regressions.
  • Adequate test plan exists for QA to validate (if applicable).
  • Issue and pull request is assigned to a milestone (should happen at merge time).

@hkarmoush hkarmoush requested a review from a team as a code owner July 8, 2023 19:25
@hkarmoush hkarmoush changed the title Fix-%237695 updated text for the restore tabs page Fix-#237695 updated text for the restore tabs page Jul 8, 2023
@soner-yuksel soner-yuksel changed the title Fix-#237695 updated text for the restore tabs page Fix #7695 updated text for the restore tabs page Jul 9, 2023
@hkarmoush
Copy link
Contributor Author

@soner-yuksel Thanks

public static let restoreTabOnCrashAlertTitle = NSLocalizedString("RestoreTabOnCrashAlertTitle", tableName: "BraveShared", bundle: .module, value: "Well, this is embarrassing.", comment: "Restore Tabs Prompt Title")
public static let restoreTabOnCrashAlertMessage = NSLocalizedString("RestoreTabOnCrashAlertMessage", tableName: "BraveShared", bundle: .module, value: "Looks like Brave crashed previously. Would you like to restore your tabs?", comment: "Restore Tabs Prompt Description")
public static let restoreTabOnCrashAlertTitle = NSLocalizedString("RestoreTabOnCrashAlertTitle", tableName: "BraveShared", bundle: .module, value: "Brave Closed Unexpectedly", comment: "Restore Tabs Prompt Title")
public static let restoreTabOnCrashAlertMessage = NSLocalizedString("RestoreTabOnCrashAlertMessage", tableName: "BraveShared", bundle: .module, value: "Unfortunately, Brave crashed or did not close properly. \nRestore your tabs?", comment: "Restore Tabs Prompt Description")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You do not need the empty space, the other next sentence is already starting in new line.

@iccub iccub closed this Aug 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Text update for the restore tabs page
3 participants