Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 361413 - [patch] "Select All" is missing from "Edit Software site" dialog in target definition
Summary: [patch] "Select All" is missing from "Edit Software site" dialog in target de...
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.2   Edit
Hardware: PC Windows 7
: P3 normal with 2 votes (vote)
Target Milestone: 4.5 M6   Edit
Assignee: pragya gaur CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2011-10-19 10:48 EDT by Tobias Bertelsen CLA
Modified: 2015-03-16 07:59 EDT (History)
6 users (show)

See Also:


Attachments
Added Select All and DeSelect All buttons in "Edit Software Site" dialog in target definition (3.72 KB, patch)
2014-08-22 04:45 EDT, pragya gaur CLA
no flags Details | Diff
Last patch in sync with master code (3.82 KB, patch)
2015-02-05 09:47 EST, Vikas Chandra CLA
no flags Details | Diff
Have done the necessary changes. Please review it. (3.86 KB, patch)
2015-02-09 08:25 EST, pragya gaur CLA
no flags Details | Diff
Last patch aligned with master code (3.69 KB, patch)
2015-03-14 10:45 EDT, Vikas Chandra CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Bertelsen CLA 2011-10-19 10:48:16 EDT
Build Identifier: I20110613-1736

When installing new plugins from a software site, there is some nice "select all" and "deselect all" buttons, but they are missing when adding/editing a software site in the target definition.

This is really missed when combining Bug 275999 and Bug 361412

Reproducible: Always

Steps to Reproduce:
1. Create a target definition file
2. Add->Software site
3. Compare the "Edit Software Site" dialog to the "Install" dialog (Help->Install New Software)
Comment 1 Curtis Windatt CLA 2011-10-19 11:57:34 EDT
Yes, they are missing and should probably be added.  It's probably not very difficult to copy the buttons (the UI reuses as much of the p2 components as possible).  I don't have time for this at the moment, so I will mark it as help wanted.
Comment 2 Michael Vorburger CLA 2012-06-08 08:56:19 EDT
+1 for this... sorry to rant, but that UI *really* is a pain and could use a bit of love.  Add All, but ideally it would be great if the "Edit Software site" UI looked and worked like one is familiar with from a PDE Launch Configuration Plug-ins tab, with Select/Deselect All and also an [X] Only show selected option, and if possible an Add Required and Validate.
Comment 3 pragya gaur CLA 2014-08-22 04:45:25 EDT
Created attachment 246239 [details]
Added Select All and DeSelect All buttons in "Edit Software Site" dialog in target definition
Comment 4 pragya gaur CLA 2014-10-22 06:10:06 EDT
Hi Vikas,
Please review my submission...
Comment 5 Vikas Chandra CLA 2014-12-03 03:23:41 EST
Planning to review this in 4.5M5
Comment 6 Vikas Chandra CLA 2015-02-05 09:47:01 EST
Created attachment 250533 [details]
Last patch in sync with master code

I have put the last patch in sync with master code.
Comment 7 Vikas Chandra CLA 2015-02-05 09:50:44 EST
There are few issues with this patch

1) When you do select all, xyz items selected. The part selected is not fully visible

2) Deselect all doesnt give default 0 item selected

3) There is a TODO block in  the code. Is that part of code taken from somewhere else in eclipse.

4) There is a deprecated api use. Are there no other options?
Comment 8 pragya gaur CLA 2015-02-06 09:47:04 EST
I do not get what you mean by:
1) When you do select all, xyz items selected. The part selected is not fully visible
Do you mean that the selected tree items should be expanded??

Also I will add default 0 selected.

And I have taken this code from Install new Software available under Help toolbar item. I am looking into finding another API for achieving this functionality since its deprecated.
Comment 9 Vikas Chandra CLA 2015-02-06 10:04:14 EST
It shows "10 items selec" instead of "10 items selected". The rest of the text is hidden.
Comment 10 pragya gaur CLA 2015-02-09 08:25:58 EST
Created attachment 250620 [details]
Have done the necessary changes. Please review it.
Comment 11 Vikas Chandra CLA 2015-03-14 10:45:21 EDT
Changed patch to be in sync with master, Added license year, changed access type of some functions for consistency

Fixed with http://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=5e403d52449492c6b9613ae36a446447ce6a280b

Pragya,
1) Please put a patch which is with respect to latest code
2) Change license year to 2015


Looks like a minimal risk change. Tested this to be working fine and now this is in sync with Available Software dialog.
Comment 12 Vikas Chandra CLA 2015-03-14 10:45:52 EDT
Created attachment 251554 [details]
Last patch aligned with master code
Comment 13 Vikas Chandra CLA 2015-03-16 07:59:43 EDT
Verified in
Version: Mars (4.5)
Build id: N20150315-1630