Support the fenced code block's syntax-highlighting language part without `lang=` prefix
Categories
(Conduit :: Phabricator, enhancement)
Tracking
(Not tracked)
People
(Reporter: arai, Assigned: arai)
Details
Attachments
(1 file)
Steps to reproduce:
- comment on the phabricator revision with the following
https://searchfox.org/mozilla-central/rev/f818cf7d7a46728a66b4dcfece73dc017da43f98/js/src/builtin/TestingFunctions.cpp#585-589
```cpp
static bool IsLCovEnabled(JSContext* cx, unsigned argc, Value* vp) {
CallArgs args = CallArgsFromVp(argc, vp);
args.rval().setBoolean(coverage::IsLCovEnabled());
return true;
}
```
Actual result:
the cpp
becomes part of the fenced code block
Expected result:
the cpp
part is handled as language for the syntax highlighting, or just ignored
Assignee | ||
Comment 1•3 years ago
|
||
looks like it's not markdown, but it's another syntax called "remarkup"
https://phabricator.services.mozilla.com/book/phabricator/article/remarkup/
Assignee | ||
Comment 2•3 years ago
|
||
and the following works:
https://searchfox.org/mozilla-central/rev/f818cf7d7a46728a66b4dcfece73dc017da43f98/js/src/builtin/TestingFunctions.cpp#585-589
```lang=cpp
static bool IsLCovEnabled(JSContext* cx, unsigned argc, Value* vp) {
CallArgs args = CallArgsFromVp(argc, vp);
args.rval().setBoolean(coverage::IsLCovEnabled());
return true;
}
```
I wonder if we can tweak the syntax to support language without lang=
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 3•3 years ago
|
||
Comment 4•2 years ago
|
||
(Updating the subject to include "syntax-highlighting" because I almost filed a dupe of this and I expect this will now be a very common request as I do constantly find myself using the new searchfox options and then constantly a little sad that phabricator doesn't handle it while github does in its github flavored markdown fenced code block support. I'm very excited to see :arai already has a patch up, though! Woo!)
Comment 5•2 years ago
|
||
Merged
https://github.com/mozilla-conduit/phabricator/commit/341ac91da393ae54c86203f8a6c12f35225a7fae
Description
•