Bug 171897 - [Schema][Editors] Definition page of schema editor is initially heavy on the left
Summary: [Schema][Editors] Definition page of schema editor is initially heavy on the ...
Status: RESOLVED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 RC1   Edit
Assignee: Noam Chitayat CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
: 174668 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-01-27 00:51 EST by Wassim Melhem CLA
Modified: 2007-06-05 18:46 EDT (History)
2 users (show)

See Also:

Attachments
Patch for bug 171897. (Affects ElementSection.java) (902 bytes, patch)
2007-05-08 15:50 EDT, Noam Chitayat 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 Wassim Melhem CLA 2007-01-27 00:51:16 EST 
1. Open an extension point schema in the schema editor.  Any schema will do (e.g. the newExtension.exsd file in the schema/folder of the org.eclipse.pde.ui plug-in)

2. Switch to the Definition page.

Note how the page is heavy to the left and empty on the right

Since the tree to the left has content, it would be good to select the first item in the tree when the page is initialized so that the right hand side gets populated and the page looks even.
Comment 1 Wassim Melhem CLA 2007-01-27 00:52:09 EST 
assigning to Sean and cc Mike to review/release.
Comment 2 Mike Pawlowski CLA 2007-02-19 14:24:32 EST 
*** Bug 174668 has been marked as a duplicate of this bug. ***
Comment 3 Wassim Melhem CLA 2007-05-03 19:21:48 EDT 
Re-re-assigning to Noam.  Mike, please review.
Comment 4 Noam Chitayat CLA 2007-05-08 15:50:01 EDT 
Created attachment 66354 [details]
Patch for bug 171897. (Affects ElementSection.java)

Added a check in ElementSection#refresh() that will check if any XML elements are selected. If the list of elements is non-empty and there is no element selected, then the first element on the list is selected automatically.
Comment 5 Wassim Melhem CLA 2007-05-08 15:54:33 EDT 
cc Mike to review.
Comment 6 Wassim Melhem CLA 2007-05-08 23:45:11 EDT 
Works well.  Thanks Noam.