Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 321131 - [Usability] Remove extra column in Attribute/Constant/Method list of editors
Summary: [Usability] Remove extra column in Attribute/Constant/Method list of editors
Status: VERIFIED FIXED
Alias: None
Product: Tigerstripe
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 0.5M0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-28 10:25 EDT by Navid Mehregani CLA
Modified: 2010-12-13 12:20 EST (History)
3 users (show)

See Also:

Attachments
Editor (69.54 KB, image/jpeg)
2010-07-28 10:26 EDT, Navid Mehregani CLA 		no flags Details
patch_321131.txt (3.33 KB, patch)
2010-11-09 17:27 EST, Daniel Johnson CLA 		no flags Details | Diff
321131_2 patch (9.39 KB, patch)
2010-12-07 05:06 EST, Anton Salnik CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Navid Mehregani CLA 2010-07-28 10:25:12 EDT 
Build Identifier: 

In most editors (e.g. Entity form editor) there is now an extra column displayed under Attribute list.  This is most likely due to the recent changes in bug#320416.  
The extra column needs to be removed.  Please see screenshot for more details.

Reproducible: Always
Comment 1 Navid Mehregani CLA 2010-07-28 10:26:35 EDT 
Created attachment 175412 [details]
Editor

Notice extra column under Attribute list.
Extra column also appear under Constant and Method lists.
Comment 2 Daniel Johnson CLA 2010-11-09 17:27:30 EST 
Created attachment 182778 [details]
patch_321131.txt

Added weights to artifact editor window so tables takes up 33% of screen and the rest is for editing the values selected in the tables.
Classes Changed: 
ArtifactAttributesSection
ArtifactConstantsSection
ArtifactMethodsSection
Comment 3 Navid Mehregani CLA 2010-11-18 13:13:44 EST 
Thanks Dan! I've committed your patch, but I think I'm going to keep this defect open.  Now that I see the result, I think it would be better to remove the extra column altogether.  I know this isn't what I said originally, but I thought it would look better if we simply adjusted the widths.  We'll treat this defect as a low priority item.
Comment 4 Anton Salnik CLA 2010-12-07 05:06:18 EST 
Created attachment 184700 [details]
321131_2 patch

Attibutes/constants/methods table columns widths adjusted. Patch attached.
Comment 5 Yuri Strot CLA 2010-12-13 05:40:01 EST 
Patch 321131_2 applied.
Comment 6 Navid Mehregani CLA 2010-12-13 12:20:02 EST 
Thanks Anton.  Looks good!