Project

General

Profile

Actions

Bug #4836

closed

Column Annotation page overwrites changes made in Full Annotation

Added by ben leinfelder about 14 years ago. Updated about 14 years ago.

Status:
Resolved
Priority:
Normal
Category:
Morpho Plugin
Target version:
Start date:
02/24/2010
Due date:
% Done:

0%

Estimated time:
Bugzilla-Id:
4836

Description

Because of how I've set up the listener on the madlib tab, if you make a change directly to the annotation table, then click on a column, you'll lose the change because the (hidden) madlib page doesn't have that change and it tries to save it's state whenever there's a selection change event (column or tab).

I knew it was less than ideal to trigger the annotation page to save it's state this way, but I'm not sure where exactly it should be.
Perhaps I should be checking what tab is visible when the column selection changes and basing the save on this...


Related issues

Blocked by Semtools - Bug #4819: "Column Annotation" view duplicates mappingsResolvedben leinfelder02/22/2010

Actions
Actions #1

Updated by Matt Jones about 14 years ago

Alternatively, create StateChangeEvents that reflect the change regardless of where it happens, so that updates to the annotation model can be reflected in both the tabular and the madlib views.

Actions #2

Updated by ben leinfelder about 14 years ago

here's my thinking:
-there should be a [rather lightweight] class that holds the tabbed pane
-this AnnotationTabPane keeps track of which tab is currently active
-if the Column Annotation tab is active and a column selection change event
occurs, the current state of the Column Annotation is saved before anything
else happens.
-if the tab is not active, then nothing needs to be saved.
-if the tab was just active (and is being navigated away from) then the
annotation needs to be saved.

*I use "save" in the temporary sense, not in the "to disk" sense.

Actions #3

Updated by ben leinfelder about 14 years ago

this is working now. I've restricted how you can use the AnnotationPage's methods so as to avoid duplicating mappings.
The AnnotationTabPane is now responsible for keeping track of the selected state of the tabs and saving when need be. More importantly it keeps the tabs in synch with the latest state of the annotation as it is modified through various parts of the UI.

Actions #4

Updated by ben leinfelder about 14 years ago

and it's back

Actions #5

Updated by ben leinfelder about 14 years ago

-added logging
-use member variable for annotation page reference (used in event handling methods)
-reset the annotation page values when the appropriate column/tab selection events occur (this was not always happening, hence the duplication bug)

i'm pretty sure this won't be reopened...

Actions #6

Updated by Redmine Admin about 11 years ago

Original Bugzilla ID was 4836

Actions

Also available in: Atom PDF