Project

General

Profile

feature request #9287

Show term details in details view of term tree editor

Added by Katja Luther 12 months ago. Updated 4 months ago.

Status:
Closed
Priority:
New
Assignee:
Category:
taxeditor
Target version:
Start date:
11/09/2020
Due date:
% Done:

100%

Severity:
normal

Description

From ticket #8774:

Can we show the details of a term anyway even if it is not editable in the term tree editor. This is because the pure label is often not enough to decide if one took the right term.
If not all information is easily available at least the most important should be shown.

But ofcourse it should be disabled for editing. (but Maybe in future when vocabularies will also use term trees for hierarchy editing might be a requested feature, just in case this is important to know for implementation).


Related issues

Related to Edit - bug #8774: Reimplement term tree editors to implement the general save strategy Closed 12/17/2019
Related to Edit - feature request #9337: Open issues for reimplementation of termtree editors Closed 11/30/2020
Copied to Edit - bug #9688: Make disabled fields copyable Closed 06/30/2021

Associated revisions

Revision ee4b4e44 (diff)
Added by Katja Luther 4 months ago

ref #9287: show disabled term details in term tree

Revision 74ca5142 (diff)
Added by Katja Luther 4 months ago

ref #9287: show disabled term details in term tree

Revision 12addc56 (diff)
Added by Katja Luther 4 months ago

ref #9337, #9287: show details of term and supplemental data of term node

Revision 8ee0429a (diff)
Added by Katja Luther 4 months ago

ref #9287: disable translation elements of term

Revision c9e78864 (diff)
Added by Katja Luther 4 months ago

ref #9287: disable translation elements of term - continue

Revision 3a553a43 (diff)
Added by Katja Luther 4 months ago

ref #9287: fix NPE when multilanguage is disabled

Revision ec4a9eb1 (diff)
Added by Katja Luther 4 months ago

ref #9287: adapt all toggleable text fields to enableCache

Revision 8fbdff7b (diff)
Added by Katja Luther 4 months ago

ref #9688: make disabled fields copyable

History

#1 Updated by Katja Luther 12 months ago

  • Related to bug #8774: Reimplement term tree editors to implement the general save strategy added

#2 Updated by Andreas Müller 11 months ago

  • Target version changed from Unassigned CDM tickets to Release 5.18

As this is related to #8774 I move it to current milestone as we should implement it soon (might also be an interesting feature for the "integrated taxonomy" course.

#3 Updated by Andreas Müller 11 months ago

Maybe we do not need to load the data with the initial DTO but asynchronously when ever it is needed. Just in case loading takes to long for term types with a large number of term nodes.

#4 Updated by Andreas Müller 11 months ago

#5 Updated by Katja Luther 11 months ago

  • Target version changed from Release 5.18 to Release 5.19

#6 Updated by Andreas Müller 9 months ago

  • Target version changed from Release 5.19 to Release 5.21

#7 Updated by Andreas Müller 8 months ago

  • Target version changed from Release 5.21 to Release 5.22

#8 Updated by Andreas Müller 7 months ago

  • Subject changed from Show term details in details view of term editor to Show term details in details view of term tree editor

#9 Updated by Katja Luther 6 months ago

  • Target version changed from Release 5.22 to Release 5.29

#10 Updated by Katja Luther 4 months ago

  • Status changed from New to Resolved
  • Assignee changed from Katja Luther to Andreas Müller
  • Target version changed from Release 5.29 to Release 5.25

this should be fixed. Please review.

#11 Updated by Andreas Müller 4 months ago

  • Tracker changed from bug to feature request
  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther
  • % Done changed from 0 to 50

The buttons for multi-language editing are still enabled. So it is possble to remove a description for a given language or to open the the translation editor.

Note: Als the Open URI button is enabled even if no URI is given. It should be enabled only if a value exists (in general, not only in context of this ticket).

#12 Updated by Katja Luther 4 months ago

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

Andreas Müller wrote:

The buttons for multi-language editing are still enabled. So it is possble to remove a description for a given language or to open the the translation editor.

Note: Als the Open URI button is enabled even if no URI is given. It should be enabled only if a value exists (in general, not only in context of this ticket).

This is fixed now, please review.

The URI issue is fixed as well.

#13 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

Works as expected now and we can close the ticket.

There is only one final issue which is valid for all disabled fields. Is it possible to generally make disabled fields copyable? Currently they only show the text but it is not possble to copy it. At some places this is already implemented. We probably should move this issue to a new ticket.

#14 Updated by Andreas Müller 4 months ago

See also #7309#note-19 which is maybe related to this ticket.

#15 Updated by Katja Luther 4 months ago

  • Assignee changed from Katja Luther to Andreas Müller

Andreas Müller wrote:

Works as expected now and we can close the ticket.

There is only one final issue which is valid for all disabled fields. Is it possible to generally make disabled fields copyable? Currently they only show the text but it is not possble to copy it. At some places this is already implemented. We probably should move this issue to a new ticket.

It is possible and I implemented for toggleable and text fields, but now it is possible to enter something, but the changes disappear immediately.

Please have a look whether this is ok.

#16 Updated by Andreas Müller 4 months ago

  • Assignee changed from Andreas Müller to Katja Luther

It works as expected for the fields in the term tree details view (which are not toggleable).

For toggleable they still set the dirty flag (tested on Person details view with bulk editor). This is unwanted.

I do not yet fully understand why it is not possible to use the same technique as for ordinary text fields in toggleable fields.

#17 Updated by Andreas Müller 4 months ago

  • Copied to bug #9688: Make disabled fields copyable added

#18 Updated by Katja Luther 4 months ago

Andreas Müller wrote:

It works as expected for the fields in the term tree details view (which are not toggleable).

For toggleable they still set the dirty flag (tested on Person details view with bulk editor). This is unwanted.

I do not yet fully understand why it is not possible to use the same technique as for ordinary text fields in toggleable fields.

The same technique is used for text fields and toggleable text fields but maybe I need to have a look on the event handling of the toggleable text field.

#19 Updated by Andreas Müller 4 months ago

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

As there is a followup ticket for the general "copyable" issue we can close this ticket and move discussion on "copyable" to #9688

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)