Community
Participate
Working Groups
The Target Editor's Overview page has a section titled "Additional Locations". This section contains a list, but sadly since the section does not consume the available vertical whitespace, you get scrollbars if you have more than 4 items in the list. Please make the "Additional Locations" section consume all, or more of, the available vertical whitespace.
how many locations do you have? a thousand? ;)
Not quite thousands... In one case we have 6 additional locations. But nevertheless, whenever there's a list widget and available vertical whitespace, I like the list to be whitespace-greedy.
Adam Archer, meet Simon Archer.
Adam: In addition to sharing your name, I'm a member of "The Friends of PDE"... don't let anyone tell you otherwise. ;-)
Brian to assist Bartek and review his patch.
Created attachment 75361 [details] example of how to resize this section Overview page is controlled by TableWrapLayout and to compare extensions page is controlled by GridLayout which allows components to expand to all available space The only thing I've been able to do was to resize it to static specified height in table wrap layout. please correct me if I'm wrong.
Brian, please review.
adding contributed keyword
Hi Bartosz, unfortunately I think there was a miscommunication on what section needed to be edited. The patch you supplied is for the Plug-in Manifest Editor. The editor in question is the Target Definition Editor. To open the editor, you can create a new Target Definition (New > Other > Target Definition). In this editor, in the middle of the Overview page there is an Additional Locations section. This is the section in discussed in the bug report. The class for this section is org.eclipse.pde.internal.ui.editor.target.LocationsSection. If you have any questions about this class or the Target Editor, please don't hesitate to post them.
Created attachment 75515 [details] path for the right component ;) This is a path for the right editor. Sorry for last misunderstanding. I'm waiting for change suggestions if any.
Bartosz, sorry about not applying this patch right away. It plays very well with the editor. The only thing thing I fixed were some really minor spacing issues. Thanks a lot!
