Use visibility hidden on remarkup checkboxes
ClosedPublic
Actions

Authored by chad on Jun 18 2017, 6:52 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 18, 6:43 AM

	Unknown Object (File)
	Wed, Apr 17, 5:57 AM

	Unknown Object (File)
	Wed, Apr 17, 3:32 AM

	Unknown Object (File)
	Sat, Apr 13, 5:30 PM

	Unknown Object (File)
	Sat, Apr 13, 5:30 PM

	Unknown Object (File)
	Sat, Apr 13, 9:15 AM

	Unknown Object (File)
	Thu, Apr 11, 8:32 AM

	Unknown Object (File)
	Thu, Apr 4, 1:43 PM

View All 87 Files

Subscribers

jboning

Korvin

Details

Reviewers

epriestley
amckinley
avivey

Maniphest Tasks

T12850: Checklists with dangling checkboxes overflow onto subsequent text

Commits

rP9540ed7ec2ee: Use visibility hidden on remarkup checkboxes

Summary

Fixes T12850, marks it hidden with zero width so the element still exists in the DOM

Test Plan

T12850 code, locally

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Jun 18 2017, 6:52 AM

Herald added a subscriber: Korvin. · View Herald TranscriptJun 18 2017, 6:52 AM

Harbormaster completed remote builds in B17517: Diff 43625.Jun 18 2017, 6:53 AM

avivey accepted this revision.Jun 18 2017, 7:53 AM

This revision is now accepted and ready to land.Jun 18 2017, 7:53 AM

Closed by commit rP9540ed7ec2ee: Use visibility hidden on remarkup checkboxes (authored by chad, committed by chad). · Explain WhyJun 18 2017, 8:00 AM

This revision was automatically updated to reflect the committed changes.

Much nicer! Thank you!

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

6 lines

webroot/

rsrc/

css/

core/

remarkup.css

3 lines

Diff 43627

View Options

resources/celerity/map.php