This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 221549 - Status Reponse dialog box useability issues
Summary: Status Reponse dialog box useability issues
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Cosmos (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: amehrega CLA
QA Contact:
URL:
Whiteboard: useability
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-05 12:16 EST by Sheldon Lee-Loy CLA
Modified: 2012-01-03 13:47 EST (History)
2 users (show)

See Also:


Attachments
snapshot (48.44 KB, image/jpeg)
2008-03-05 12:17 EST, Sheldon Lee-Loy CLA
no flags Details
Patch (61.10 KB, patch)
2008-04-29 14:53 EDT, amehrega CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sheldon Lee-Loy CLA 2008-03-05 12:16:34 EST
Why is there a “Register” button under the response dialog?

What does cancel mean in this context?

We need to display a progress of the registration process.
Comment 1 Sheldon Lee-Loy CLA 2008-03-05 12:17:18 EST
Created attachment 91676 [details]
snapshot
Comment 2 Sheldon Lee-Loy CLA 2008-03-07 10:09:57 EST
Also the "Registration results:" is a text box.  The user is able to modify the text.  We shouldn't be using a text box in this case.  We should just make this a label.
Comment 3 Sheldon Lee-Loy CLA 2008-03-07 10:18:40 EST
Also the "Deregistration results:" is a text box.  The user is able to modify the text.  We shouldn't be using a text box in this case.  We should just make this a label.
Comment 4 Leonard Richardson CLA 2008-04-17 07:02:19 EDT
Removed the register button and changed the Cancel button to be an Ok button.

Registration and deregistration labels changed from a dojo text area to a label.
Comment 5 Sheldon Lee-Loy CLA 2008-04-17 14:03:16 EDT
(In reply to comment #4)
> Removed the register button and changed the Cancel button to be an Ok button.
> 
> Registration and deregistration labels changed from a dojo text area to a
> label.
> 

Hi Leonard,

I checked COSMOS-1.0.0-200804171200 build at http://build.eclipse.org/technology/cosmos/downloads/builds.php 

It doesn't seem that it has the changes you mentioned.  Were the checked in before 12 today EST?

Comment 6 Leonard Richardson CLA 2008-04-18 04:19:54 EDT
I have checked my build environment against what is in CVS and what is on the latest build on http://build.eclipse.org/technology/cosmos/downloads/builds.php and what is on the latest build on the cosmos development download page and they all have the changes that were checked in on Tuesday AM EST.

So I do not know why you are not seeing the changes.

Have you cleared your browser cache before launching?

Can you send me a screenshot?

Comment 7 Sheldon Lee-Loy CLA 2008-04-18 15:53:38 EDT
The partial registration and deregistration dialog box still have this problem.
Comment 8 Sheldon Lee-Loy CLA 2008-04-18 15:53:59 EDT
Targeting to i11 since this is not a blocking issue for i10.
Comment 9 amehrega CLA 2008-04-29 14:52:22 EDT
Taking over this defect
Comment 10 amehrega CLA 2008-04-29 14:53:53 EDT
Created attachment 98044 [details]
Patch

Sheldon,

Please review the patch (which also includes a fix for 228270)
Comment 11 Sheldon Lee-Loy CLA 2008-05-05 10:48:48 EDT
(In reply to comment #10)

A couple of questions with the patch.


1.  You modified org.eclipse.cosmos.dc.cmdbf.services.internal.registration.transform.output.artifacts.Response as part of the patch.  Does this file need to change to fix this bug?

2.  The updated status response dialog does not show up for the full deregistratoin, partial registration and partial deregistration cases.
Comment 12 amehrega CLA 2008-05-06 09:45:12 EDT
See my comments on bug 228270 
Comment 13 amehrega CLA 2008-05-14 14:49:31 EDT
Fix checked into CVS - HEAD
Comment 14 Sheldon Lee-Loy CLA 2008-06-20 15:45:30 EDT
Verified bug with build COSMOS-1.0.0-200806201443
Comment 15 Sheldon Lee-Loy CLA 2008-07-28 13:10:37 EDT
Marked as closed based on previous comments.