If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Tidy up Helper App dlg UI

VERIFIED WONTFIX

Status

Core Graveyard
File Handling
P3
normal
VERIFIED WONTFIX
17 years ago
a year ago

People

(Reporter: Ben Goodger (use ben at mozilla dot org for email), Assigned: Ben Goodger (use ben at mozilla dot org for email))

Tracking

({polish})

Trunk
mozilla1.0
polish

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

There are some text formatting and focus ring issues in the helper app dialog, 
and it is a little too wide. Also, the keyset id is wrong so keybindings don't 
work. Here's a patch to address the issues:
Created attachment 33827 [details] [diff] [review]
patch
Bill, can you review this? 
Alec, sr? 
Status: NEW → ASSIGNED
Created attachment 33828 [details] [diff] [review]
sorry, use this patch. noticed a glitch.
Blocks: 78106

Comment 4

17 years ago
I'm worried that splitting up the "intro" into those smaller pieces might box in
translators.  Do you think it's OK to impose that ordering?  What's the benefit
of splitting this up this way?  I might go for it it if fixed the redisplay
problems (bug 79543).

Comment 5

17 years ago
your tabs are all messed up in your XUL, but the rest looks ok to me.

Comment 6

17 years ago
(just noticed bill's comments - didn't realize that was going on ... the
validity of my sr= is pending bill's approval of course)
I considered i18n issues, but figured that separate elements was only really
following an ordering in the english version. The patch implements a field-like UI:

Name               Value
Download:          <what>
from:              <where>

In the english translation, it happens to read like a sentence, and in other
translations this is probably possible too. In a language where this is not
possible, it's alternatively possible to specify what and where from as above. 
btw, the patch is a whitespace-indifferent diff, which explains the weird
indentation

Comment 9

17 years ago
I applied your patch but the dialog doesn't work right.  E.g., I see only
       applicatioin/octet-stream

       temp

       from #1
at the top (where I expected to see "You have downloaded a file...").

Also, If "Save to disk" is selected under "Use a different action for this
file", but "Use default action for this file" is selected, then for some reason
the radio button next to "save to disk" is invisible.

My diffs seem to match your patch so I don't think it's something I did.  It
seems like you'd have to do more code changes to deal with the different xul
structure and the composition of that intro text.
Hm, well since we're under i18n freeze anyhow, let's just ignore my wording
changes. I'll produce another patch that just modifies the layout, and we'll see
if that works... 
Aside from the wording, the dialog is too narrow using Modern on the Mac. All of
the buttons on the right are cropped out, including "OK" which makes the dialog
useless. Doing this:

Index: mozilla/embedding/components/ui/helperAppDlg/nsHelperAppDlg.xul
===================================================================
RCS file: /cvsroot/mozilla/embedding/components/ui/helperAppDlg/nsHelperAppDlg.xul,v
retrieving revision 1.3
diff -u -2 -r1.3 nsHelperAppDlg.xul
--- nsHelperAppDlg.xul	2001/04/16 22:04:24	1.3
+++ nsHelperAppDlg.xul	2001/05/16 23:24:40
@@ -41,5 +41,4 @@
         title="&caption.label;"
         onload="dialog.initDialog()"
-        style="width: 40em;"
         class="dialog"
         align="vertical">

at least makes the dialog intrinsically sized and it works.
Created attachment 34948 [details]
screenshot showing cropped dialog
display bugs that are related here:

* bug 79236 covers clipped buttons/textfields in this [helper app/download] dialog.
* bug 79889 covers clipped stuff in the download progress dialog.
* bug 79543 covers poor painting of the helper app/download dialog [and edit/new
mime dialog] after dismissing the edit/new mime dialog.
Keywords: nsbeta1, polish
OS: Windows 2000 → All
Hardware: PC → All

Updated

17 years ago
Priority: -- → P3
Target Milestone: --- → mozilla0.9.2

Updated

17 years ago
Keywords: nsbeta1 → nsbeta1+
OK, I have some better ideas here. Will post a new patch soon that fixes these 
issues and some others (like half the dialog UI getting shunted offscreen when 
there's a long file path)
Nav triage: 
I recommend this not be pushed back because of unaddressed layout issues that 
can frequently cause content to be pushed offscreen in this window. 

Setting duration to full day for revised XUL + testing. 
Whiteboard: Duration: 1 day.

Comment 16

17 years ago
again, remind you that german translation may need 150% of the space used by 
English becuase German sentance is usually longer. 

add rchen@netscape.com and msanz@netscape.com to the cc list.

Comment 17

17 years ago
style="width: 40em;"
may not work for german translation. Again, the goal for mozilla translation is 
not touching xul but only property file and dtd file. Please do not hard code 
size in xul based on English assumption. 

Comment 18

17 years ago
Adding danielmc and rsmyth to cc: list.
I plan to replace the path sections with readonly edit fields. 

the width of 40em shouldn't matter as text will wrap and the dialog will become
as tall as it needs to be to accommodate more text. 
If this isn't the place for this, let me know and I'll open a new bug.

When typing the application name into the last field in this dialog, entering
trailing spaces (and I presume leading spaces) leads to "Application not found"

Trailing and leading spaces should be removed.

Linux, 2001061308
Setting ETA. Estimated duration - 1-2 days. Need to:

a) convert fields with variable length text to use readonly text fields
b) Modify layout slightly. 
Whiteboard: Duration: 1 day. → ETA: 6/21
As per PDT discussion, moving P3 to mozilla0.9.3. 
Target Milestone: mozilla0.9.2 → mozilla0.9.3

Comment 23

17 years ago
nav triage team:

Pushing out to mozilla1.0 since we're redoing the helper app dialog
Whiteboard: ETA: 6/21
Target Milestone: mozilla0.9.3 → mozilla1.0
the bug for redoing the helper app dialog is bug 86640.
Component: XP Apps → File Handling
nevermind
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → WONTFIX
v
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.