Page MenuHomePhabricator

(Redesign) Clean up older "Tile" code
ClosedPublic

Authored by epriestley on Jun 3 2014, 3:11 PM.
Tags
None
Referenced Files
F14018037: D9358.id22314.diff
Tue, Nov 5, 3:36 AM
F14009671: D9358.id22314.diff
Wed, Oct 30, 10:43 PM
F14008935: D9358.id22314.diff
Wed, Oct 30, 8:05 AM
F14007729: D9358.id22283.diff
Tue, Oct 29, 10:41 AM
F13993104: D9358.diff
Tue, Oct 22, 8:41 PM
F13970731: D9358.id22284.diff
Thu, Oct 17, 9:35 AM
Unknown Object (File)
Oct 1 2024, 12:49 AM
Unknown Object (File)
Sep 21 2024, 12:01 AM
Subscribers

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rP6df1a024137c: (Redesign) Clean up older "Tile" code
Summary

This does some backend cleanup of the tile stuff, and some general cleanup of other application things:

  • Users who haven't customized preferences get a small, specific set of pinned applications: Differential, Maniphest, Diffusion, Audit, Phriction, Projects (and, for administrators, Auth, Config and People).
  • Old tile size methods are replaced with isPinnnedByDefault().
  • Shortened some short descriptions.
  • shouldAppearInLaunchView() replaced by less ambiguous isLaunchable().
  • Added a marker for third-party / extension applications.
Test Plan

Faked away my preferences and viewed the home page, saw a smaller set of default pins.

Diff Detail

Repository
rP Phabricator
Branch
hometiles
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 821
Build 821: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to (Redesign) Clean up older "Tile" code.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
  • Pin Phriction and Projects by default.
chad edited edge metadata.
chad added inline comments.
src/applications/meta/query/PhabricatorAppSearchEngine.php
229

fa-times ?

This revision is now accepted and ready to land.Jun 3 2014, 8:07 PM
src/applications/meta/query/PhabricatorAppSearchEngine.php
229

Oh, good catch.

epriestley updated this revision to Diff 22314.

Closed by commit rP6df1a024137c (authored by @epriestley).