Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 213639

Summary:	Remove the "open editor" button on the Report Wizard first page
Product:	z_Archived	Reporter:	Liz Dancy <lizdancy>
Component:	TPTP	Assignee:	Paul Slauenwhite <paulslau>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P1	CC:	labadie, paulslau, zung
Version:	unspecified	Keywords:	plan
Target Milestone:	---
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Liz Dancy

2007-12-20 15:46:37 EST

This button can be confusing. It also prevents classes which extend the wizard from controlling the editor which gets opened when the report is created.

Comment 1 Sheldon Lee-Loy

2007-12-20 16:00:30 EST

This bug is assigned the wrong component .  This bug should be assigned to Platform.UI.Reporting under the TPTP Testing product.

Comment 2 Sheldon Lee-Loy

2007-12-20 16:01:12 EST

Reassigning owner to Platform.UI.Reporting component owner.

Comment 3 Paul Slauenwhite

2007-12-20 21:55:47 EST

Hi Liz, are you looking for:

-the button/check box to be removed and the report is always opened in the default editor 

or

-to make the default editor configurable and the label for the button/check box easier to understand.

Comment 4 Paul Slauenwhite

2007-12-20 22:06:06 EST

(In reply to comment #3)

> -to make the default editor configurable and the label for the button/check box
> easier to understand.

For example, have an Open Report Automatically check box, similar to the test reports.

Comment 5 Liz Dancy

2007-12-21 11:05:22 EST

(In reply to comment #3)
> Hi Liz, are you looking for:
> 
> -the button/check box to be removed and the report is always opened in the
> default editor 
> 
> or
> 
> -to make the default editor configurable and the label for the button/check box
> easier to understand.
> 

I think the second option would be ideal. Thanks Paul.

Comment 6 Paul Slauenwhite

2008-01-22 12:36:21 EST

The type of editor that is opened when the report is generated is configurable by overriding the org.eclipse.hyades.ui.report.ReportGeneratorWizard.getEditorId() method.

Changed the label for the check box in the test/log/trace/stats report wizards to be consistent and clearer.

Fixed a bug with the trace reports not storing/using the user preference for the check box.

The following files checked in to CVS (HEAD):

org.eclipse.hyades.ui.internal.wizard.HyadesReportSelectionWizardPage.java
/org.eclipse.hyades.test.ui/src/org/eclipse/hyades/test/ui/internal/resources/messages.properties
/org.eclipse.tptp.platform.common.ui/src-common-internal/org/eclipse/tptp/platform/common/ui/internal/messages.properties

Comment 7 Paul Slauenwhite

2009-02-25 11:58:33 EST

Closing by default since not closed by the originator in the 7+ months since
being resolved.  

Please reopen if the issue is still present in the latest TPTP release or the
resolution is not correct.