310907 - document "All files downloaded to:" button in DM

Status: RESOLVED FIXED

(Firefox Graveyard :: Help Documentation, defect)

Description

[Steffen Wilberg]

The download manager has a button right to the label "All files downloaded to:".
We should document that because the button is hard to discover since it doesn't
look like a button.

Comment 1

[Gids Goldberg]

Attachment: documents 'All files downloaded to' button

I couldn't find a good way of putting the button label into the description. 

Also, two other problems with this document I found whilst preparing this
patch:
1)<strong> is used in this document, where <em> should be used.
2)The user is advised to 'type' a keyboard shotcut rather than 'press' it.

I am happy to fix these issues as well, but I'm guessing they fall outside the
scope of this patch.

Comment 2

[Steffen Wilberg]

Comment on attachment 199931 [details] [diff] [review]
documents 'All files downloaded to' button

> <p>The Download Manager will appear when you download files, when you select
s/select/press/
>   <span class="menuPath">Tools &gt; Downloads</span>, and when you type
>   <span class="noUnix">&accelKey;+<kbd>J</kbd></span><span class="unix">&accelKey;+<kbd>Y</kbd></span>
>-  on your keyboard.</p>
>+  </p>
There's a dot missing - put that at the end of the line before.

>+  <dt>Show Download Folder</dt>
>+  <dd>If you have set &brandShortName; to save all downloads to a folder, you
>+  can open this folder by clicking the <strong>Folder Name</strong> at the
>+  bottom of the Download Manager window.</dd>
> </dl>
Please add where that pref is located - in <a
href="#download_manager_options">Download Manager Options</a>.

Comment 3

[Steffen Wilberg]

Replacing <strong> by <em> is indeed outside the scope of this bug.

Comment 4

[Gids Goldberg]

Attachment: documents 'All files downloaded to' button

Addresses Steffen's points.

Comment 5

[Steffen Wilberg]

Comment on attachment 200134 [details] [diff] [review]
documents 'All files downloaded to' button

r=me. I'm not sure why you dropped the first part of your previous patch, but
now let's get this one in.

Comment 6

[Jeff Walden (gone starting June 8)]

Comment on attachment 200134 [details] [diff] [review]
documents 'All files downloaded to' button

>+  <a href="#download_manager_options">Download Manager Options</a>, you

This should be changed to "Download Manager &pref.pluralCaps;" before checkin.

Comment 7

[Steffen Wilberg]

Checked in, with comment 6 addressed and indenting fixed.