1267071 - cs_nav_prefs_advanced.xhtml: Dead link behind "End User Guide to Proxy AutoConfiguration"

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Rainer Bielefeld]

Steps how to reproduce with  English SeaMonkey 2.45a1  (Windows NT 6.1; WOW64; rv:48.0)  Gecko/20100101 Firefox/48.0 Build 20160308001946  (Default Classic Theme)  on German WIN7 64bit:

1. Browser → F1
   » Help appears
2. Type search strint "proxy" → Click hit "proxiy preferences"
   » Related Help appears
3. Click Link at text "End User Guide to Proxy AutoConfiguration"
   » opens <https://www.mozilla.org/en-US/catalog/end-user/customizing/enduserPAC.html> 
   Bug: "Whoops!" (Page does not exist)

a) I do not know where a help text can be found
b) This one might be related to "Bug 213699 - Write PAC docs "

Comment 1

[rsx11m]

It's still in the archive for reference, but that's probably not what we want to link to:
http://www-archive.mozilla.org/catalog/end-user/customizing/enduserPAC.html

A rather technical description is here, which is good for writing PAC files but not as starting point:
https://developer.mozilla.org/en-US/docs/Mozilla/Projects/Necko/Proxy_Auto-Configuration_%28PAC%29_file

Comment 2

[Rainer Bielefeld]

(In reply to rsx11m from comment #1)
> A rather technical description is here, which is good for writing PAC files

Indeed, that's information for administrators, not for users.

Comment 3

[Frank-Rainer Grahl (:frg)]

Someone on irc inquired today. Please contact me if you read this.

This link has some explanation and even covers SeaMonkey:
https://calomel.org/proxy_auto_config.html

And/or we link just to the wikipedia article:
https://en.wikipedia.org/wiki/Proxy_auto-config

Comment 4

[Srujana Peddinti [:srujana121]]

Hi, can I take it up as my first bug?

Comment 5

[Rainer Bielefeld]

@Srujana Peddinti:
Thank you for your offer, of course.
If you need advice and /or assistance, please follow hints on
https://wiki.mozilla.org/SeaMonkey/FAQ#How_To_Start_Contributing_Code
step by step.

For any difficulties please do step 5 and ask for help in Newsgroup mozilla.dev.apps.seamonkey (or mailinglist)

You also might find some helpful hints in latest FIXED help/documentation bugs from https://bugzilla.mozilla.org/buglist.cgi?list_id=14597653&resolution=FIXED&classification=Client%20Software&chfieldto=Now&query_format=advanced&chfield=resolution&chfieldfrom=2016-01-01&chfieldvalue=FIXED&bug_status=RESOLVED&bug_status=VERIFIED&component=Help%20Documentation&product=SeaMonkey

Comment 6

[Nuno Silva [:njsg]]

Attachment: cs_nav_prefs_advanced.xhtml: replace PAC end user guide link, v1

A draft patch which adds a very, very brief explanation of PAC files and replaces the End User Guide link with the MDN one. Would this look acceptable?

Comment 7

[Frank-Rainer Grahl (:frg)]

Comment on attachment 9147816 [details] [diff] [review]
cs_nav_prefs_advanced.xhtml: replace PAC end user guide link, v1

Look fine to me. Best ask for feedback or r? every time you have a patch. Might fall thru a crack otherwise.

Comment 8

[Ian Neal]

Comment on attachment 9147816 [details] [diff] [review]
cs_nav_prefs_advanced.xhtml: replace PAC end user guide link, v1

LGTM r/a=me

[Triage Comment]

Comment 9

[Ian Neal]

Hi, are you working on which locales also need this fix?

Comment 10

[Nuno Silva [:njsg]]

For the ones that have cs_nav_prefs_advanced.xhtml in English, I can do the same as the non-l10n patch. For locales with translated help, should I make a patch that just swaps the link, without touching the surrounding text?

Comment 11

[Ian Neal]

(In reply to Nuno Silva from comment #10)

For the ones that have cs_nav_prefs_advanced.xhtml in English, I can do the same as the non-l10n patch. For locales with translated help, should I make a patch that just swaps the link, without touching the surrounding text?

Yes, please, I think that should be the starting point.

Comment 12

[Nuno Silva [:njsg]]

Attachment: cs_nav_prefs_advanced.xhtml: remove PAC end user guide link l10n patch, v1

A starting point patch :-)

Hungarian had the PAC End User Guide title both in English and Hungarian, so this removes that Hungarian title translation and adds the new title only in English.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/e43591416144
Reword text about proxy auto-configuration and replace dead link. r=IanN

Comment 14

[Ian Neal]

Comment on attachment 9149990 [details] [diff] [review]
cs_nav_prefs_advanced.xhtml: remove PAC end user guide link l10n patch, v1

[Triage Comment]
LGTM r/a=me

Comment 15

[Frank-Rainer Grahl (:frg)]

Target 2.53.3
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/097f844d3ded4382f4d8b1b77c4c525e0409c3b2
Reword text about proxy auto-configuration and replace dead link. r=IanN a=IanN