[jira] Commented: (GERONIMODEVTOOLS-325) "Download and Install" button should be disabled/enabled appropriately

["B.J. Reed (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GERONIMODEVTOOLS-325?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12619526#action_12619526 ] 

B.J. Reed commented on GERONIMODEVTOOLS-325:
--------------------------------------------

This is an improvement.  I had a problem installing the patch, specifically, there is a snippet of code that was changed from 

ConfirmInstallDialog dialog = new ConfirmInstallDialog(shell, getRuntimeName(), installDir.getText(), license);

to 

ConfirmInstallDialog dialog = new ConfirmInstallDialog(shell, getRuntimeName(), installDir, license);

This line did not compile.  Changing it back made it compile.

After making that change, the processing seems a little better, but it would be nice to have the "download and install" button visually grayed out so that the user knew by looking at it that it is not enabled. Also what happens now if I want to download to a new directory doesn't really seem very good.  Clarifying my steps and what I see

I want to download into a new direcotry of c:\f

1. type in C:\f\geronimo-tomcat6-javaee5-2.1.1 in the entry field and click "download and install" since it looks enabled (nothing happens).
2. Click "Browse" and create the necessary folders
3. Now clicking "download and install" brings up the same dialog I just had when I clicked on the browse button.  (why is this dialog here?  I already have the correct text in the entry field for where I want to download to)  Then it proceeds to the license and downloading.

> "Download and Install" button should be disabled/enabled appropriately
> ----------------------------------------------------------------------
>
>                 Key: GERONIMODEVTOOLS-325
>                 URL: https://issues.apache.org/jira/browse/GERONIMODEVTOOLS-325
>             Project: Geronimo-Devtools
>          Issue Type: Bug
>          Components: eclipse-plugin
>    Affects Versions: 2.1.2
>            Reporter: B.J. Reed
>            Assignee: B.J. Reed
>            Priority: Trivial
>             Fix For: 2.1.2
>
>         Attachments: GERONIMODEVTOOLS-325.patch, GERONIMODEVTOOLS-325_updated.patch, GERONIMODEVTOOLS-325_updated_latest.patch
>
>
> After selecting which server I want to use, the "Application Server Installation Directory" field is blank and the "Download and Install" button is enabled.  Hitting this button does nothing -1) it should be disabled on the page when the page is first shown.
> If I enter a directory that does not exist, a message shows at the top of the dialog saying "The specified directory does not exist".  2) the button should be disabled when that message appears and enabled when one of the "download allowable" messages appears (there seem to be several).
> 3) Also, in stead of disabling for 2), it would be slightly more usable to allow the user to click the "Download and Install" button and then ask the user if they wanted to create that directory.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.