Improve the collapsing logic around entries with status updates.

Review Request #9299 — Created Oct. 20, 2017 and submitted — Latest diff uploaded

Information

Review Board
release-3.0.x
0561fd7...

Reviewers

This updates the Initial Status Updates and Change entries to factor in
the timestamp and state of any status updates when determining whether
an entry should be collapsed.

Both types of entries will be uncollapsed if there are any pending
status updates. These updates are triggering periodic checks, and their
entries might update while the user is viewing the page. We don't want
to risk hiding the content by default by collapsing their boxes.

They also both now factor in the user's last visited time. If the user
has visited the page before, and the status update has been updated
since that last visit, the entry won't be collapsed.

The Initial Status Updates entry used to default to always stay expanded
if there weren't any Change entries yet on the page. Now this only
happens if the user has not visited the page before, ensuring that it's
always shown by default for new visitors and for anonymous users, but
that users who have already viewed the page will see it collapsed unless
there's actual new content to show.

Unit tests pass.