423942 - "Clear List" in download manager should be "Clear Current List" during the search

Status: RESOLVED WONTFIX

(Toolkit :: Downloads API, defect)

Description

[dynamis (Tomoya ASAI)]

In bug 414850 "Clear List" in download manager now only remove visible downloads.
But the label "Clear List" may make users think that this context menu will remove all download history not only currently visible ones.

To avoid confusion we should change the label dynamically when we searched and only search result are listed in the dialog.That is, we should use
  without search: Clear List
   during search: Clear All List
for the "Clear List" context menu in the download manager.

see:
https://bugzilla.mozilla.org/show_bug.cgi?id=414850#c3
https://bugzilla.mozilla.org/show_bug.cgi?id=414850#c20

note:
This is late-l10n after b5 but IMHO we should accept late-l10n like this.
Localizers don't only translate literally but we sometimes write/select better
expression for users. In some case l10n depends on the behavior of the feature
and localizers will have trouble not only with late l10n change but also with
late behavior change.

When the behavior change, it's natural to change strings and we should not be
afraid about late-l10n change. IMHO if behavior change was accepted, string change should also be accepted.

Comment 1

[Axel Hecht [:Pike]]

I'd rather have beltzner make a call here.

Comment 2

[Mike Beltzner [:beltzner, not reading bugmail]]

This definitely doesn't block. We changed the way the behaviour works in bug 414850 because we believed that it would be more common for people to interpret "Clear List" as "clear the list of things that are currently showing", which I think is still correct.

Not a bad suggestion, overall, but not something we should focus on for this release, adding wanted and ue and polish so we don't lose track.

Comment 3

[Ed Lee :Mardak]

Attachment: v1

"Clear Current List" should work for all cases.. searching or not.

Also get rid of the clear list button's tooltip because.. we have no clear list button.

Comment 4

[Shawn Wilsher :sdwilsh]

renoming - drivers please see bug 414850 comment 20 when making your decision.  I think this is pretty serious.

Comment 5

[Shawn Wilsher :sdwilsh]

Comment on attachment 310555 [details] [diff] [review]
v1

>-<!ENTITY cmd.clearList.tooltip            "Removes completed, canceled, and failed downloads from the list">
I think we should probably keep this, but maybe phrase it as "Removes completed, canceled, and failed downloads from the displayed list".

regardless, r=sdwilsh - needs ui-r though.

Comment 6

[Ed Lee :Mardak]

Why should we keep it around? Are we going to use it? It's not entirely correct right now either because it'll remove finished, canceled, failed, blocked parental, dirty, blocked policy downloads from the displayed list.

Comment 7

[Shawn Wilsher :sdwilsh]

Right, so it needs work, but I don't feel we should get rid of it.

Comment 8

[Ed Lee :Mardak]

What do we need it for? If we fix it, it just makes extra work for localizers for a string that we don't even use.

Comment 9

[Madhava Enros [:madhava]]

So, to be sure, before I ui-r... it will say "Clear Current List" only during a search, right?

Comment 10

[Mike Beltzner [:beltzner, not reading bugmail]]

(In reply to comment #4)
> renoming - drivers please see bug 414850 comment 20 when making your decision. 
> I think this is pretty serious.

I think that's a bug on the ja-JP locale, based on overzealous translation. Feel free to add a new localizer note, but this issue on its own doesn't block.

Comment 11

[dynamis (Tomoya ASAI)]

(In reply to comment #10)
> I think that's a bug on the ja-JP locale, based on overzealous translation.
> Feel free to add a new localizer note, but this issue on its own doesn't block.

Our ja/ja-JP-mac is special case and our case is not the reason to request this bug (I also don't think this as blocker).

What I meant to say in the bug 414850 comment 20 was:
We should accept late-l10n string changes too as long as we accept behavior/feature changes since it sometimes have l10n impact.
# we should not split late-l10n deadline and behavior change deadline
Localizers also want to make better Firefox and at least our locale can accept any late-l10n string change to make better Firefox.

Thanks

Comment 12

[dynamis (Tomoya ASAI)]

(In reply to comment #9)
> So, to be sure, before I ui-r... it will say "Clear Current List" only during a
> search, right?

I think it's better to say "Clear Current List" only during a search to make thing s clearer but as Edward wrote on comment #3 we can use same "Clear Curret List" label for both case as far as implementation cost is too high (in this period).

Comment 13

[Damian Shaw [Quan]]

This still going to make it now Clear List is a button?

Comment 14

[(mostly gone) XtC4UaLL [:xtc4uall]]

(In reply to comment #8)
> What do we need it for? If we fix it, it just makes extra work for localizers
> for a string that we don't even use.

i guess Shawn thought of a "cmd.clearCurrentList.tooltip" to be added with the rephrased description. why not simply use "Removes all your downloads from the displayed list" anyways?

(In reply to comment #9)
> So, to be sure, before I ui-r... it will say "Clear Current List" only during a
> search, right?
no, see comment 3.

Comment 15

[:Paolo Amadini]

This user interface was removed, and the command in Firefox is now called "Clear Preview Panel".