Project

General

Profile

Actions

feature request #7439

closed

Use NatTable for BulkEditor

Added by Patrick Plitzner almost 6 years ago. Updated over 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

Use NatTable composite for displaying the information in the bulk editor instead of the current SWT table viewer.

For example code look at CharacterMatrix class


Files

picture998-1.png (7.63 KB) picture998-1.png Andreas Müller, 06/15/2018 09:00 PM
picture257-1.png (22.2 KB) picture257-1.png Andreas Müller, 06/20/2018 10:31 AM
picture970-1.png (12.3 KB) picture970-1.png Andreas Müller, 06/20/2018 10:37 AM
picture942-1.png (6.85 KB) picture942-1.png Andreas Müller, 06/26/2018 11:58 AM

Related issues

Related to EDIT - feature request #6321: Implement parallel loading + progress bar for loading objects into the bulk editorClosedPatrick Plitzner

Actions
Related to EDIT - bug #7484: Protected titleCache handling not correct in Reference detail viewClosedKatja Luther

Actions
Related to EDIT - feature request #7485: Use tree table for reference and author bulk editor NewKatja Luther

Actions
Related to EDIT - feature request #7506: Allow direct editing bulk editorDuplicatePatrick Plitzner

Actions
Related to EDIT - feature request #7507: Advanced "Copy" in bulk editorNewKatja Luther

Actions
Precedes EDIT - feature request #7502: Add further columns to bulk editorClosedPatrick Plitzner

Actions
Precedes EDIT - feature request #7503: Make cells in bulk editor editable NewKatja Luther

Actions
Copied to EDIT - feature request #7518: remaining issues for usage of NatTable for bulkeditorClosedPatrick Plitzner

Actions
Actions #1

Updated by Katja Luther almost 6 years ago

the selection as candidate or target is not possible, the menu items are not enabled.
and the details view does not show up.
the menu items are not correct, for example there are menu items like "change to person" for references

Actions #2

Updated by Andreas Müller almost 6 years ago

  • Status changed from New to In Progress
  • Target version changed from Unassigned CDM tickets to Release 5.1
Actions #3

Updated by Patrick Plitzner almost 6 years ago

  • Status changed from In Progress to Resolved
  • Assignee changed from Patrick Plitzner to Katja Luther
  • % Done changed from 0 to 50
Actions #4

Updated by Katja Luther almost 6 years ago

  • Status changed from Resolved to Feedback

this works fine now

maybe one minor issue: when set a multi selection as deduplication candidate, the marker is not shown before changing the selection, this is a little bit confusing.

Actions #5

Updated by Katja Luther almost 6 years ago

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

Updated by Patrick Plitzner almost 6 years ago

Katja Luther wrote:

this works fine now

maybe one minor issue: when set a multi selection as deduptlication candidate, the marker is not shown before changing the selection, this is a little bit confusing.

This works for me. I select two elements, right-click "Set as candidate" and the marker appears in the label

Actions #7

Updated by Andreas Müller almost 6 years ago

Can we have left alignment instead of center alignment. Especially for the titleCache columns this is easier to read.

Actions #8

Updated by Andreas Müller almost 6 years ago

  • Related to feature request #6321: Implement parallel loading + progress bar for loading objects into the bulk editor added
Actions #9

Updated by Andreas Müller almost 6 years ago

For focus loss after save see #6321#note-23

Actions #10

Updated by Andreas Müller almost 6 years ago

  • Related to bug #7484: Protected titleCache handling not correct in Reference detail view added
Actions #11

Updated by Andreas Müller almost 6 years ago

Critical: sometimes the table does not show up after opening a bulk editor.

Even after "search" the table does not show up. Only re-sizing helps. Something users may not know.
Reproduction is difficult but for me always worked when opening at least 3 name bulk editors without doing anything else in between.

Also the details view is not emptied when the new bulk editor is opened (see attachment)

Actions #12

Updated by Andreas Müller almost 6 years ago

Can we remove the "Sort by" field (see attachment above)? It think it is not in use in all/most bulk editors.

Instead it would be nice and very helpful to enable sorting by columns in the NAT tables.
And of course it would be nice to allow more columns such as further cache fields,id, uuid, created, updated,... which can also be used for sorting (but this might be a new ticket)

Actions #13

Updated by Andreas Müller almost 6 years ago

Adding a marker does not update the "+" in supplemental data view. Maybe this is not related to NAT.

Actions #14

Updated by Andreas Müller almost 6 years ago

Copying parts of the titleCache within the table is not possible. This is a regression compared to the old table.

Can we make cells "editable" in the sense that it is possible to select (and copy) parts of the text but at same time not make it possible to change data in the field? This is how it was before.

Actions #15

Updated by Andreas Müller almost 6 years ago

Copying always copies the titlecache column value even if the other column has the focus. For now this might be acceptable as we have only one important column but may change once we have multiple columns with information that users may want to copy.
This may become irrelevant once the above issue #7439#note-14 is implemented.

Actions #16

Updated by Andreas Müller almost 6 years ago

If the whole row has the focus we may want to copy all fields in csv or tsv format.

Actions #17

Updated by Andreas Müller almost 6 years ago

"Copy" is not yet translated to German.

Actions #18

Updated by Andreas Müller almost 6 years ago

For multi-select copy is not yet possible but might be a very nice feature (e.g. for copy and paste larger datasets to Excel). Feel free to create a new ticket as this is a new feature.

Actions #19

Updated by Andreas Müller almost 6 years ago

Actions #20

Updated by Patrick Plitzner over 5 years ago

Comment #7439#note-7: Cell text is left aligned
Comment #7439#note-11: I can reproduce this as well when opening the third editor. Question: Do we really want to allow opening the same type of bulk editor multiple times?
Comment #7439#note-12: Enabled sorting (Please open a new ticket specifying which additional columns should be present)
Comment #7439#note-13: Could not reproduce this behavior. Which bulk editor did you use? On which element did you test this?

Actions #21

Updated by Andreas Müller over 5 years ago

Patrick Plitzner wrote:

Comment #7439#note-11: I can reproduce this as well when opening the third editor. Question: Do we really want to allow opening the same type of bulk editor multiple times?

Sure. Searching for "B*" is different from "A*". One may want to have open both for various reasons.

Actions #22

Updated by Andreas Müller over 5 years ago

Also the problem is not necessarily related to opening multiple editors. But this is how you can easily reproduce the behavior.

Actions #23

Updated by Andreas Müller over 5 years ago

Patrick Plitzner wrote:

Comment #7439#note-13: Could not reproduce this behavior. Which bulk editor did you use? On which element did you test this?

No matter which bulk editor. Just take any name, taxon, reference or author (...) without a marker => no + shows up in supplemental data view beside marker.
Add a marker via bulk editor. Still no + shows up (but should).

Actions #24

Updated by Andreas Müller over 5 years ago

Actions #25

Updated by Andreas Müller over 5 years ago

Andreas Müller wrote:

Copying parts of the titleCache within the table is not possible. This is a regression compared to the old table.

Can we make cells "editable" in the sense that it is possible to select (and copy) parts of the text but at same time not make it possible to change data in the field? This is how it was before.

By the way, changing data within the columns may become an option now. E.g. changing the specific epithet of a name might be more comfortable to do within the table then within the details view as the workflow is probably faster if you go through a long list of names.
It was no option as long as only the titleCache was shown. Editing the titleCache should still only be possible if the cache is protected (same as in details view). Should we open a new ticket for editing fields (this is also related to #7502)

Actions #26

Updated by Andreas Müller over 5 years ago

Actions #27

Updated by Andreas Müller over 5 years ago

Andreas Müller wrote:

For multi-select copy is not yet possible but might be a very nice feature (E.g. for copy and paste larger datasets to Excel). Feel free to create a new ticket as this is a new feature.

By the way, it might be nice to have a simple "copy" operation and an advanced "copy ..." which then allow setting further configuration to the copy command, like format (csv, tsv, ...), include header, NULL value handling, etc.
HeidiSQL has a nice configuration dialog:

Feel free to create new ticket.

Actions #28

Updated by Andreas Müller over 5 years ago

Patrick Plitzner wrote:

Comment #7439#note-7: Cell text is left aligned

Great! Looks much better now.

One more possible improvement: after first search, can we adapt column width to the percentage of the longest text that occurs per columns (or some similar algorithm that tries to optimize the column width. Now, we get something like

which is less functional.

Feel free to create new ticket if not trivial. Also dependend on #7502.

Actions #29

Updated by Andreas Müller over 5 years ago

Patrick Plitzner wrote:

Comment #7439#note-12: Enabled sorting (Please open a new ticket specifying which additional columns should be present)

Great! But there is a problem now. If sorting is e.g. DESC on titleCache, reloading (#6321) should take place in the same order.

Actions #30

Updated by Patrick Plitzner over 5 years ago

Comment #7439#note-9: Focus loss is fixed
Comment #7439#note-11: Rendering problem is fixed
Comment #7439#note-12: Removed sort combo
Comment #7439#note-14: Cell text can be selected but not edited
Comment #7439#note-16: new ticket #7507
Comment #7439#note-17: i18n for "copy"
Comment #7439#note-18: new ticket #7507
Comment #7439#note-23: This is fixed but other issue occured (#7504)
Comment #7439#note-25: new ticket #7506

Actions #31

Updated by Patrick Plitzner over 5 years ago

Actions #32

Updated by Patrick Plitzner over 5 years ago

Actions #33

Updated by Patrick Plitzner over 5 years ago

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

Andreas Müller wrote:

Patrick Plitzner wrote:

Comment #7439#note-12: Enabled sorting (Please open a new ticket specifying which additional columns should be present)

Great! But there is a problem now. If sorting is e.g. DESC on titleCache, reloading (#6321) should take place in the same order.

The problem is, that the initial search is sorted in the cdmlib service method. When clicking on the header in the nattable the search is done on the local list of results. If this sort order is changed to DESC and we would do a re-search with also DESC on the cdmlib side we would get a descending order first which would then be sorted again in the local list leading to ASC.
Maybe we can ignore this issue because it will be resolved when we avoid doing the re-search after save.

Actions #34

Updated by Andreas Müller over 5 years ago

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

Updated by Andreas Müller over 5 years ago

Having the markers for deduplication (target, candidate) in the text fields is not very comfortable and scalable. When we have multiple columns and users change the order of the columns the order gets lost. Also editing the columns (#7503) will not be possible anymore.
Is there no other possibility to add markers rows in NAT tables? If not we should create a new column which then handles the deduplication information seperately (preferably with a 3 state icon - none, target, candidate).

Actions #37

Updated by Andreas Müller over 5 years ago

Using the standard right click menu is cumbersome as clicking on a column cell immediately changes the cell into the editing mode with a less comfortable menu:

We had the same discussion when introducing the standard tables for bulk editor. Please refer to that discussion for details. Generally we need to go to editing mode only by double click. And even better we should not show the above menu in editing mode but a more suitable one.

This is release critical.

Actions #38

Updated by Andreas Müller over 5 years ago

Changing the text in a cell for now should not be possible until #7503 is implemented. If NAT tables don't offer such a state we could probably catch the change events and handle them such that the text does not change.

Actions #39

Updated by Andreas Müller over 5 years ago

Andreas Müller wrote:

Having the markers for deduplication (target, candidate) in the text fields is not very comfortable and scalable. When we have multiple columns and users change the order of the columns the order gets lost. Also editing the columns (#7503) will not be possible anymore.
Is there no other possibility to add markers rows in NAT tables? If not we should create a new column which then handles the deduplication information seperately (preferably with a 3 state icon - none, target, candidate).

We decided to use row colors instead. Additionally offering tooltips saying that a row is target or candidate for deduplication.

Actions #42

Updated by Andreas Müller over 5 years ago

Andreas Müller wrote:

Using the standard right click menu is cumbersome as clicking on a column cell immediately changes the cell into the editing mode with a less comfortable menu:

We had the same discussion when introducing the standard tables for bulk editor. Please refer to that discussion for details. Generally we need to go to editing mode only by double click. And even better we should not show the above menu in editing mode but a more suitable one.

This is release critical.

Now, the editing mode shows the standard menu but in many cases no menu appears at all when doing some kind of right click on the table. This is very critical. Maybe we better revert everything related to editing mode.

Actions #43

Updated by Andreas Müller over 5 years ago

Now, column resizing is disabled which is better then before. However, we should resize the table to window size at the beginning when opening the bulk editor.

Actions #44

Updated by Katja Luther over 5 years ago

The editing mode of the table caused a widget is disposed exception and no menu was shown.
I reverted these changes and the resizing.

Actions #45

Updated by Katja Luther over 5 years ago

Actions #46

Updated by Katja Luther over 5 years ago

  • Status changed from Feedback to Closed

close this ticket and created a new one with remaining issues (#7518)

Actions #47

Updated by Andreas Müller over 5 years ago

Actions #48

Updated by Andreas Müller over 5 years ago

Actions #49

Updated by Andreas Müller over 5 years ago

  • % Done changed from 50 to 100
Actions

Also available in: Atom PDF