Page MenuHomePhabricator

Add tabs to Diffusion for consistent navigation
ClosedPublic

Authored by chad on Jun 27 2017, 7:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 10:26 PM
Unknown Object (File)
Fri, Nov 22, 10:45 AM
Unknown Object (File)
Sat, Nov 2, 6:25 AM
Unknown Object (File)
Oct 20 2024, 6:17 AM
Unknown Object (File)
Oct 15 2024, 12:06 PM
Unknown Object (File)
Oct 14 2024, 7:24 PM
Unknown Object (File)
Oct 9 2024, 8:07 AM
Unknown Object (File)
Sep 12 2024, 8:49 AM
Subscribers

Details

Summary

Adds a responsive tab bar navigation to Diffusion. Working through the new design here in pieces, so keep in mind M1477 is the target. Notably:

  • Removes "branches" and "tags" from RevisionView, now on tabs
  • Keeps "browse", "history", "readme" on RevisionView
  • Adds tabs for all main views, including Graph... unless how that feels, so let me know.
Test Plan

Browse all pages, desktop and mobile. Test hg, svn, git repositories.

Diff Detail

Repository
rP Phabricator
Branch
diffusion-tabs (branched from master)
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/aphront/configuration/AphrontApplicationConfiguration.php:274PHL1Unknown Symbol
Unit
Tests Passed
Build Status
Buildable 17581
Build 23597: Run Core Tests
Build 23596: arc lint + arc unit

Event Timeline

chad retitled this revision from [WIP] Add tabs to Diffusion for consistent navigation to Add tabs to Diffusion for consistent navigation.Jul 1 2017, 10:03 AM
chad edited the summary of this revision. (Show Details)
chad edited the test plan for this revision. (Show Details)
chad edited the summary of this revision. (Show Details)
chad planned changes to this revision.Jul 1 2017, 10:09 AM

derp, meant to test SVN

  • test more repository types
epriestley added inline comments.
src/applications/diffusion/controller/DiffusionController.php
430

$drequest->supportsBranches() is possibly a better test, but there's no equivalent for tags and this is flimsy anwyay.

This revision is now accepted and ready to land.Jul 5 2017, 7:59 PM
This revision was automatically updated to reflect the committed changes.