Bug 328275 - Description for the Deployment and Assembly page, is not accessible and not Wrapped
Summary: Description for the Deployment and Assembly page, is not accessible and not W...
Status: RESOLVED FIXED
Alias: None
Product: WTP Common Tools
Classification: WebTools
Component: wst.common (show other bugs)
Version: 3.2   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 3.2.3   Edit
Assignee: Carl Anderson CLA
QA Contact: Carl Anderson CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2010-10-20 13:47 EDT by Zina Mostafia CLA
Modified: 2010-10-26 22:17 EDT (History)
1 user (show)

See Also:
ccc: review+


Attachments
Patch to fix Deployment and Assembly property page Description text (1.41 KB, patch)
2010-10-20 14:22 EDT, Zina Mostafia CLA
ccc: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zina Mostafia CLA 2010-10-20 13:47:29 EDT
Build Identifier: 3.2.2

Deployment and Assembly property pages might add some description on the top of that page to what is called the Description Composite.
This would use this method to show the Description Text: 
org.eclipse.wst.common.componentcore.ui.propertypage.ModuleAssemblyRootPage.fillDescription to set the text.
There are two problems with this method:
1) The text does not wrap, so if the description is long, it would be cutoff
2) The method is using Label instead of Text, which is not accessible.

We need a simple fix to allow wrapping the text properly, and use SWT Text to allow it to be accessible.

Reproducible: Always

Steps to Reproduce:
1. Create a Web Module with an EAR
2. Select the EAR and go to the Properties > Deployment and Assembly page
3. Make the Page smaller and you would see that the text would not wrap properly, same behavior if the text is translated to a long line, or other extenders of this page would like to add a longer description.
4. Use the keyboard to navigate through that page, you will see that the focus would never be on the Text, and that causes page readers not to read it.
Comment 1 Zina Mostafia CLA 2010-10-20 14:22:15 EDT
Created attachment 181325 [details]
Patch to fix Deployment and Assembly property page Description text

Patch to fix the problem is attached.
Comment 2 Carl Anderson CLA 2010-10-26 22:17:06 EDT
Committed to R3_2_maintenance and HEAD for WTP 3.2.3 and WTP 3.3

Thanks for the contribution!