Always show the unified review banner.

Review Request #13152 — Created July 18, 2023 and submitted

Information

Review Board
release-6.x

Reviewers

The initial implementation of the unified banner made it so it was only
visible when a review request was open. Unfortunately, this hid a lot of
useful actions and information as soon as the review request was closed.
There's no reason to prevent people from publishing reviews on closed
review requests, and in fact, that can be a really useful thing when
someone sees something late.

This change makes it so the unified banner is always shown, even when
the "this change has been marked as completed" banner is also shown.

Looked at a closed review request and saw the banner.

Verified that everything looked nice:

  • The banners stack nicely when the page is scrolled to the top.
  • As the page is scrolled, the unified banner docks to the top of the
    window and the closed banner scrolls out of view.
  • As the file list on the diff page scrolls past, it docks into the
    unified banner as before.
Summary ID
Always show the unified review banner.
The initial implementation of the unified banner made it so it was only visible when a review request was open. Unfortunately, this hid a lot of useful actions and information as soon as the review request was closed. There's no reason to prevent people from publishing reviews on closed review requests, and in fact, that can be a really useful thing when someone sees something late. This change makes it so the unified banner is always shown, even when the "this change has been marked as completed" banner is also shown. Testing Done: Looked at a closed review request and saw the banner. Verified that everything looked nice: - The banners stack nicely when the page is scrolled to the top. - As the page is scrolled, the unified banner docks to the top of the window and the closed banner scrolls out of view. - As the file list on the diff page scrolls past, it docks into the unified banner as before.
962fefe81add6a89e784ac88c868db219231a24f
maubin
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-6.x (bb802d4)
Loading...