Differential D16968

Remove default margin on selects
ClosedPublic
Actions

Authored by chad on Dec 1 2016, 5:05 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T11935: Misaligned UI elements in Differential UI

Commits

rP98057091f67c: Remove default margin on selects

Summary

Fixes T11935. Safari even with appearance: none still assigns margin to selects. Reset margin to 0.

Test Plan

Review selects on Safari, Chrome.

Diff Detail

Repository

rP Phabricator

Branch

safari-margin (branched from master)

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 14722
Build 19236: Run Core Tests
Build 19235: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 40836.Dec 1 2016, 5:05 PM

chad retitled this revision from to Remove default margin on selects.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

chad added a task: T11935: Misaligned UI elements in Differential UI.

Herald added a subscriber: Korvin. · View Herald TranscriptDec 1 2016, 5:05 PM

epriestley accepted this revision.Dec 1 2016, 5:07 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 1 2016, 5:07 PM

Closed by commit rP98057091f67c: Remove default margin on selects (authored by chad, committed by chad). · Explain WhyDec 1 2016, 5:08 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

6 lines

webroot/

rsrc/

css/

phui/

phui-form.css

1 line

Diff 40836

View Options

resources/celerity/map.php