Page MenuHomePhabricator
Differential D12001

Remove AphrontPanelView from Diffusion
ClosedPublic
Actions

Authored by chad on Mar 6 2015, 4:45 PM.
Tags
None
Referenced Files
F14103268: D12001.diff
Tue, Nov 26, 9:37 PM
Unknown Object (File)
Mon, Nov 18, 8:35 PM
Unknown Object (File)
Oct 27 2024, 6:07 PM
Unknown Object (File)
Oct 20 2024, 9:10 PM
Unknown Object (File)
Oct 14 2024, 4:02 AM
Unknown Object (File)
Oct 5 2024, 7:37 AM
Unknown Object (File)
Sep 15 2024, 2:54 PM
Unknown Object (File)
Sep 10 2024, 2:53 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T7427: Remove remaining AphrontPanelViews
Commits
Restricted Diffusion Commit
rP6909e6206e44: Remove AphrontPanelView from Diffusion
Summary

Removes remaining AphrontPanelView calls in Diffusion for UI Consistency.

Test Plan

Tested each page except lint details, which I couldn't quite find a path to. Everything looks right.

Diff Detail

Repository
rP Phabricator
Branch
aphront-panel-diffusion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4788
Build 4804: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 28889.Mar 6 2015, 4:45 PM
chad retitled this revision from to Remove AphrontPanelView from Diffusion.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMar 6 2015, 4:45 PM
chad added a task: T7427: Remove remaining AphrontPanelViews.Mar 6 2015, 4:46 PM
chad updated this revision to Diff 28890.Mar 6 2015, 4:48 PM
  • move pager outside of ObjectBox
btrahan accepted this revision.Mar 6 2015, 11:01 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 6 2015, 11:01 PM
Closed by commit rP6909e6206e44: Remove AphrontPanelView from Diffusion (authored by chad, committed by chad). · Explain WhyMar 6 2015, 11:32 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 28889

View Options

src/applications/diffusion/controller/DiffusionBranchTableController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionBrowseDirectoryController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionExternalController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionHistoryController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionLintController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionLintDetailsController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionSymbolController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionTagListController.php

Loading...
Phabricator · Built by Phacility