670513 - Recently Closed Windows/tabs have been moved to the Go menu. Document this.

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Edmund Wong (:ewong)]

Bug #599731 will have moved "Recently Closed Windows" and "Recently Closed Tabs" from the File menu to the Go menu.  This needs to be added to the Help.

Comment 1

[Edmund Wong (:ewong)]

Attachment: Added "Recently Closed menus" to Help.

Comment 2

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 545070 [details] [diff] [review]
Added "Recently Closed menus" to Help.

Review of attachment 545070 [details] [diff] [review]:
-----------------------------------------------------------------

r- for now since a new patch is needed.

::: suite/locales/en-US/chrome/common/help/nav_help.xhtml
@@ +249,5 @@
>  
> +<h2 id="reopening_closed_tabs_windows">Reopening Closed Tabs or Windows</h2>
> +
> +<p>If you closed any windows or tabs during your browsing session, and you
> +   can reopen them.</p>

"If ... and" is horribly wrong. Besides that, by default we do not allow to restore an unlimited amount of tabs/windows (only 10/3, but I'd rather not have the exact amount in the docs). Suggestion/draft:
"&brandShortName; keeps track of your most recently closed tabs and windows and allows you to get back to them easily."

@@ +251,5 @@
> +
> +<p>If you closed any windows or tabs during your browsing session, and you
> +   can reopen them.</p>
> +
> +<p>To re-open closed Tabs:</p>

For the sake of consistency, "reopen" throughout please, and lowercase "tabs" (since we're neither referring to menu items here nor using title case).

@@ +256,5 @@
> +
> +<ul>
> +  <li>Open the Go Menu and go to the Recently Closed Tabs menu.</li>
> +  <li>Select any of the previously closed tabs amongst the listed sites.
> +      Once selected, a new tab will open to the page selected.</li>

* "Go menu" (lowercase "m")
* "and expand the" or similar (it's a submenu; "go" indicates clicking to me)
* "... closed tabs from the list." (the list either contains tabs or sites, i.e. you cannot pick a tab from a list of sites!)
* "open to" sounds strange. Maybe just "Once selected, it will be restored and removed from the list."

@@ +259,5 @@
> +  <li>Select any of the previously closed tabs amongst the listed sites.
> +      Once selected, a new tab will open to the page selected.</li>
> +</ul>
> +
> +<p>Similarly, to re-open Closed Windows:</p>

"reopen", lowercase "closed windows".

@@ +264,5 @@
> +
> +<ul>
> +   <li>Open the Go Menu and go to the Recently Closed Windows menu.</li>
> +   <li>Select any of the previously closed windows and once selected,
> +       a new window will open to the site selected.</li>

* "Go menu" / "and expand the" or similar again
* Full stop after "closed windows" (optionally with "from the list" as above)
* "Once selected, it will be ..." as above (that formulation also has the benefit of not incorrectly suggesting that restoring a window will just load a single site: windows can contain multiple tabs!)

Comment 3

[Edmund Wong (:ewong)]

Attachment: Added "Recently Closed menus" to Help. (v2)

Comment 4

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 546399 [details] [diff] [review]
Added "Recently Closed menus" to Help. (v2)

Review of attachment 546399 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the nits addressed.

::: suite/locales/en-US/chrome/common/help/nav_help.xhtml
@@ +251,5 @@
> +
> +<p>&brandShortName; keeps track of your most recently closed tabs and 
> +   windows and allows you to get back to them easily.</p>
> +
> +<p>To reopen closed Tabs:</p>

"tabs" (lower case)

@@ +256,5 @@
> +
> +<ul>
> +  <li>Open the Go menu and expand the Recently Closed Tabs menu.</li>
> +  <li>Select any of the previously closed tabs from the list.
> +      Once selected, it will be restored and removed from the list.</li>

Only two spaces per indentation level, so two less before "Once" (the <li> is indented below <ul>, and all but the first line of a <li> is indented below the <li>). Your indentation style is valid in general, too, just not the one we're using in Help.

@@ +264,5 @@
> +
> +<ul>
> +   <li>Open the Go Menu and expand the Recently Closed Windows menu.</li>
> +   <li>Select any of the previously closed windows. Once selected,
> +       it will be restored and removed from the list.</li>

As above, two less spaces of indentation before "it".

Comment 5

[Edmund Wong (:ewong)]

Attachment: Added "Recently Closed menus" to Help. (v3) [Checkin: comment 6]

Comment 6

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 547989 [details] [diff] [review]
Added "Recently Closed menus" to Help. (v3) [Checkin: comment 6]

http://hg.mozilla.org/comm-central/rev/b49da44d861f