479848 - Point Options window help-topic links to sub-articles

Status: VERIFIED FIXED

(support.mozilla.org :: General, defect)

Description

[Chris Ilias [:cilias]]

We need the .htaccess file changed to point the Options window help-topic links from http://support.mozilla.com/%LOCALE%/kb/Options+window... to the appropriate sub articles. I'll post the destination links here.

I've also set the target to 1.0, but if there's going to be a 0.9.1, this should set to 0.9.1.

Comment 1

[Chris Ilias [:cilias]]

New destinations:
prefs-main\/$  /$4/kb/Options+window+-+Main+panel?style_mode=inproduct [R,NE]
prefs-clear-private-data\/$  /$4/kb/Options+window+-+Privacy+panel?style_mode=inproduct#private_data [R,NE]
prefs-fonts-and-colors\/$  /$4/kb/Options+window+-+Content+panel?style_mode=inproduct#fonts_and_colors [R,NE]
prefs-privacy\/$  /$4/kb/Options+window+-+Privacy+panel?style_mode=inproduct [R,NE]
prefs-applications\/$  /$4/kb/Options+window+-+Applications+panel?style_mode=inproduct [R,NE]
prefs-connection-settings\/$  /$4/kb/Options+window+-+Advanced+panel?style_mode=inproduct#connection_settings [R,NE]
prefs-tabs\/$  /$4/kb/Options+window+-+Tabs+panel?style_mode=inproduct [R,NE]
prefs-advanced-javascript\/$  /$4/kb/Options+window+-+Content+panel?style_mode=inproduct#advanced_javascript [R,NE]
prefs-languages\/$  /$4/kb/Options+window+-+Content+panel?style_mode=inproduct#languages [R,NE]
prefs-content\/$  /$4/kb/Options+window+-+Content+panel?style_mode=inproduct [R,NE]
prefs-security\/$  /$4/kb/Options+window+-+Security+panel?style_mode=inproduct [R,NE]
prefs-advanced-general\/$  /$4/kb/Options+window+-+Advanced+panel?style_mode=inproduct#advanced_general [R,NE]
prefs-advanced-network\/$  /$4/kb/Options+window+-+Advanced+panel?style_mode=inproduct#advanced_network [R,NE]
prefs-advanced-update\/$  /$4/kb/Options+window+-+Advanced+panel?style_mode=inproduct#advanced_update [R,NE]
prefs-advanced-encryption\/$  /$4/kb/Options+window+-+Advanced+panel?style_mode=inproduct#advanced_encryption [R,NE]

Comment 2

[Chris Ilias [:cilias]]

Attachment: patch

David, could you check this? I may have made a mistake while copying and pasting.

Comment 3

[David Tenser [:djst]]

I'm not sure about prefs-advanced-general; do we want to point it to the anchor when it's the first section of the article?

Comment 4

[Chris Ilias [:cilias]]

Attachment: removed anchor from advanced-general

Comment 5

[David Tenser [:djst]]

Comment on attachment 364346 [details] [diff] [review]
removed anchor from advanced-general

Looks good!

Comment 6

[Chris Ilias [:cilias]]

Could someone please check this in?

Comment 7

[E. Cooper [:ecooper]]

(In reply to comment #6)
> Could someone please check this in?

I will as soon as I get in this afternoon.

Comment 8

[E. Cooper [:ecooper]]

(In reply to comment #7)
> (In reply to comment #6)
> > Could someone please check this in?
> 
> I will as soon as I get in this afternoon.

Sorry for the delay. This is in r23081/r23082. htaccess.sh will have to be run on staging (and prod after the push) for the changes to be reflected.