Bug #2860

Annotations, save button fails if no annotation bar is open

Added by Serge Heiden about 1 month ago. Updated about 1 month ago.

Status:New Start date:06/05/2020
Priority:Normal Due date:
Assignee:- % Done:

80%

Category:Annotation Spent time: -
Target version:TXM 0.8.1

Description

steps to reproduce :
- make a concordance <text>[] with VOEUX
- annotate the first word with test=shoudsave
- close the annotation bar by clicking on the "-" button
- try saving with the save button
- nothing is saved and no message is shown in console

Solution 0

  • add a "Save Annotations" button in the Corpus main toolbar
  • add a "Save Annotations" entry in the Corpus main menu
  • add a "Save Annotations" entry in the Corpus view contextual menu

only show the button IF there are annotations to save.

The action is: save corpus annotations that need to be saved or do nothing if no corpus annotations need to be saved.

Evolution: add "URS Annotations Save" to that action (combined or choosed in a checklist)

Solution 1

  • when an annotation needs to be saved, the Save button must be active and its action must save the annotations
  • when no annotation needs to be saved, the Save button must be inactive and it doesn't have any action

Solution 2

  • the Save button is always active and its action saves annotations that need to be saved or does nothing if nothing needs to be saved

Solution 3

  • the Save button is activated when any annotation bar is open
  • the Save button is desactivated when the last annotation bar is closed
    • if corpus annotations need to be saved (even done from another concordance) a modal dialog box is opened with :
      • Message "Save corpus annotations:"
      • two buttons
        • "Now"
        • "Later"

Related issues

related to Bug #2842: Annotations, save button fails if no annotation bar is open New 06/05/2020

Associated revisions

Revision 2914
Added by Matthieu Decorde about 1 month ago

fix KR history refs #2819
add SaveAnnotations command refs #2860

History

#1 Updated by Serge Heiden about 1 month ago

  • % Done changed from 80 to 0

#2 Updated by Serge Heiden about 1 month ago

  • Description updated (diff)
  • Target version changed from TXM 0.8.2 to TXM 0.8.1

#3 Updated by Matthieu Decorde about 1 month ago

  • % Done changed from 0 to 80
  • Description updated (diff)

Also available in: Atom PDF