Wrap single lines of code; horizontally scroll multi-line code blocks

What does this MR do?

Original problem: code blocks were wrapping in issue body

• Multi-line code blocks (```) should scroll horizontally
• Single lines of code (`) should wrap

Why was this MR needed?

Inconsistent scrolling vs wrapping behavior

What are the relevant issue numbers?

Closes #19619 (closed)

Screenshots (if relevant)

Added 1 commit:

• 1a50a822 - Wrap single lines of code; horizontally scroll multi-line code blocks

Marked this merge request as a Work In Progress

Unmarked this merge request as a Work In Progress

Added 1 commit:

• c4d8c6eb - Remove unneeded code

Assignee removed

Reassigned to @alfredo1

pre Sequences of whitespace are preserved, lines are only broken at newline characters in the source and at
elements.

---

pre-wrap Sequences of whitespace are preserved. Lines are broken at newline characters, at
, and as necessary to fill line boxes.

Why do we want pre-wrap instead of pre?

If you change that to pre, the single lines of code scroll instead of wrapping.

Looks good.

@annabeldunstone @jschatz1 is this correct?

@alfredo1 if it's a code block, then yep. Should scroll. Only single-tick lines of code should wrap. If you put a line break in that though, it should preserve that line break

But the form should not be that wide IMO or is this intended?

@alfredo1 I see what you mean. It looks like that's how it currently functions in production as well, but we should address that in https://gitlab.com/gitlab-org/gitlab-ce/issues/20640. The goal will be to have the comment box scrolling independently of the diff table

@annabeldunstone alright so this LGTM then!

Reassigned to @iamphill

Reassigned to @jschatz1

Added 318 commits:

• c4d8c6eb...11eefba8 - 316 commits from branch master
• 3eba2e4f - Wrap single lines of code; horizontally scroll multi-line code blocks
• 4d757ea5 - Remove unneeded code

Added ~149423 label

Status changed to merged

mentioned in commit 39c44977

Picked into 8-11-stable, will go into 8.11.0-rc6.

Removed ~149423 label

mentioned in commit 24477045

@annabeldunstone I don't know if this is related, but adding a code block with a really long line in a comment makes all of the comments (and the code) scroll. I'd expect code blocks to scroll separately from their containers.

See https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5968#note_14451132 for an example.

@smcgivern Yeah we're aware of that issue. That's going to be the case until we can pull the comment blocks completely out of the tables (there's an issue for that somewhere)

OK, thanks! I'm guessing this is the issue: https://gitlab.com/gitlab-org/gitlab-ce/issues/20640