fire 'importannotations' on comments import and add listener in ViewLayoutWriter

ViewLayoutWriter listens for zoom, resize and annotation events and
based on that centers the document if there are any comments and
a possibility of bringing them into the view by adjusting margins.

but if the window is just wide enough to show the comments and the
document, and we reload the document, the resize event doesn't find
any comments (as they haven't been imported by then). thus the comments
are half visible/ cut-off.

this isn't noticable when the sidebar is not collapsed as the sidebar
appears after the comments have been imported and it invalidates the
view which fires 'resize' which ViewLayoutWriter listens for.

here we fire an event 'importannotations' after importing the comments
in writer and then the document adjusts it's horizontal position to
bring comments into the view.

Signed-off-by: Sahil Gautam <[email protected]>
Change-Id: Ice393ec507de6a1b79fc7cd4aa0bb21abf447ee2

Author: printfdebugging

browser/src/app/ViewLayoutWriter.ts

@@ -24,6 +24,7 @@ class ViewLayoutWriter extends ViewLayoutBase {
 		app.map.on('resize', this.documentZoomOrResizeCallback, this);
 		app.map.on('deleteannotation', this.annotationOperationsCallback, this);
 		app.map.on('insertannotation', this.annotationOperationsCallback, this);
+		app.map.on('importannotations', this.annotationOperationsCallback, this);
 		app.map.on(
 			'showannotationschanged',
 			this.annotationOperationsCallback,

browser/src/layer/tile/WriterTileLayer.js

@@ -51,6 +51,7 @@ window.L.WriterTileLayer = window.L.CanvasTileLayer.extend({
 				comment.parent = comment.parentId.toString();
 			});
 			app.sectionContainer.getSectionWithName(app.CSections.CommentList.name).importComments(values.comments);
+			app.map.fire('importannotations');
 		}
 		else if (values.redlines && values.redlines.length > 0) {
 			app.sectionContainer.getSectionWithName(app.CSections.CommentList.name).importChanges(values.redlines);