Opened 9 years ago

Closed 3 years ago

#637 closed defect (wontfix)

Annotations may only be opened in one window at once

Reported by: dstillman Owned by: simon
Priority: major Milestone:
Component: annotations Version: 1.5
Keywords: Cc:

Description

Annotations and highlights can be lost if the same snapshot is opened in multiple windows/tabs and changes are made to the first one to be closed.

Change History (8)

comment:1 Changed 9 years ago by simon

Dan, how are you handling issues like this with notes? This probably calls for a similar approach.

comment:2 Changed 9 years ago by dstillman

Well, two things. Notes use a save timer, so changes are automatically saved within 1.5 seconds or so (see #636). And then the note editor registers itself with the Notifier so that it receives item-modify notifications and updates itself automatically.

So, this is blocked by #666 (which depends on #665). Once that's done, I think browser.js could just register a callback with the Notifier when a snapshot is loaded and refresh on an item modification (and clear all annotations on a delete event, I guess). Even if we had more granular notification events, doing it on the page probably makes more sense than registering a callback for each annotation box.

comment:3 Changed 9 years ago by simon

(In [1586]) addresses #637, Annotations can be lost if a snapshot is opened in multiple places. since notifier support will not be ready in time, this patch presents a warning when two copies of the same snapshot are opened, and does not enable annotation on the second.

comment:4 Changed 9 years ago by simon

  • Priority changed from critical to major

Now major, not critical

comment:5 Changed 9 years ago by simon

  • Summary changed from Annotations can be lost if a snapshot is opened in multiple places to Annotations may only be opened in one window at once

comment:6 Changed 9 years ago by simon

  • Milestone changed from 1.0 RC 1 to 1.5 Alpha 1
  • Version changed from 1.0 to 1.5

comment:7 Changed 6 years ago by dstillman

  • Milestone 2.0 Beta 3 deleted

Milestone 2.0 Beta 3 deleted

comment:8 Changed 3 years ago by simon

  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.