643677 - bug #133119 changed the Image Acceptance Policy prefs. Help needs updating.

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Edmund Wong (:ewong)]

Update the help in accordance to the changes as made in #133119 (or
subsequently #643676).

Comment 1

[Edmund Wong (:ewong)]

Attachment: Updated Image Acceptance Policy Pref help from changes from bug #133119.

Comment 2

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 520870 [details] [diff] [review]
Updated Image Acceptance Policy Pref help from changes from bug #133119.

>-  <li><strong>Accept images that come from the originating server
>-    only</strong>: Select this option if you do not want to load images from
>+  <li><strong>Load only images that come from the originating server
>+    </strong>: Select this option if you do not want to load images from
>     third-party websites.</li>

Actually this is a case where copy/paste is good. Use it and you'll see that the string here is not equal to the one added through bug 133119.

Also please don't have whitespace between text and directly enclosing tags, in this case "server" and "</strong>". If wrapping needs so be applied, also move the word to the next line ("server</strong>"). Exception from the rule: In case the closing tag is the last one on the line (like "websites.</li>"), you may keep it where it is if the overhead is less than say four characters and the word before it is long.

For the general case that means if in the end you need to re-wrap a whole paragraph, that's OK.

Minus'ing for the moment, but just because I want to see a new patch and have some time to double-check the old strings are not used elsewhere in Help (which I cannot do from here).

Comment 3

[Edmund Wong (:ewong)]

Attachment: Updated Help from bug #133119 changes. [Checkin: comment 4]

Comment 4

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 520901 [details] [diff] [review]
Updated Help from bug #133119 changes. [Checkin: comment 4]

http://hg.mozilla.org/comm-central/rev/55b2258fab38