The default bug view has changed. See this FAQ.

Adapt Help for changes from Bug 536374 - Places history changes due to async expiration

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
Help Documentation
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: InvisibleSmiley, Assigned: Robert Kaiser)

Tracking

Trunk
seamonkey2.1a1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

7 years ago
From Bug 536374 comment 22:
"Also I suggest adding a relnote explaining people that the old prefs allowed for making some false assumptions (e.g. only the minimum number of days to keep history could be set effectively since the upper bound really was determined by the maximum number of pages to keep). Cf. MaK's blog post:
<http://blog.bonardo.net/2010/01/20/places-got-async-expiration>"
(Assignee)

Comment 1

7 years ago
Argh, sorry about forgetting the help changes, I knew there was something I had this marked TODO for...
(Reporter)

Comment 2

7 years ago
(In reply to comment #1)
> Argh, sorry about forgetting the help changes, I knew there was something I had
> this marked TODO for...

You may still want to set the Target Milestone (I refrained from doing it myself since you're the assignee).
(Assignee)

Updated

7 years ago
Assignee: nobody → kairo
Target Milestone: --- → seamonkey2.1a1
(Assignee)

Comment 3

7 years ago
Created attachment 437959 [details] [diff] [review]
adapt help for history prefs

So this one is easy :)
Attachment #437959 - Flags: review?(iann_bugzilla)
(Assignee)

Comment 4

7 years ago
Created attachment 437968 [details] [diff] [review]
adapt help for history prefs a bit more

Oops, I found one more stance that becomes obsolete with this.
Attachment #437959 - Attachment is obsolete: true
Attachment #437968 - Flags: review?(iann_bugzilla)
Attachment #437959 - Flags: review?(iann_bugzilla)
(Assignee)

Updated

7 years ago
Status: NEW → ASSIGNED

Comment 5

7 years ago
Comment on attachment 437968 [details] [diff] [review]
adapt help for history prefs a bit more

>+++ b/suite/locales/en-US/chrome/common/help/cs_nav_prefs_navigator.xhtml
>@@ -144,35 +144,20 @@
>+      <li><strong>Remember visited pages</strong>: Select this to make
>+        &brandShortName; remember pages you visit in the browsing history.
"within" instead of "in" perhaps?
 You
>+        need that for example to be able to search for pages you have already
>+        visited from the location bar or history window.</li>
Use "For example, you need this to be able to..."

>diff --git a/suite/locales/en-US/chrome/common/help/nav_help.xhtml b/suite/locales/en-US/chrome/common/help/nav_help.xhtml
>--- a/suite/locales/en-US/chrome/common/help/nav_help.xhtml
>+++ b/suite/locales/en-US/chrome/common/help/nav_help.xhtml
>@@ -218,20 +218,16 @@
>     <span class="mac">&brandShortName;</span><span class="noMac">Edit</span>
>     menu and choose Preferences.</li>
>   <li>Under the Browser category, click History. (If no subcategories are
>     visible, double-click Browser to expand the list.)</li>
>   <li>Click Clear History and Clear Location Bar to remove all previously
>     visited web pages from the lists.</li>
> </ol>
> 
>-<p><strong>Tip</strong>: Use the preferences under <a
>-  href="cs_nav_prefs_navigator.xhtml#history">Browsing History</a> to set
>-  how long and how many pages will remain in the history list.</p>
>-
Rather than deleting should the tip just be changed to:
"Use the preference under Browsing History to set if the history of visited pages is remembered."

r- for the moment.
Attachment #437968 - Flags: review?(iann_bugzilla) → review-
(Assignee)

Comment 6

7 years ago
(In reply to comment #5)
> Rather than deleting should the tip just be changed to:
> "Use the preference under Browsing History to set if the history of visited
> pages is remembered."

Actually, that "tip" doesn't sound really helpful to me. It's only and on/off switch now anyhow, and this is in "About History Lists" - do people reading that really want to know how to turn off having content in one of those lists at all?
(Assignee)

Updated

7 years ago
Whiteboard: [history]

Comment 7

7 years ago
(In reply to comment #6)
> (In reply to comment #5)
> > Rather than deleting should the tip just be changed to:
> > "Use the preference under Browsing History to set if the history of visited
> > pages is remembered."
> 
> Actually, that "tip" doesn't sound really helpful to me. It's only and on/off
> switch now anyhow, and this is in "About History Lists" - do people reading
> that really want to know how to turn off having content in one of those lists
> at all?

okay, so leave the deletion of the tip in then.

Comment 8

7 years ago
Comment on attachment 437968 [details] [diff] [review]
adapt help for history prefs a bit more

r=me with those first 2 changes.
Attachment #437968 - Flags: review- → review+
(Assignee)

Comment 9

7 years ago
Pushed as http://hg.mozilla.org/comm-central/rev/6d1782409f38
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [history]
You need to log in before you can comment on or make changes to this bug.