Page MenuHomePhabricator

Use RemarkupControl in Differential inline comment UI
ClosedPublic

Authored by epriestley on Jan 14 2014, 10:51 PM.
Tags
None
Referenced Files
F14658727: D7969.id18031.diff
Sat, Jan 11, 9:57 PM
F14658696: D7969.id18028.diff
Sat, Jan 11, 9:54 PM
F14655965: D7969.diff
Sat, Jan 11, 7:03 PM
Unknown Object (File)
Mon, Jan 6, 11:38 AM
Unknown Object (File)
Dec 12 2024, 5:20 AM
Unknown Object (File)
Dec 12 2024, 3:38 AM
Unknown Object (File)
Dec 11 2024, 3:10 PM
Unknown Object (File)
Dec 9 2024, 8:28 PM
Subscribers

Details

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.

i will do what I can to make facebook hate this