Bug 339500 - [client] No Select All/Deselect All in the file table
Summary: [client] No Select All/Deselect All in the file table
Status: RESOLVED WONTFIX
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: Client (show other bugs)
Version: 0.2   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: libing wang CLA
QA Contact:
URL:
Whiteboard:
Keywords: usability
: 370437 (view as bug list)
Depends on: 377018
Blocks: 369732
  Show dependency tree
 
Reported: 2011-03-10 07:05 EST by Szymon Brandys CLA
Modified: 2015-05-05 14:52 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Szymon Brandys CLA 2011-03-10 07:05:12 EST
It would be good to have an option to Select/Deselect All files in the navigator. I think it is an issue with all tables with checkboxes.
Comment 1 Susan McCourt CLA 2011-04-19 16:44:57 EDT
reassigning all bugs without milestones to inbox.  I'm not sure this is important for our polish pass.
Comment 2 Susan McCourt CLA 2011-10-03 11:46:44 EDT
we have something odd in the git status page that works around this limitation, so I think we need to provide basic support so that all pages will behave consistently with respect to checkbox selection.
Comment 3 Susan McCourt CLA 2012-02-02 10:49:58 EST
*** Bug 370437 has been marked as a duplicate of this bug. ***
Comment 4 Susan McCourt CLA 2012-02-02 13:40:26 EST
as much as I wanted to see this get in, I just don't have time.
Comment 5 libing wang CLA 2012-02-02 15:56:25 EST
I have a logic in search replace now. I've implemented some call backs in search explorer. Lemme see if I can easily generalize it.
Comment 6 libing wang CLA 2012-02-02 15:59:46 EST
 Created bug 370496 , I targeted it for 0.4 just for remnider for now. 
If I can fix that then we can close this for 0.4.
Comment 7 Susan McCourt CLA 2012-02-03 10:06:11 EST
Libing, note that "select all" as described here is not the "parent selects children" model, but rather the general "checkbox in the heading that selects everything."

There are several things in play:

- including the header checkbox that triggers select all/deselect all.  Is this in the table header itself or does it appear on the toolbar (like gmail)
- we need to determine how "deep" a select all is...is it just the exposed nodes or would we actually traverse depthwise to understand the child structure.  I think this depends on what action is taken on select all.  For example moving a folder implicitly moves its children so select all/move would we deep.  But if we did select all/make favorite it's probably just the visible items.  Something to think through.
 

Implicit in this is that we would change git status table to use the generic support rather than what is implemented now.
Comment 8 Susan McCourt CLA 2012-02-22 12:10:38 EST
(In reply to comment #7)

> - including the header checkbox that triggers select all/deselect all.  Is this
> in the table header itself or does it appear on the toolbar (like gmail)


See bug 359730.  In general I think we want select all in toolbars, not headers.  This helps with (some) scrolling issues.
Comment 9 libing wang CLA 2012-02-22 12:31:29 EST
(In reply to comment #8)
> (In reply to comment #7)
> 
> > - including the header checkbox that triggers select all/deselect all.  Is this
> > in the table header itself or does it appear on the toolbar (like gmail)
> 
> 
> See bug 359730.  In general I think we want select all in toolbars, not
> headers.  This helps with (some) scrolling issues.

I think we should also consider the ability to select all on a folder. Replace page already had this and I added some supports in the general place.

Regarding the scrolling issues, the top level select all needs to be in the tool bar but in addition to that, not general though, we need to move information out of header as well. E.g. the result summary in search page.
Comment 10 Susan McCourt CLA 2012-04-19 14:14:43 EDT
per bug 377018 we are going to look at getting rid of checkboxes.
If that is true, then I think we simply need to support
Ctrl+A = select all.

For deselecting all, the user can just click on any item and they get automatic deselect.
Comment 11 Malgorzata Janczarska CLA 2012-04-20 04:56:18 EDT
(In reply to comment #10)
> per bug 377018 we are going to look at getting rid of checkboxes.
> If that is true, then I think we simply need to support
> Ctrl+A = select all.
> 
> For deselecting all, the user can just click on any item and they get automatic
> deselect.

How about Ctrl+A second time?
Comment 12 libing wang CLA 2012-05-22 09:53:10 EDT
With the selection model, the issue becomes really minor.
We need to think about a generic way in the selection model level to do this. But it is post 0.5.
Comment 13 John Arthorne CLA 2015-05-05 14:52:56 EDT
Closing as part of a mass clean up of inactive bugs. Please reopen if this problem still occurs or is relevant to you. For more details see:

https://dev.eclipse.org/mhonarc/lists/orion-dev/msg03444.html