Page MenuHomePhabricator
Differential D12961

[Redesign] PHUIHeaderView goes to table layout
ClosedPublic
Actions

Authored by chad on May 21 2015, 4:06 AM.
Tags
None
Referenced Files
F15284560: D12961.id31222.diff
Tue, Mar 4, 10:01 AM
Unknown Object (File)
Mon, Mar 3, 11:42 PM
Unknown Object (File)
Mon, Mar 3, 11:02 AM
Unknown Object (File)
Mon, Feb 17, 7:38 AM
Unknown Object (File)
Fri, Feb 14, 11:43 PM
Unknown Object (File)
Mon, Feb 10, 11:06 PM
Unknown Object (File)
Sun, Feb 9, 3:20 PM
Unknown Object (File)
Sat, Feb 8, 4:47 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T8099: Mid-2015 Phabricator Redesign
Commits
Restricted Diffusion Commit
rP3cd27a0881b1: [Redesign] PHUIHeaderView goes to table layout
Summary

Ref T8099. This changes makes flexible layouts easier, and aligns icons, buttons better vertically.

Test Plan

Test various header locations, Phriction, Conpherence, Dashboards, Workboards etc.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 31218.May 21 2015, 4:06 AM
chad retitled this revision from to [Redesign] PHUIHeaderView goes to table layout.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad added a task: T8099: Mid-2015 Phabricator Redesign.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMay 21 2015, 4:06 AM
chad updated this revision to Diff 31222.May 21 2015, 5:21 AM
  • Clean up some Diffusion layouts
chad added a comment.May 22 2015, 2:31 PM

3ea373bea61d2c176b2dbf060cde4b31.gif (230×500 px, 962 KB)

epriestley accepted this revision.May 22 2015, 2:34 PM
epriestley edited edge metadata.

I am only saddened that this is a fake CSS table rather than a wholesome, red-blooded, hard-working American <table /> tag.

This revision is now accepted and ready to land.May 22 2015, 2:34 PM
Closed by commit rP3cd27a0881b1: [Redesign] PHUIHeaderView goes to table layout (authored by chad, committed by chad). · Explain WhyMay 22 2015, 3:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
36 lines
src/
applications/
diffusion/
controller/
4 lines
9 lines
view/
phui/
76 lines
webroot/
rsrc/
css/
application/
conpherence/
4 lines
diffusion/
7 lines
layout/
10 lines
phui/
2 lines
43 lines
16 lines
4 lines
2 lines

Diff 31278

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionRepositoryController.php

Loading...
View Options

src/view/phui/PHUIHeaderView.php

Loading...
View Options

webroot/rsrc/css/application/conpherence/widget-pane.css

Loading...
View Options

webroot/rsrc/css/application/diffusion/diffusion-icons.css

Loading...
View Options

webroot/rsrc/css/layout/phabricator-hovercard-view.css

Loading...
View Options

webroot/rsrc/css/phui/phui-document.css

Loading...
View Options

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

Loading...
View Options

webroot/rsrc/css/phui/phui-object-box.css

Loading...
View Options

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

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility