Make comment ordering consistent throughout the UI.

Review Request #9233 — Created Sept. 29, 2017 and submitted

Information

Review Board
release-3.0.x
197416e...

Reviewers

Inside reviews, comments are ordered with general comments first, then
screenshot comments (which usually don't exist), then file attachment comments,
and finally diff comments. Unfortunately, the comments listed in the issue
summary table and the review dialog didn't follow the same order. This change
fixes that up, so everything is consistent. As a bonus, when adding new
general comments within the review dialog, we'll now automatically scroll to
the newly-created comment editor.

  • Ran unit tests.
  • Ran js-tests.
  • Verified that comments appeared in the correct order for entries, the issue
    summary table, and the review dialog.
  • Verified that adding new general comments in the review dialog had the new
    comments created in the correct spot, and that the dialog scrolled to show
    the new editor.
Description From Last Updated

Just to help with potential page conflicts, can you prefix the IDs with something like review-dialog-?

chipx86chipx86
chipx86
  1. 
      
  2. reviewboard/static/rb/js/views/reviewDialogView.es6.js (Diff revision 1)
     
     
     
     
     
     
     
    Show all issues

    Just to help with potential page conflicts, can you prefix the IDs with something like review-dialog-?

  3. 
      
david
chipx86
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-3.0.x (c1579bc)