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

Bug 344577

Summary:

UI glitches on 'API Use Report' launch configuration page

Product:

[Eclipse Project] PDE

Reporter:

Dani Megert <daniel_megert>

Component:

API Tools

Assignee:

Michael Rennie <Michael_Rennie>

Status:

VERIFIED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

curtis.windatt.public

Version:

3.7

Flags:

curtis.windatt.public: review+

Target Milestone:

3.7 RC1

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
fix	Michael_Rennie: review?

Description Dani Megert

2011-05-03 10:02:37 EDT

3.7 M7.

- mnemonic clashes for 'n', 'c', 'y' (BAD as it hides Apply)
- labels in sentence style (e.g. "Report Output Location", "Report Type")

Comment 1 Michael Rennie

2011-05-04 15:21:22 EDT

Created attachment 194758 [details]
fix

patch fixes up the mnemonic issues and the casing for the use and pattern tab and some pattern wizard pages.

Comment 2 Curtis Windatt

2011-05-04 16:05:00 EDT

-1 for the new sentence.  

"Initialize report directory before reporting new results"

The sentence sounds fine, but it doesn't convey the correct meaning.  When I read it my first thought is that the option is asking to create the directory structure, not delete the previous data.

With only J, K, Q, Z available, I would rather not have a mnemonic over using initialize.

Everything else in the patch is good though.

Comment 3 Curtis Windatt

2011-05-09 15:42:02 EDT

+1 for putting the patch in without the sentence change.

Comment 4 Michael Rennie

2011-05-09 15:56:59 EDT

Committed patch without the sentence change and removing the mnemonic from the sentence.

Comment 5 Ankur Sharma

2011-05-16 15:34:36 EDT

Verified in I20110512-2000