Page MenuHomePhabricator

In Differential: when the file tree is enabled, default to the "History" tab instead of "Files"
ClosedPublic

Authored by epriestley on Aug 24 2018, 5:27 PM.
Tags
None
Referenced Files
F18345596: D19601.diff
Tue, Aug 26, 11:51 AM
F18233206: D19601.id46862.diff
Wed, Aug 20, 4:26 PM
F18233103: D19601.id46863.diff
Wed, Aug 20, 4:19 PM
F18112964: D19601.id.diff
Tue, Aug 12, 10:26 PM
F18103381: D19601.diff
Sun, Aug 10, 4:51 AM
F17985340: D19601.id.diff
Fri, Aug 1, 8:44 PM
F17840325: D19601.id46862.diff
Jul 26 2025, 9:02 PM
F17828389: D19601.id46863.diff
Jul 26 2025, 9:09 AM
Subscribers
None

Details

Summary

Ref T13187. See PHI811. If the file tree is enabled and visible, set the default tab to "History".

  • This is a bit magic.
  • It won't work entirely on mobile (we can't tell that you're on mobile on the server, so we'll pick the "History" tab even though the file tree isn't actually visible on your device).
  • There's no corresponding logic in Diffusion. Diffusion doesn't have the same tab layout, but this makes things somewhat inconsistent.

So I don't love this, but we can try it and see if it's confusing or helpful on the balance.

Test Plan

With filetree on and off, reloaded revisions. Saw appropriate tab selected by default.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Aug 24 2018, 5:29 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.