Add the new unified banner.

Review Request #12734 — Created Nov. 22, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-6.x

Reviewers

This change adds the unified banner, a major new UI feature in Review
Board 6.

This new banner replaces several existing banners:

  • The review request draft banner
  • The pending review banner
  • Review reply banners (at least the ones that float)

The goal here is twofold. First, we want to make it very clear whenever
there's any unpublished work (often review reply drafts can get lost in
long pages). Second, we want to enable people to publish everything all
in one go (the typical use case being publishing a bunch of review
replies and a review request update all together).

This also gives us room for future work:

  • We have space at the right hand side of the banner for a future
    block showing info about the current review, popping up into a
    redesigned review dialog.
  • We have a "dock" area (currently empty, but present in the DOM and
    properly styled), which we intend to use for a quick file switcher in
    the diffviewer, and possibly other things.

This is mostly complete. The current things left on my to-do list are:

  • Further testing of the banner in mobile mode. Right now interaction
    with the menus has some problems but it's possible that's just
    Firefox's devtools.
  • Possibly moving the "Add General Comment" action into the review menu.
  • Add more js-tests.
  • Ran js-tests.
  • Ran python unit tests.
  • Tested all different combinations of review requset changes, reviews,
    and review replies. Saw that the draft mode selector showed the
    correct options.
  • Tested publishing with individual items and as a batch.
  • Tested discard with various individual items.
  • Tested the change description field.
  • Tested the new actions (create review, edit review, ship it) that live
    in the "Review" menu.

Commits

Files

    Loading...