Page MenuHomePhabricator
Differential D7969

Use RemarkupControl in Differential inline comment UI
ClosedPublic
Actions

Authored by epriestley on Jan 14 2014, 10:51 PM.
Tags
None
Referenced Files
F15580275: D7969.diff
Wed, May 7, 4:36 AM
F15570262: D7969.id18028.diff
Sun, May 4, 2:03 PM
F15512164: D7969.diff
Thu, Apr 17, 8:33 AM
F15454091: D7969.id18028.diff
Mar 29 2025, 4:15 PM
F15449390: D7969.id18029.diff
Mar 28 2025, 9:54 AM
F15448626: D7969.id18031.diff
Mar 28 2025, 5:59 AM
F15443494: D7969.id.diff
Mar 27 2025, 4:42 AM
F15440670: D7969.diff
Mar 26 2025, 2:02 PM
View All Files
Subscribers
aran
chad

Details

Reviewers
btrahan
chad
Maniphest Tasks
T4317: "Monospace textareas" isn't effective for inline comments
Commits
Restricted Diffusion Commit
rP42d9fa34e21b: Use RemarkupControl in Differential inline comment UI
Summary

Fixes T4317. Update the "inline comment" control to a RemarkupControl. This could maybe use some padding/spacing/design touches eventually but seems OK for the moment.

Test Plan

{F101825}

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Jan 14 2014, 10:52 PM
  • Also setUser() to actually pick up monospacing.
chad accepted this revision.Jan 14 2014, 11:02 PM
chad added a comment.Jan 14 2014, 11:14 PM

i will do what I can to make facebook hate this

epriestley added a comment.Jan 14 2014, 11:15 PM

yessss

epriestley closed this revision.Jan 14 2014, 11:15 PM

Closed by commit rP42d9fa34e21b (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
view/
11 lines
infrastructure/
diff/
13 lines
webroot/
rsrc/
css/
application/
differential/
5 lines

Diff 18031

View Options

src/applications/differential/view/DifferentialInlineCommentEditView.php

Loading...
View Options

src/infrastructure/diff/PhabricatorInlineCommentController.php

Loading...
View Options

webroot/rsrc/css/application/differential/changeset-view.css

Loading...
Phabricator · Built by Phacility