Remove default margin on selects
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Dec 3, 7:56 AM

	Unknown Object (File)
	Mon, Dec 2, 6:51 AM

	Unknown Object (File)
	Fri, Nov 29, 3:08 PM

	Unknown Object (File)
	Fri, Nov 29, 3:08 PM

	Unknown Object (File)
	Fri, Nov 29, 2:51 PM

	Unknown Object (File)
	Oct 24 2024, 7:18 AM

	Unknown Object (File)
	Oct 23 2024, 1:17 AM

	Unknown Object (File)
	Oct 19 2024, 11:11 PM

View All Files

Subscribers

Korvin

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 40837

View Options

resources/celerity/map.php