Closed Bug 670513 Opened 9 years ago Closed 9 years ago

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

Categories

(SeaMonkey :: Help Documentation, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.5

People

(Reporter: ewong, Assigned: ewong)

References

Details

Attachments

(1 file, 2 obsolete files)

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.
Blocks: 599731
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #545070 - Flags: review?(jh)
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!)
Attachment #545070 - Flags: review?(jh) → review-
Attachment #545070 - Attachment is obsolete: true
Attachment #546399 - Flags: review?(jh)
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".
Attachment #546399 - Flags: review?(jh) → review+
Keywords: checkin-needed
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
Attachment #547989 - Attachment description: Added "Recently Closed menus" to Help. (v3) → Added "Recently Closed menus" to Help. (v3) [Checkin: comment 6]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.5
You need to log in before you can comment on or make changes to this bug.