Provide a model/view separation for comment blocks.

Review Request #3341 — Created Sept. 22, 2012 and submitted

Information

Review Board

Reviewers

Provide a model/view separation for comment blocks.

Comment blocks on a screenshot have been reworked into a model/view
setup, and are now designed to be reused outside of screenshot
commenting. This makes use of Backbone.js to define the model and view.

We now have an AbstractCommentBlock model that keeps track of serialized
comments coming from the page, as well as any draft comment being made,
and the total count of comments.

There's a ScreenshotCommentBlock that knows how to store coordinate
information and create ScreenshotComments.

And finally, a ScreenshotCommentBlockView that knows how to render this
to the screen.

The code is not ideal yet, largely because we're still working with
lots of older code, so we can't use such constructs as Backbone
collections. In time, these models will be cleaned up a bit.
Tested viewing existing comments on a screenshot (including counts and tooltips).

Tested creating an empty comment and saving it.

Tested deleting the only comment in a block. Saw the bubble, and then the tooltip
and box fade out.

Tested deleting a draft comment when there were other comments. Saw the style
revert back to blue and the count and tooltip change.
david
  1. 
      
  2. reviewboard/static/rb/js/screenshots.js (Diff revision 1)
     
     
    What's the xxx for?
    1. A note to myself that the positioning is accessing a private variable. This is replaced in my comment model change.
  3. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Loading...