1398183 - Update copy for about:sessionrestore

Status: RESOLVED FIXED

(Firefox :: Session Restore, enhancement, P1)

Description

[mheubusch]

+++ This bug was initially created as a clone of Bug #1394457 +++

Based on copy and mocks found in Bug 1358915
Breaking this off as  copy-only bug

Comment 1

[mheubusch]

Hello - Here is final copy validated by Justin Dolske.  Thanks, Dao, for catching this and bringing it to the attention of UX!

<h1>Sorry. We’re having trouble getting your pages back.</h1>

<text>We are having trouble restoring your last browsing session. Select Restore Session to try again.



Still not able to restore your session? Sometimes a tab is causing the issue. View previous tabs, remove the checkmark from the tabs you don’t need to recover, and then restore. 
View previous tabs>

[Restore Session] [Start New Session]

Comment 2

[Erica Wright [:ewright]]

(In reply to mheubusch from comment #1)
> Hello - Here is final copy validated by Justin Dolske.  Thanks, Dao, for
> catching this and bringing it to the attention of UX!
> 
> <h1>Sorry. We’re having trouble getting your pages back.</h1>
> 
> <text>We are having trouble restoring your last browsing session. Select
> Restore Session to try again.
> 
> 
> 
> Still not able to restore your session? Sometimes a tab is causing the
> issue. View previous tabs, remove the checkmark from the tabs you don’t need
> to recover, and then restore. 
> View previous tabs>
> 
> [Restore Session] [Start New Session]

:mheubusch the buttons on the bottom, in the old mock had "Restore X Tabs", this gives the user some more information since the tab/window list will be hidden by default now. Do you want this removed? 

https://mozilla.invisionapp.com/share/ZKBC94BPQ#/screens/229787806

Comment 3

[mheubusch]

Hi Erica - we are standardizing around Restore Session (not tabs) so yes, we are losing the count of tabs as well as the need to build singular and plural.

Comment 4

[Nihanth Subramanya [:nhnt11]]

Attachment: Bug 1398183 - Update copy for about:sessionrestore.

Review commit: https://reviewboard.mozilla.org/r/181448/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/181448/

Comment 5

[Nihanth Subramanya [:nhnt11]]

The patch updates the strings in the interest of getting them landed in m-c ASAP. I haven't yet updated the positions of the restore session/new session buttons because I'd like to confirm the consistency of the confirm/cancel buttons across our about: pages. I also haven't messed with the amount of spacing between the "We are having trouble..." and "Still not able to restore..." lines.

Comment 6

[Johann Hofmann [:johannh]]

Comment on attachment 8909963 [details]
Bug 1398183 - Update copy for about:sessionrestore.

https://reviewboard.mozilla.org/r/181448/#review186710

Comment 7

[Nihanth Subramanya [:nhnt11]]

Comment on attachment 8909963 [details]
Bug 1398183 - Update copy for about:sessionrestore.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/181448/diff/1-2/

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/332b5c019493
Update copy for about:sessionrestore. r=johannh

Comment 9

[Sebastian Hengst [:aryx] (needinfo on intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/332b5c019493

Comment 10

[Francesco Lodolo [:flod]]

Comment on attachment 8909963 [details]
Bug 1398183 - Update copy for about:sessionrestore.

https://reviewboard.mozilla.org/r/181448/#review187158

::: browser/locales/en-US/chrome/browser/aboutSessionRestore.dtd:13
(Diff revision 2)
>       and regret that we are unable to restore the session for the user -->
> -<!ENTITY restorepage.errorTitle     "Well, this is embarrassing.">
> -<!ENTITY restorepage.problemDesc    "&brandShortName; is having trouble recovering your windows and tabs. This is usually caused by a recently opened web page.">
> -<!ENTITY restorepage.tryThis        "You can try:">
> -<!ENTITY restorepage.restoreSome    "Removing one or more tabs that you think may be causing the problem">
> -<!ENTITY restorepage.startNew       "Starting an entirely new browsing session">
> +<!ENTITY restorepage.errorTitle2    "Sorry. We’re having trouble getting your pages back.">
> +<!ENTITY restorepage.problemDesc2   "We are having trouble restoring your last browsing session. Select Restore Session to try again.">
> +<!ENTITY restorepage.tryThis2       "Still not able to restore your session? Sometimes a tab is causing the issue. View previous tabs, remove the checkmark from the tabs you don’t need to recover, and then restore.">
> +
> +<!ENTITY restorepage.tryagainButton2 "Restore Session">

Notes for the future.

Since the label changed completely, probably better to call this restorepage.restoreSessionButton, and reference it in restorepage.problemDesc2 instead of writing "Restore Session".

<!ENTITY restorepage.problemDesc2   "We are having trouble restoring your last browsing session. Select &restorepage.restoreSessionButton; to try again.">