Project

General

Profile

bug #8662

"Remove Taxon" in DescriptiveDataSetEditor should only be visible if taxon is selected

Added by Patrick Plitzner 11 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Highest
Assignee:
Category:
taxeditor
Target version:
Start date:
11/07/2019
Due date:
% Done:

90%

Severity:
normal
Found in Version:
Tags:

Description

… im Taxon filter als Kontextmenü. Auch wenn noch gar kein Taxon dort angezeigt wird.

Auch wenn Taxa existieren, wird das Kontextmenü angezeigt und willkürlich das erste Taxon gewählt wenn man gar nicht auf ein Taxon geklickt hat. Das ist gefährlich, da man im schlimmsten Fall gar nicht mitbekommt, dass man das Taxon ausgewählt hat. Das Menü sollte nur angezeigt werden, wenn man auf das entsprechende Taxon klickt.

Associated revisions

Revision b529af04 (diff)
Added by Katja Luther 5 months ago

fix #8662: adapt canExecute to show remove taxon only when top level node is selected

History

#1 Updated by Andreas Müller 10 months ago

  • Assignee changed from Patrick Plitzner to Katja Luther
  • Priority changed from New to Highest
  • Target version changed from 287 to Release 5.12

#2 Updated by Katja Luther 9 months ago

  • Target version changed from Release 5.12 to Release 5.13

#3 Updated by Andreas Müller 7 months ago

  • Target version changed from Release 5.13 to Release 5.14

#4 Updated by Andreas Müller 6 months ago

  • Target version changed from Release 5.14 to Release 5.15

#5 Updated by Katja Luther 5 months ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50

#6 Updated by Katja Luther 5 months ago

  • Assignee changed from Katja Luther to Andreas Müller

now the menu item is available only if the selection is a taxon node, so it should show up, only when the node can be removed (only the top level nodes).

please review.

#7 Updated by Andreas Müller 4 months ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther
  • % Done changed from 50 to 90

If the tree list is empty it works now as expected.

Auch wenn Taxa existieren, wird das Kontextmenü angezeigt und willkürlich das erste Taxon gewählt wenn man gar nicht auf ein Taxon geklickt hat. Das ist gefährlich, da man im schlimmsten Fall gar nicht mitbekommt, dass man das Taxon ausgewählt hat. Das Menü sollte nur angezeigt werden, wenn man auf das entsprechende Taxon klickt.

When clicking somewhere in the tree controle still the first entry gets the focus even if it was never clicked. This is not really expected. However, it is the same in taxon navigator (but there it is more critical). So for now in the DescriptiveDataSetEditor we can probably live with it.

Please decide if you want to close the ticket or fix the remaining issue.

#8 Updated by Katja Luther 4 months ago

  • Status changed from Feedback to Closed

Andreas Müller wrote:

If the tree list is empty it works now as expected.

Auch wenn Taxa existieren, wird das Kontextmenü angezeigt und willkürlich das erste Taxon gewählt wenn man gar nicht auf ein Taxon geklickt hat. Das ist gefährlich, da man im schlimmsten Fall gar nicht mitbekommt, dass man das Taxon ausgewählt hat. Das Menü sollte nur angezeigt werden, wenn man auf das entsprechende Taxon klickt.

When clicking somewhere in the tree controle still the first entry gets the focus even if it was never clicked. This is not really expected. However, it is the same in taxon navigator (but there it is more critical). So for now in the DescriptiveDataSetEditor we can probably live with it.

Please decide if you want to close the ticket or fix the remaining issue.

The selection is set by the treeViewer and I could not find a solution to avoid this, therefore I close this ticket. If the users complain about it, we have to do some more research to find a solution.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)