3478: Allow disabling Markdown through an option/checkbox
- Fixed
- Review Board
| thom.******@gmai***** (Google Code) (Is this you? Claim this profile.) | |
 |
|
| Dec. 16, 2014 | |
| 3492 |
What version are you running? 2.0.x What's the URL of the page this enhancement relates to, if any? Everything that supports markdown; description/testing done, comments, replies, etc. Describe the enhancement and the motivation for it. It would be nice to be able to disable markdown/auto-escape text for fields that support markdown. Some users prefer plain text; it would also be nice to give them a user level option for a default markdown/plaintext. I have several users that prefer using the web UI for submitting their review requests, especially for SVN based diffs. Prior to 2.0, everything was plain text. Now, everything is markdown by default. Many users don't immediately notice this and they enter text as they always had. Supposing someone mentions some code in description or testing done: "int some_function(int argument_)", Those "_" have meaning in markdown and they end up rendering something like: "int somefunction(int argument)" with some italics. Also this is inconsistent with rbt post: if you use rbt post with description or testing done, it will default to being plain text and auto escape unless you specify "--markdown": There's no way to get the same effect when using the web UI. What operating system are you using? What browser? N/A. Please provide any additional information below. I personally don't feel strongly about this; The workaround is to educate and learn markdown, with my example, " `int some_function(int argument_, )` " would render correctly. Some users see it as an additional hurdle to code review to have to learn another markup language just to make their text not get mangled.
Same request here!
We often use ascii arts as ship it message.
This is my beautifull Dolly, and I cannot post her anymore :(
,@;@,
,@;@;@;@;@;@/ )@;@;
,;@;@;@;@;@;@|_@' e\
(|@;@;@\@;@;@;@:@( \
'@;@;@|@;@;@;@;'"__'
'@;@;/@;@;/@;'
) // | || Sheep It!
\ \\ ) \\
'"' '"''
-
+
If that isn't a critical use case I don't know what is.
We will find a way to save your sheep! At the very least, you should be able to prefix each line with 4 spaces, or do something like: ``` <sheep here> ```
Please make a setting to disable markdown. The ``` workaround doesn't work very well as RB then attempts to perform the syntax highlighting on the text within.
I second this. Our company has a big push back on markdown. Please allow disabling it.
2.0.9 is getting a large Markdown overhaul that should fix a number of the problems people have encountered. I would like everyone experiencing a Markdown escaping or formatting problem to show samples of the breakage on https://code.google.com/p/reviewboard/issues/detail?id=3602
-
+ Confirmed
-
- Confirmed + PendingReview -
+ Component-Reviews -
+ chipx86
Maybe a bit late, but weird people like me would like to not see the checkbox for every comment field. I can set my preferences globally once, which is great. But I cannot make the checkbox disappear, at least not in 2.0.18.
We're unlikely to add an option that hides a piece of UI in this way as part of Review Board. It'd end up causing more problems than it solves (wait until someone forgets they set it and wants to go from Markdown -> plain text or vice-versa), and sets a bad precedent for future bits of UI people don't like. Best bet is to use something like Stylish to simply hide the checkbox in your browser.