Annozilla 'to do' list

Preferences

Preference wizard for easy first-time setup. (Now pretty much working I think, except, if Annozilla sidebar is open, the server settings don't seem to be used until after a restart - seems a bit of an edge case).
Remove server as default post server if that server is deleted from the list
"Basic" server prefs - is there any way we can set the data up only on "OK"?
Remove username/password handling from Annozilla
In the Server Options panel, display "Local Server" not "urn:localserver"
Move multiple server support into an 'Advanced' panel. Underway. (if possible)

Editor window

Annotation passed to postAnnotationOnSuccess from remote server doesn't have proper annotation type; need to reload to get it and sometimes author is something like "rdf#mjAmH". (We send RDF to the server and get back an annotation to indicate success; however the annotation data is incomplete and so we try to supplement it with the data we originally passed. The test on 'creator' goes wrong, because we pass it as a literal and get it back as a resource. We fail to overwrite the more "specific" type of "Comment" because the "Annotation" type is already present.) Mostly fixed, but can still be a problem with loading a newly-created annotation in a pop-up window.
After creating a reply - or presumably after all other post/updates - all open windows get their status updated to say "SUCCESS". Partial fix.
Don't enable Save until "type" is populated

Structural

Convert "annotationLookups" into some kind of RDF thing which users could install more types to. Maybe the icon lookup too? Update: started work, adding annotationTypes.rdf and hooking this up to editor.xul.
IconManager trouble getting body from server? due to asynchronous body load: see loadBodyOnSuccess
Don't want to load annotation bodies during annotation loading (in general case - OK for local server)
Rework classes using model/view/controller terminology. (Underway as part of the tabbed browsing support.)

UI

Don't write to window.status?
Make all sidebar options available via menu (not just context menu)
~~Use XBL language binding everywhere~~ Now expecting to remove the language options from the UI

Errors

Annozilla fails with "error loading annotations" during loading of annotations if there is an annotation that has no context property. (This property is not a requirement for annotations, I think.)
Also, it would be better if annozilla tried to continue the loading of annotations if one annotation is "malformed".
iconClick->openEditorWindow->doesn't get fragment to add temporary icon Error: [Exception... "'openEditorWindow requires either a fragment or an existing annotation' when calling method: [nsIDOMEventListener::handleEvent]" nsresult: "0x8057001e (NS_ERROR_XPC_JS_THREW_STRING)" location: "<unknown> data: no]. (Does it need it? The condition in the code doesn't seem to match the error message.) Updated code, some errors now reported to console.
When creating an annotation, an icon is supposed to be inserted into the document, but this fails.
Invoke "load annotation" twice - get annotations appearing twice in sidebar

Others

Implement Reply/Check for Replies for LocalServer.
No reason for there to be (internally) two buttons for Save on the editor window.
Test cases
Unit tests

Fixed issues

~~"Check for replies" as available as a context menu option, even on a reply~~

~~"test " turned into "testĀ " when saving local annotation.~~ Update: seems to have started working in beta 3?

~~Show annotation title (not text of annotated region) in sidebar~~

~~User-editable annotation titles~~

~~In beta 3, editor shows XML error, because of changes to textcontext.dtd.~~ Resulting changes make the code incompatible with beta 1/2 and earlier.

~~In beta 3, the "Annozilla" text disappears from the Sidebar menu (although the menu option is present and works). The "Bookmarks" text also vanishes! (Don't know why.)~~

Improved behavior for tabbed browsing (autoload for multiple tabs at once etc.) This works well now. The sidebar throbber doesn't always get updated correctly though, and some methods don't care about the "target" parameter when they probably should.

~~After opening an annotation in a window, "delete" works but doesn't close the window when finished. Update: removing this functionality (auto-close of window) anyway.~~

~~Replies don't properly consider "threadRoot".~~

~~Load a page in a new tab and create annotation: get ERROR currentAnnotations is null. Update: set model.js to auto-create an empty object as required. Fixed now I think.~~

~~"Delete" from an open editor window confuses things a bit: no status indicator in open editor window, get error "undefined" when the deletion completes, sidebar starts saying "Waiting for -1 servers"~~

~~Can't see replies to replies (created in Amaya)~~

~~Format dates sensibly~~ Fixed (in sidebar)

~~It shouldn't be possible to change server when replying.~~ Fixed.

~~"Check for Replies" only seems to add one item to the sidebar~~

When opening an annotation window, makeUpdate() gets called to create a function, but it looks like it gets called at startup as well as after the success or failure of an operation. Is it supposed to? Update: YES.

~~Change from Local to Remote server and load annotations: old annotation icons and highlighting are not removed. (They are removed from the sidebar list correctly.)~~ Fixed in CVS.

"Basic" server prefs - got the following setting up a remote server:
Error: uncaught exception: [Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsIRDFService.GetResource]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)" location: "JS frame :: chrome://annozilla/content/prefs/pref-annozilla-servers.js :: setBasicServer :: line 429" data: no]
Fixed in CVS.

After changing server types and trying to select an annotation icon, get the error "Unable to delete the selected annotation due to an internal error. Try reloading the annotations for the page." - but I was only attempting to open the annotation, not delete it. (These errors are not properly localised anyway.) Fixed in CVS.

~~Move 'Server prefs' and 'Edit Site List' back into the main panel on Firefox~~ (fixed in CVS)

~~Rename "Post"/"Update" to "Save"~~ (fixed in CVS)

~~Fix memory leaks (3 objects leaked per window)~~ fixed in CVS

~~Get "Stylesheet annotations.css not found: using chrome://annozilla/content/annotations.css" in error console.~~ Update: this is intended behaviour, however I have removed the logging to the console.

~~Autoloading for tabbed browsing?~~ Implemented by CGH.

From an empty sidebar (showing "No annotations loaded"), create an annotation without first loading the annotations for the page. The new annotation is added for the list but "No annotations loaded" is still present. Fixed, but other scenarios exist where this could be problematic.

~~Remove extra nodes from DOM when a highlight is deleted. (At the moment we just remove the highlighting.)~~

Open annotation window by double-clicking on icon opens window at unreasonably small size. ~~Or sometimes get "Sorry, couldn't download the annotation body."~~ - fixed by allowing multiple annotation windows

~~Integrate Firefox 2 built-in spelling checker.~~ fixed in CVS

~~No animation when deleting annotation. Update: fixed for delete, but the sidebar ends up saying "Annotations loaded OK".~~

When loading up an editor window, the buttons shouldn't be enabled until the document has finished loading. Pretty much works I think. But if you post a new annotation, then "Save" should effectively be an update, not a new Post

~~After saving an annotation, "delete" should be available. (But after "delete", you shouldn't be able to save).~~

~~nsIAnnotationMetadata doesn't deal with multiple values, eg multiple types~~ Fixed for types only.

~~"When listing the replies in the sidebar, the annotated region is determined based on the wrong document (the main browser page instead of the annotation).~~

~~On sidebar load with no servers, make it easier to load the Server options panel~~ There's an "Options" menu item now.

~~XBL for annotation icons and/or highlights~~ Doesn't seem to be possible because we can't bind to arbitrary text nodes