Page MenuHomePhabricator
Differential D15022

Remove icon nav entirely
ClosedPublic
Actions

Authored by epriestley on Jan 14 2016, 3:18 PM.
Tags
None
Referenced Files
F18645905: D15022.id36276.diff
Fri, Sep 19, 8:44 AM
F18625266: D15022.id.diff
Mon, Sep 15, 10:21 PM
F18616472: D15022.id.diff
Sun, Sep 14, 5:42 PM
F18612647: D15022.id.diff
Sun, Sep 14, 9:35 AM
F18601847: D15022.diff
Sat, Sep 13, 1:36 PM
F18601254: D15022.diff
Sat, Sep 13, 11:56 AM
F18367998: D15022.id36276.diff
Aug 28 2025, 12:22 AM
F18363202: D15022.id36276.diff
Aug 27 2025, 4:03 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10054: Define product direction for Project/Profile UIs
Commits
Restricted Diffusion Commit
rPf4c8cdb39a14: Remove icon nav entirely
Summary

Ref T10054. People, projects, and instances no longer use icon nav, so we can get rid of it in favor of profile menus.

Test Plan

Grepped for everything, looked through workboards/profiles again.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 36276.Jan 14 2016, 3:18 PM
epriestley retitled this revision from to Remove icon nav entirely.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10054: Define product direction for Project/Profile UIs.
Herald edited edge metadata. · View Herald TranscriptJan 14 2016, 3:18 PM
chad accepted this revision.Jan 14 2016, 9:30 PM
chad edited edge metadata.

filepicker-UV0iKO6pRLWHVFPIfqTu_olde_english_pouring.jpg (329×347 px, 43 KB)

This revision is now accepted and ready to land.Jan 14 2016, 9:30 PM
epriestley added a comment.Jan 14 2016, 9:32 PM

hehehe

Closed by commit rPf4c8cdb39a14: Remove icon nav entirely (authored by epriestley, committed by epriestley). · Explain WhyJan 15 2016, 5:13 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
18 lines
src/
view/
layout/
29 lines
phui/
1 line
webroot/
rsrc/
css/
application/
base/
4 lines
layout/
62 lines
phui/
1 line
12 lines

Diff 36305

View Options

resources/celerity/map.php

Loading...
View Options

src/view/layout/AphrontSideNavFilterView.php

Loading...
View Options

src/view/phui/PHUIListItemView.php

Loading...
View Options

webroot/rsrc/css/application/base/standard-page-view.css

Loading...
View Options

webroot/rsrc/css/layout/phabricator-side-menu-view.css

Loading...
View Options

webroot/rsrc/css/phui/phui-profile-menu.css

Loading...
View Options

webroot/rsrc/css/phui/phui-workboard-view.css

Loading...
Phabricator · Built by Phacility