Use visibility hidden on remarkup checkboxes
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F19478959: D18130.id43627.diff
	Wed, Jan 7, 9:29 PM

	F19097639: D18130.diff
	Dec 4 2025, 2:15 PM

	F18954710: D18130.diff
	Nov 12 2025, 7:58 PM

	F18867316: D18130.id.diff
	Nov 3 2025, 11:07 PM

	F18861134: D18130.diff
	Nov 2 2025, 2:17 PM

	F18842591: D18130.diff
	Oct 28 2025, 4:45 PM

	F18829987: D18130.id.diff
	Oct 25 2025, 2:58 AM

	F18824524: D18130.diff
	Oct 23 2025, 4:05 PM

View All 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