Use visibility hidden on remarkup checkboxes
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15507796: D18130.id43627.diff
	Tue, Apr 15, 11:00 PM

	F15486372: D18130.id43625.diff
	Thu, Apr 10, 6:42 AM

	F15486297: D18130.id43627.diff
	Thu, Apr 10, 6:24 AM

	F15486286: D18130.id43625.diff
	Thu, Apr 10, 6:21 AM

	F15475729: D18130.id.diff
	Sun, Apr 6, 10:50 PM

	F15472451: D18130.diff
	Sat, Apr 5, 1:47 PM

	F15409913: D18130.id43625.diff
	Mar 19 2025, 5:55 AM

	F15380430: D18130.diff
	Mar 14 2025, 3:01 AM

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

Branch

checkboxes (branched from master)

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 17517
Build 23495: Run Core Tests
Build 23494: arc lint + arc unit

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 43625

View Options

resources/celerity/map.php