Page MenuHomePhabricator
Differential D17005

Fix some form layout issues in settings
ClosedPublic
Actions

Authored by chad on Dec 7 2016, 5:14 AM.
Tags
None
Referenced Files
F15464865: D17005.id40912.diff
Wed, Apr 2, 12:57 PM
F15444503: D17005.id40919.diff
Thu, Mar 27, 9:50 AM
F15444501: D17005.id.diff
Thu, Mar 27, 9:50 AM
F15444454: D17005.diff
Thu, Mar 27, 9:41 AM
F15440177: D17005.id40912.diff
Wed, Mar 26, 11:19 AM
F15423951: D17005.id40913.diff
Sat, Mar 22, 7:38 PM
F15421485: D17005.id40913.diff
Sat, Mar 22, 12:13 AM
F15421096: D17005.id40919.diff
Fri, Mar 21, 8:59 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP8bbed7974d62: Fix some form layout issues in settings
Summary

Form inputs with just text or certain selects had extra space. This uses better text centering.

Test Plan

Review forms on all Settings pages, Custom Policy, Herald.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 40912.Dec 7 2016, 5:14 AM
chad retitled this revision from to Fix some form layout issues in settings.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptDec 7 2016, 5:14 AM
chad updated this revision to Diff 40913.Dec 7 2016, 5:24 AM
  • minor other spacings
epriestley accepted this revision.Dec 7 2016, 8:53 PM
epriestley edited edge metadata.

By the way, I really like that PHUIInfoView tweak you did a while back. Error messages are far more pleasant now.

This revision is now accepted and ready to land.Dec 7 2016, 8:53 PM
Closed by commit rP8bbed7974d62: Fix some form layout issues in settings (authored by chad, committed by chad). · Explain WhyDec 7 2016, 9:28 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
10 lines
webroot/
rsrc/
css/
phui/
2 lines
10 lines

Diff 40919

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/css/phui/phui-comment-form.css

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility