60948 - Can't type in the "open using" field in "downloading" dialog

Status: VERIFIED DUPLICATE of bug 49787

(SeaMonkey :: UI Design, defect, P3)

Description

[Boris Zbarsky [:bzbarsky]]

Linux build 2000-11-21-08

Steps to reproduce:

1) Go to the url provided (or anyplace where you have to get content you don't
have a helper for)
2) Try to click on a link to play streaming music
3) Get the "downloading" dialog
4) Try to type an application name

Actual results:  can't type in textfield

Comment 1

[Fabian Guisset]

Confirming build 2000112304 win32. I'm not sure we are supposed to be able to
write in it though, because you have to chose the file using "Choose...".
However I agree that it would be a great addition to have a second textfield
specifying the location of the program, so you could simply type in the local
url of the program to use, as it happens in most other softwares. What do you
think Boris?
Fabian.
Changing OS to all, marking new.

Comment 2

[Boris Zbarsky [:bzbarsky]]

In every single other instance of such fields (eg the file upload field or even 
the field in which you enter the application under Preferences -> Helpers), you 
can type in the text field.  So we should be consistent with that behavior.

Comment 3

[Fabian Guisset]

A simple matter of removing readonly="true" in
source/xpfe/components/ucth/resources/helperAppLauncher.xul
However there is a reason (that I ignore) for this readonly to be there...
Fabian.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Attachment: patch to xpfe/components/ucth/resources/helperAppLauncher.xul to remove readonly="true"

Comment 5

[Boris Zbarsky [:bzbarsky]]

I can't think of a good reason that should be there....
CVS blame shows that the readonly="true" was there in the original revision of
the file.

CVS comments do not list a reason for that to be there.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Actually, the patch I have provided based on Fabian's observation fixes nothing
useful (again, per Fabian's observation).

The application to be used comes from an internal variable in
helperAppLauncher.js instead of coming from the textfield, so just modifying the
textfield by typing in it is not very helpful...

I'll talk to mpt about this and if he says that the current behavior needs to be
changed I'll look for a way to make this work more like the file upload form field.

Comment 7

[sairuh (rarely reading bugmail)]

handing over to mscott, at least to review the patch --pls change if this
shouldn't be yours. thx!

Comment 8

[Boris Zbarsky [:bzbarsky]]

This is actually a duplicate of bug 49787, but I'm not sure which direction I
should dup in.

Comment 9

[Scott MacGregor]

*** This bug has been marked as a duplicate of 49787 ***

Comment 10

[[e-mail] (gone - use [e-mail] instead)]

verified dup.