Page MenuHomePhabricator

Put a <span> around "NOTE:" in Remarkup
ClosedPublic

Authored by epriestley on Mar 23 2014, 12:19 AM.
Tags
None
Referenced Files
F15469526: D8596.id20384.diff
Fri, Apr 4, 2:10 PM
F15464730: D8596.id20384.diff
Wed, Apr 2, 12:38 PM
F15462076: D8596.id20384.diff
Tue, Apr 1, 12:47 PM
F15462075: D8596.id20382.diff
Tue, Apr 1, 12:47 PM
F15462074: D8596.id.diff
Tue, Apr 1, 12:47 PM
F15462073: D8596.diff
Tue, Apr 1, 12:47 PM
F15459654: D8596.diff
Mon, Mar 31, 2:56 PM
F15416483: D8596.id20384.diff
Thu, Mar 20, 11:32 AM
Subscribers

Details

Summary

I suspect these will look better if we bold/grey the "NOTE:" part.

Test Plan

Ran unit tests. Will try styling next.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Put a <span> around "NOTE:" in Remarkup.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
chad edited edge metadata.

Any way to i18n these

This revision is now accepted and ready to land.Mar 23 2014, 12:27 AM

This gets us closer, but we probably need some kind of "this is a translatable list of words, which can have an arbitrary number of items in it, and each language may have a different number of items" thing (which we don't currently have). We need this in a couple of other places, too (one is the Fixes/Closes/Etc T123 type text).

epriestley updated this revision to Diff 20384.

Closed by commit rPHUa25afc6c64b0 (authored by @epriestley).