Project

General

Profile

feature request #8756

Refresh button for structure tree and property tree in Character Editor

Added by Andreas Müller 6 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Priority14
Category:
taxeditor
Target version:
Start date:
12/10/2019
Due date:
% Done:

50%

Severity:
normal
Tags:

Description

copied from #8755, explanations see there


Related issues

Related to Edit - feature request #8755: Show all structure/property trees in CharacterEditor Closed 12/09/2019
Blocked by Edit - bug #8774: Reimplement term tree editors to implement the general save strategy In Progress 12/17/2019

Associated revisions

Revision a0f4664b (diff)
Added by Patrick Plitzner 6 months ago

fix #8756 Add Refresh button for Character Editor

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

fix #8756: add refresh button to structure and property tree viewer

Revision 5ad5e48c (diff)
Added by Katja Luther 4 months ago

fix #8756: code cleaning

History

#1 Updated by Andreas Müller 6 months ago

I assigned this to KL for now. PP if you think it is a no-brainer and you can immediately do it please feel free to take over. It is definitely something we could use immediately in the integrated taxonomy course. But it is not a blocker.

#2 Updated by Patrick Plitzner 6 months ago

  • Status changed from New to In Progress
  • Assignee changed from Katja Luther to Patrick Plitzner
  • Target version changed from Unassigned CDM tickets to Release 5.12

#3 Updated by Patrick Plitzner 6 months ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 50

#4 Updated by Patrick Plitzner 6 months ago

  • Assignee changed from Patrick Plitzner to Andreas Müller

#5 Updated by Andreas Müller 6 months ago

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

It does not work. I changed the order of the properties tree in demo bupleurum 01. After saving an pushing refresh button nothing happened in the Property tree of the Character Editor.

Also: I think we need this button for each tree seperate. If you changed only the structure tree you may not want to reload the property tree also and the other way round.
And also the current position behind the "load character voc" button suggests that it is related to the character vor.
I suggest to put the buttons above the according trees on the upper right corner. E.g. for Structure tree the label "Structure" is on the left corener above the tree and the button should be on the right corner. (more or less followed by "Property" from the property tree)

#6 Updated by Andreas Müller 5 months ago

#7 Updated by Andreas Müller 5 months ago

  • Assignee changed from Patrick Plitzner to Katja Luther

#8 Updated by Katja Luther 4 months ago

  • Target version changed from Release 5.12 to Release 5.13

#9 Updated by Katja Luther 4 months ago

  • Status changed from Feedback to Resolved

#10 Updated by Katja Luther 4 months ago

  • Assignee changed from Katja Luther to Andreas Müller

please review

#11 Updated by Andreas Müller 3 months ago

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

It still does not work for me. In rem_conf_am I added a 3. term to Structure tree "Test" via Structre Tree Editor but pressing the refresh button did not show this term. Also moving the order of a tree did not show any result.

Please decide if this can be fixed easily now or should better be fixed after refactoring the term (tree) editor - but I guess it is not related.

#12 Updated by Katja Luther 3 months ago

  • Assignee changed from Katja Luther to Andreas Müller

Andreas Müller wrote:

It still does not work for me. In rem_conf_am I added a 3. term to Structure tree "Test" via Structre Tree Editor but pressing the refresh button did not show this term. Also moving the order of a tree did not show any result.

MMh, I tested with actual nightly + rem_conf_am and added untitled to structure tree "Test" and after pressing refresh the newly added term was shown. Also removing a term from a tree works.

#13 Updated by Andreas Müller 3 months ago

  • Priority changed from New to Highest

#14 Updated by Andreas Müller 3 months ago

  • Priority changed from Highest to Priority14

#15 Updated by Andreas Müller 3 months ago

  • Status changed from Feedback to Resolved
  • Target version changed from Release 5.13 to Release 5.14

Now I tested on another machine and it first worked. But then I tested to change the tree on 1 machine and then refresh on the other and again it did not work.

Maybe it is best to wait for implementation of #8774 and then to review again

#16 Updated by Andreas Müller 3 months ago

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

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)