Project

General

Profile

Actions

feature request #7912

closed

Enhance checkbox tree viewer

Added by Patrick Plitzner over 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
New
Assignee:
Patrick Plitzner
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal

Description

Implement buttons:

  • to (de-)select all children
  • select all children recursively or only direct children

Visualize selection state:

  • Text of tree node
    • italic when some children are selected
    • bold when all children are selected
    • default otherwise
  • Checkboxes
    • checked when item is selected
    • grayed when item is not selected but some or all children are selected

Files

picture672-1.png (6.85 KB) picture672-1.png Andreas Müller, 01/27/2019 09:16 AM
picture900-1.png (3.06 KB) picture900-1.png Andreas Müller, 02/01/2019 12:22 PM

Related issues

Related to EDIT - feature request #7063: Improve the area selection wizard for distribution editorClosedKatja Luther

Actions
Copied to EDIT - feature request #8054: Remaing issues - CheckBoxTreeCompositeNewKatja Luther

Actions
Actions #1

Updated by Patrick Plitzner over 5 years ago

  • Status changed from New to Resolved
  • Assignee changed from Patrick Plitzner to Andreas Müller
  • % Done changed from 0 to 50
Actions #2

Updated by Patrick Plitzner over 5 years ago

  • Assignee changed from Andreas Müller to Katja Luther
Actions #3

Updated by Katja Luther over 5 years ago

  • Status changed from Resolved to Feedback

this seems to work, but the checkboxes are not grayed when a child or some children are selected.

Actions #4

Updated by Katja Luther over 5 years ago

  • Assignee changed from Katja Luther to Patrick Plitzner
Actions #5

Updated by Patrick Plitzner over 5 years ago

  • Assignee changed from Patrick Plitzner to Katja Luther

Katja Luther wrote:

this seems to work, but the checkboxes are not grayed when a child or some children are selected.

Setting the checkboxes to gray seems to be platform dependant. On the Lubuntu linux distribution, for example, this state cannot be shown. Therefore the font is changed according to the selection state. bold if all children are selected, italic if some children are selected and standard if nothing is selected.

Can you please test in some cases if this works for you especially when re-opening a dialog with saved areas? If the grayed state is not that important we may close this ticket.

Actions #6

Updated by Andreas Müller over 5 years ago

  • Status changed from Feedback to Resolved
  • Assignee changed from Katja Luther to Andreas Müller

To remind me to review.

Actions #7

Updated by Andreas Müller about 5 years ago

It works already very good. 2 minor issues:

  1. The state check should always include all children AND recursive children. E.g. here

MyAreas should have state "some" not "all" as Area1 is not selected (though all of it's subareas are selected so the current behaviour has some logic).

  1. Even if greying the checkbox does not work on Ubuntu for some reason I think we should still try to do it and see if it works e.g. on Windows. A grey checkbox is more intuitive then italics. But of course we will keep the italics too to make sure that the dialog generally works on all systems.
Actions #8

Updated by Andreas Müller about 5 years ago

Patrick Plitzner wrote:

Implement buttons:

  • to (de-)select all children
  • select all children recursively or only direct children

The second issue does not seem to be implemented yet, but it is helpful if you want to check down to a certain hierarchie level.

Actions #9

Updated by Patrick Plitzner about 5 years ago

  • Status changed from Feedback to Closed
  • Assignee changed from Patrick Plitzner to Andreas Müller
  • % Done changed from 50 to 80

all implemented

Actions #10

Updated by Andreas Müller about 5 years ago

  • Status changed from Closed to Resolved
Actions #11

Updated by Andreas Müller about 5 years ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Patrick Plitzner

Andreas Müller wrote:

  1. Even if greying the checkbox does not work on Ubuntu for some reason I think we should still try to do it and see if it works e.g. on Windows. A grey checkbox is more intuitive then italics. But of course we will keep the italics too to make sure that the dialog generally works on all systems.

The checkboxes are still not grey on Windows. Is this not possible? Then leave a message. Or just forgot?

Actions #12

Updated by Andreas Müller about 5 years ago

  • % Done changed from 80 to 90

All the rest works as expected now as far as I can see.

Actions #13

Updated by Andreas Müller about 5 years ago

There is still an issue with the usage e.g. for distribution editor. Here you may not want that the vocabulary itself can be selected. So the top level nodes must not be selectable only the state should change depending on the selection of the (recursive) children. But of course it would be nice to have the typical tristate behavior here (empty if no child is selected, unknown state if some children are selected and checked state if all children are selected.
I think it should be configurable if the topmost level can be selected or not.

Actions #14

Updated by Andreas Müller about 5 years ago

  • % Done changed from 90 to 80
Actions #15

Updated by Patrick Plitzner about 5 years ago

Andreas Müller wrote:

Andreas Müller wrote:

  1. Even if greying the checkbox does not work on Ubuntu for some reason I think we should still try to do it and see if it works e.g. on Windows. A grey checkbox is more intuitive then italics. But of course we will keep the italics too to make sure that the dialog generally works on all systems.

The checkboxes are still not grey on Windows. Is this not possible? Then leave a message. Or just forgot?

I cannot really test this because I have no working environment on Windows. If I just implement this blindly I am afraid of breaking something. Maybe I can pair up with a Windows user and try it there.

Actions #16

Updated by Patrick Plitzner about 5 years ago

  • Status changed from Feedback to Resolved
  • Assignee changed from Patrick Plitzner to Andreas Müller

Andreas Müller wrote:

There is still an issue with the usage e.g. for distribution editor. Here you may not want that the vocabulary itself can be selected. So the top level nodes must not be selectable only the state should change depending on the selection of the (recursive) children. But of course it would be nice to have the typical tristate behavior here (empty if no child is selected, unknown state if some children are selected and checked state if all children are selected.
I think it should be configurable if the topmost level can be selected or not.

This is implemented

Actions #17

Updated by Andreas Müller about 5 years ago

Patrick Plitzner wrote:

Andreas Müller wrote:

There is still an issue with the usage e.g. for distribution editor. Here you may not want that the vocabulary itself can be selected. So the top level nodes must not be selectable only the state should change depending on the selection of the (recursive) children. But of course it would be nice to have the typical tristate behavior here (empty if no child is selected, unknown state if some children are selected and checked state if all children are selected.
I think it should be configurable if the topmost level can be selected or not.

This is implemented

OK, this works. Only it feels a bit strange that there is a checkbox which is always empty and can not be clicked and at the same time all the childrens are selected, e.g.:

So best would be to handle this checkbox as standard checkbox with in trees with unchecked=no child selected, checked=all children selected, tristate=some children selected. Alternatively, if possible, we could simply remove the checkbox.
However, this has no high priority, so if not easy to implement we can move it to a follow up ticket.

Actions #18

Updated by Andreas Müller about 5 years ago

Patrick Plitzner wrote:

Andreas Müller wrote:

Andreas Müller wrote:

  1. Even if greying the checkbox does not work on Ubuntu for some reason I think we should still try to do it and see if it works e.g. on Windows. A grey checkbox is more intuitive then italics. But of course we will keep the italics too to make sure that the dialog generally works on all systems.

The checkboxes are still not grey on Windows. Is this not possible? Then leave a message. Or just forgot?

I cannot really test this because I have no working environment on Windows. If I just implement this blindly I am afraid of breaking something. Maybe I can pair up with a Windows user and try it there.

@Katja: could you have a look if this can be done on Windows somehow. Also not high priority.

Actions #19

Updated by Patrick Plitzner about 5 years ago

I created a new ticket for the top-level checkbox -> #8054

Actions #20

Updated by Patrick Plitzner about 5 years ago

Andreas Müller wrote:

Patrick Plitzner wrote:

Andreas Müller wrote:

Andreas Müller wrote:

  1. Even if greying the checkbox does not work on Ubuntu for some reason I think we should still try to do it and see if it works e.g. on Windows. A grey checkbox is more intuitive then italics. But of course we will keep the italics too to make sure that the dialog generally works on all systems.

The checkboxes are still not grey on Windows. Is this not possible? Then leave a message. Or just forgot?

I cannot really test this because I have no working environment on Windows. If I just implement this blindly I am afraid of breaking something. Maybe I can pair up with a Windows user and try it there.

@Katja: could you have a look if this can be done on Windows somehow. Also not high priority.

@Katja: We can do this together next week.

Actions #21

Updated by Andreas Müller about 5 years ago

Actions #22

Updated by Patrick Plitzner about 5 years ago

  • Status changed from Feedback to Closed
  • % Done changed from 80 to 100

Added remaining issues to #8054

Actions #23

Updated by Andreas Müller about 5 years ago

Please always add the follow up tickets to the "related issues" section, too.

Actions #24

Updated by Andreas Müller about 5 years ago

Actions

Also available in: Atom PDF