Add image resolution menu to newly-added images in the diff viewer.

Review Request #13628 — Created March 11, 2024 and submitted

Information

Review Board
release-7.x

Reviewers

The image review UI has something of a UI quirk about where we stick the
resolution drop-down menu. For attachments, we either stick it in the
diff mode bar (when viewing a diff), or the caption (when viewing a
single revision). This didn't work when the review UI was embedded
inline in the diff viewer. Actual diffs for images worked fine, because
we had the diff mode bar, but newly-added image files didn't (since
there's no mode bar and no separate caption bar). This change adds an
empty mode bar to use when we're in that situation.

Viewed a diff that contained a newly-added image file. Saw that I had a
mostly-empty toolbar that allowed me to select the zoom for the image.

Summary ID
Add image resolution menu to newly-added images in the diff viewer.
The image review UI has something of a UI quirk about where we stick the resolution drop-down menu. For attachments, we either stick it in the diff mode bar (when viewing a diff), or the caption (when viewing a single revision). This didn't work when the review UI was embedded inline in the diff viewer. Actual diffs for images worked fine, because we had the diff mode bar, but newly-added image files didn't (since there's no mode bar and no separate caption bar). This change adds an empty mode bar to use when we're in that situation. Testing Done: Viewed a diff that contained a newly-added image file. Saw that I had a mostly-empty toolbar that allowed me to select the zoom for the image.
bcf794f9caa42b039b8f13330a0de93d1f31f9e5
maubin
  1. Ship It!
  2. 
      
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-7.x (28b813a)
Loading...