| Summary: | [Usability] Missing/inconsistent titles/descriptions/icons in test report dialogs. | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Paul Slauenwhite <paulslau> | ||||||
| Component: | TPTP | Assignee: | Paul Slauenwhite <paulslau> | ||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||
| Severity: | normal | ||||||||
| Priority: | P1 | Keywords: | plan | ||||||
| Version: | unspecified | Flags: | paulslau:
review-
|
||||||
| Target Milestone: | --- | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
|
Description
Paul Slauenwhite
As discussed with originator I'm changing the target to future and priority to P2. Defect owner, please provide a sizing for this defect. We are using the Test.UI.Reporting component for test reports. Reassigning to Project Lead until a Lead Committer is assigned to this component. In TPTP 4.5, the BIRT Test Report generators was moved from a General Availability (GA) component to an As-Is component. As-Is components are primarily provided for prior users but imply no support (for example, defects, news group, and mailing lists) or commitment to triage or resolve opened defects. For this defect to be considered, please re-open with an attached patch including code to resolve the symptom and test cases to test the fix. Closing. Reopening to resolve non-BIRT test report issues. Deferred from I5 to I6 with PMC approval. Created attachment 91904 [details]
patch version 1
patch version 1
Hi paul, In the attached patch version 1, I've changed some inconsistent descriptions, added missing titles for those three reports mentioned in the description above and gave the other two reports more meaningful titles and there are still some problems: 1.for icon issue, could you find more meaningful icons for the report and add them? they should be added to the same place as the new title in the patch, and the code to add icons is like the one in ReportWindowWizard.java: setDefaultPageImageDescriptor(TestCommonImages.INSTANCE.getImageDescriptor(TestCommonImages.IMG_WIZBAN_REPORT_WINDOW)); 2.since those five reports are in different types, the error message are inconsistent, like messages for empty file name/empty project/invalid file name..etc. and the meeages are saved in different places, for xml report, it's saved in org.eclipse.tptp.platform.report.birt.ui.messages.properties for other two type: BIRT/Time Frame Historic&Test Pass, I imported all birt packages in platform and test folder, but still can't find the message Could you review the patch attached above and look at these two problems? Thanks a lot! Deferring to I7 as discussed by PMC/AG (http://wiki.eclipse.org/TPTP-AG-20080331#Test_Project). Patch reviewed (see patch V2). Tony, I am assigning this defect back to myself since I will handle #1 and #2 in comment #10 (see patch V2). Hours for creating and testing version 2 of the patch. Created attachment 95537 [details]
Patch V2
Patch V2 checked in to CVS (HEAD). Verified in TPTP-4.5.0-200806070100. Closing. |