Page MenuHomePhabricator

Diff screen - Show empty state if there are no changes
Closed, ResolvedPublic

Description

If a user chooses two revisions that have no changes between them, we need to show an empty state.

No changes empty state
Empty state: diffs.png (1×750 px, 99 KB)
https://zpl.io/a8Jx3Yg

Event Timeline

JMinor triaged this task as Medium priority.

Sending to design for a quick check on the drop shadow on dark themes

IMG_731D614D44B6-1.jpeg (2×1 px, 531 KB)

IMG_D06727E3B9BD-1.jpeg (2×2 px, 428 KB)

Good catch @ABorbaWMF
Here's an updated graphic without the glow-y shadow behind it:

A few more small tweaks:

  • the HR between the two sets of revision info is looking thick on dark mode, could we reduce it to 1pt?
  • it looks like the HR below the revision info on dark mode is missing, can we make sure it's there?
  • on iPad can we extend the background color in the revision info section to be full width?