Whitespace in Markdown Descriptions is not honored in code blocks

14 views
Skip to first unread message

Wyatt Chapman

unread,
May 12, 2021, 7:33:48 PM5/12/21
to Review Board Community
Hi!

Long-ish timer user, first time bug-reporter. I'm using 3.0.14 so maybe this is already fixed, in which case you can disregard.

I was writing a description that involved a string that had extra spaces in the middle. I figured reviewboard would eat the extra spaces by default so I decided to enable markdown and wrap it in a `code` block... the spaces got eaten anyway!

Looking at the CSS it looks like the code blocks are supposed to have `white-space: pre`, but it gets overridden by another selector? I'm not a web developer so I don't really know what this means or how to fix it but I hope this helps!

Picture attached to demonstrate the bug and my findings.


rbbug.png

Christian Hammond

unread,
May 22, 2021, 3:32:40 PM5/22/21
to revie...@googlegroups.com
Hi Wyatt. Sorry, this dropped off my radar.

Your assessment is correct. I’ll see what’s causing this, and will try to get fixes into the next 3.0.x and 4.0.x releases.

Christian


--
Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups "Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/reviewboard/4ab7a4d3-64bd-48b8-ac8c-ce121494205en%40googlegroups.com.
--
--
Christian Hammond
President/CEO of Beanbag
Makers of Review Board

Christian Hammond

unread,
May 25, 2021, 5:02:23 PM5/25/21
to revie...@googlegroups.com
We have this fixed in 4.0.1. Planning to backport for 3.0.x (but hoping to get more people onto 4.0 soon, since 3.0.x will be entering maintenance mode).

Christian
Reply all
Reply to author
Forward
0 new messages