"Setting Proxy Values" has outdated block of text in wrong place

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
Help Documentation
RESOLVED FIXED
13 years ago
7 years ago

People

(Reporter: benc, Assigned: Ian Neal)

Tracking

({fixed-seamonkey2.0.6})

Trunk
seamonkey2.1a1
fixed-seamonkey2.0.6

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Reporter)

Description

13 years ago
From the bundled help files:

                o No Proxy for: Type the domains that you do not want to use a
proxy for. Separate each domain with a comma. (Example: .yourcompany.com,
.yourcompany.co.nz)
          * Automatic proxy configuration URL: Choose this if you have a proxy
configuration file or URL, then type the configuration URL.

Domain names are the part of a URL that contains the name of an organization,
business, or school—such as netscape.com or washington.org. If you use local
host names without the domain name, list them the same way. Use commas to
separate multiple host names. The wildcard character [*] cannot be used.

I think the Auto Proxy stuff got lost. Also, the last paragraph is kind of out
of date, follow the link on this page to "Advanced Preferences | Proxy" to see
what the current feature description is.

(Why do we have two sections that are almost identical anyhow?)

Updated

13 years ago
Component: Help → User
Product: Browser → Documentation
Version: Trunk → unspecified
(Reporter)

Comment 1

13 years ago
Looking at this further:

Search help files for "proxy" and get 4 hits.

"proxy" - glossary definition
"proxy preferences" - help file for prefs panel
"Setting Proxy Values"
"setting proxy values" - dupe entries for a subsection of "Browsing the Web".

The last section (which I think is a needless dupe) is the content I am quoting
here.
QA Contact: daniel.bugmail → benc

Updated

13 years ago
Assignee: rlk → nobody

Comment 2

12 years ago
This seems fixed elsewhere to me: can anyone please verify and close the bug?
Thanks.

Comment 3

11 years ago
Created attachment 228658 [details] [diff] [review]
Clean up somewhat the references

The duplicated content is still there, and I think it should stay so, because the paragraph in nav_help will get you started on using proxies with the browser, while che advanced preferences are explained in another file.
This patch cleans up the index and remove a useless contentbox & h1 heading for proxies, adding that paragraph to the "improve speed and efficency" chapter of the nav_help.
Attachment #228658 - Flags: review?(iann_bugzilla)
(Assignee)

Comment 4

11 years ago
(In reply to comment #3)
> Created an attachment (id=228658) [edit]
> Clean up somewhat the references
> 
> The duplicated content is still there, and I think it should stay so, because
> the paragraph in nav_help will get you started on using proxies with the
> browser, while che advanced preferences are explained in another file.
> This patch cleans up the index and remove a useless contentbox & h1 heading for
> proxies, adding that paragraph to the "improve speed and efficency" chapter of
> the nav_help.
> 

I'm not convinced that "Using Proxies" belongs in a chapter on improving speed and efficency. If you were to move it, having taken away an entry in one chapter, you will need to add an entry to the contents box for the chapter it goes in.
(Assignee)

Updated

11 years ago
Attachment #228658 - Flags: review?(iann_bugzilla) → review-

Comment 5

11 years ago
(In reply to comment #4)
> I'm not convinced that "Using Proxies" belongs in a chapter on improving speed
> and efficency. If you were to move it, having taken away an entry in one
> chapter, you will need to add an entry to the contents box for the chapter it
> goes in.

You mean this:
+++ extensions/help/resources/locale/en-US/nav_help.xhtml	10 Jul 2006 09:28:58 -0000
@@ -1416,16 +1416,17 @@
...
+    <li><a href="#using_proxies">Using proxies</a></li>
   </ul>
 </div>
?

Or the missing entry in help-index1 (removing the one about proxy)?
Anyway, if that chapter doesn't fit, I guess we should just point to the prefs settings, bringing over the few missing text paragraph from nav_help (about organizations and the explanation of the use of domain names). Just my €.02...

Comment 6

11 years ago
(In reply to comment #5)
> Or the missing entry in help-index1 (removing the one about proxy)?

help-toc, sorry.

Updated

10 years ago
Component: Help Viewer → Help
Product: Documentation → Mozilla Application Suite
Version: unspecified → Trunk
(Assignee)

Updated

7 years ago
QA Contact: benc → help
(Assignee)

Comment 7

7 years ago
Created attachment 443508 [details] [diff] [review]
Remove duplicated section patch v0.2

This patch:
* Removes section that is duplicated in the Advance Prefs - Proxy section
* Renames section from "Setting Proxy Values" to "Using Proxies"
Assignee: nobody → iann_bugzilla
Attachment #228658 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #443508 - Flags: review?(stefanh)

Comment 8

7 years ago
Comment on attachment 443508 [details] [diff] [review]
Remove duplicated section patch v0.2

 <h1 id="proxies">Proxies</h1>
 
 <div class="contentsBox">In this section:
   <ul>
-    <li><a href="#setting_proxy_values">Setting Proxy Values</a></li>
+    <li><a href="#using_proxies">Using Proxies</a></li>
   </ul>
 </div>
 
 <p>This section explains how to work with proxies.</p>
 
-<h2 id="setting_proxy_values">Setting Proxy Values</h2>
+<h2 id="using_proxies">Using Proxies</h2>

One thought: How about keeping the <h1/>, removing the whole contentsBox, paragraph (kind of wrong anyway, since we don't explain how to work with proxies here) and secondary heading? Or having "Using Proxies" as a main header? I'm in the middle, but it feels kind of an overkill having an index for 1 section...
(Assignee)

Updated

7 years ago
Attachment #443508 - Flags: review?(stefanh)
(Assignee)

Comment 9

7 years ago
Created attachment 443709 [details] [diff] [review]
No subsection patch v0.2a

Changes since v0.2:
* Made subsection a section.
Attachment #443508 - Attachment is obsolete: true
Attachment #443709 - Flags: review?(stefanh)

Comment 10

7 years ago
Comment on attachment 443709 [details] [diff] [review]
No subsection patch v0.2a

 <p>This section explains how to work with proxies.</p>
I think we can just remove this.

-<p>For more information on using the Proxy preferences panel, see
+<p>For information on setting the proxy preferences, see
As discussed: "For information on setting your proxy preferences, see"
Attachment #443709 - Flags: review?(stefanh) → review+
(Assignee)

Comment 11

7 years ago
Created attachment 443724 [details] [diff] [review]
No subsection for commit patch v0.2b [Checkin: comments 13+14]

Changes since v0.2a:
* As per reviewer's comments.
Attachment #443709 - Attachment is obsolete: true
Attachment #443724 - Flags: review+
(Assignee)

Updated

7 years ago
Attachment #443724 - Flags: approval-seamonkey2.1a1?
(Assignee)

Comment 12

7 years ago
Comment on attachment 443724 [details] [diff] [review]
No subsection for commit patch v0.2b [Checkin: comments 13+14]

Requesting a= for SM2.1a1, help patch should not affect stability.

Updated

7 years ago
Attachment #443724 - Flags: approval-seamonkey2.1a1? → approval-seamonkey2.1a1+
(Assignee)

Comment 13

7 years ago
Comment on attachment 443724 [details] [diff] [review]
No subsection for commit patch v0.2b [Checkin: comments 13+14]

Checkin to trunk http://hg.mozilla.org/comm-central/rev/6f07e8fee4d0
Requesting a-SM2.0.6 too
Attachment #443724 - Attachment description: No subsection for commit patch v0.2b → No subsection for commit patch v0.2b [Checkin: comment 13]
Attachment #443724 - Flags: approval-seamonkey2.0.6?
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Updated

7 years ago
Attachment #443724 - Flags: approval-seamonkey2.0.6? → approval-seamonkey2.0.6+
(Assignee)

Comment 14

7 years ago
Comment on attachment 443724 [details] [diff] [review]
No subsection for commit patch v0.2b [Checkin: comments 13+14]

http://hg.mozilla.org/releases/comm-1.9.1/rev/e3453cf8701a
Attachment #443724 - Attachment description: No subsection for commit patch v0.2b [Checkin: comment 13] → No subsection for commit patch v0.2b [Checkin: comments 13+14]
(Assignee)

Updated

7 years ago
Keywords: fixed-seamonkey2.0.6
Target Milestone: --- → seamonkey2.1a1
You need to log in before you can comment on or make changes to this bug.