Review Request #9092 — Created July 20, 2017 and submitted
js/reviewRequestPage/directory was creating to house
the models for backing review request page entries. Each model in here
lived in the
ReviewRequestPageprefixed to the
model name, making the filenames quite long and setting a bad precedent
for future classes. We also had all the view classes living in a
different location using "box" instead of "entry" in the names.
This change begins some cleanup of these directories, helping with
organizing things before some upcoming work on dynamic reloading. The
model filenames are now simpler, chopping off the
prefix, and the models themselves now live in a
The box views have been moved into
renamed to use the
Entryterminology. The base class,
CollapsableBoxView, has been renamed to
EntryView, which will be
more suitable given some of its upcoming responsibilities.
These files have also been moved out of the
and into a new
review-request-pagebundle. This will slim down the
diff viewer and file attachment pages a bit. In time, more files will
move to this bundle.
Unit tests all run.
Tested all the functionality of the various types of entries on the
review request page. This included:
- Collapsing/expanding boxes (all at once and one-at-a-time)
- Loading comments.
- Replying to comments.
- Viewing status updates.
- Viewing and working with content on the change entries.