UI: Comments - Inline code - Exceeds the comments container #694

Closed
opened 2026-02-04 21:54:23 +03:00 by OVERLORD · 3 comments
Owner

Originally created by @sai-manoharp on GitHub (May 27, 2018).

For Bug Reports

  • BookStack Version: v0.20-dev
  • PHP Version: 7.0
  • MySQL Version:
Expected Behavior

Comment content should not cross the container

Current Behavior

Inline code block - exceeds the comments container.

Steps to Reproduce

Change the styles to break the words in the code block.

Attached screenshot:
bookstack-1

Originally created by @sai-manoharp on GitHub (May 27, 2018). ### For Bug Reports * BookStack Version: v0.20-dev * PHP Version: 7.0 * MySQL Version: ##### Expected Behavior Comment content should not cross the container ##### Current Behavior Inline code block - exceeds the comments container. ##### Steps to Reproduce Change the styles to break the words in the code block. Attached screenshot: ![bookstack-1](https://user-images.githubusercontent.com/8279205/40589017-4391b3cc-6204-11e8-8260-3a0765d5e326.png)
Author
Owner

@ssddanbrown commented on GitHub (May 28, 2018):

Hi @manohar566,

Thanks for reporting this and offering a pull request.

Note that after a page reload the code block should be parsed by codemirror to be in a scrollable code-block anyway as shown here:

image

I'm hesitant to merge in your changes in #861 due to the fact the changes will mean white-space is not preserved in the code-block which I think it pretty essential for code-based content.

@ssddanbrown commented on GitHub (May 28, 2018): Hi @manohar566, Thanks for reporting this and offering a pull request. Note that after a page reload the code block should be parsed by codemirror to be in a scrollable code-block anyway as shown here: ![image](https://user-images.githubusercontent.com/8343178/40609154-b9bd9014-6265-11e8-8022-b86e15526337.png) I'm hesitant to merge in your changes in #861 due to the fact the changes will mean white-space is not preserved in the code-block which I think it pretty essential for code-based content.
Author
Owner

@sai-manoharp commented on GitHub (May 28, 2018):

Hi @ssddanbrown,
Thanks for pointing that out. I'll make the necessary change and push back soon.

@sai-manoharp commented on GitHub (May 28, 2018): Hi @ssddanbrown, Thanks for pointing that out. I'll make the necessary change and push back soon.
Author
Owner

@ssddanbrown commented on GitHub (Nov 10, 2018):

Since the last comment on this issue is relatively old I'm going to close this. If the issue remains and is something you still require to be fixed please comment and this can be reopened if required.

@ssddanbrown commented on GitHub (Nov 10, 2018): Since the last comment on this issue is relatively old I'm going to close this. If the issue remains and is something you still require to be fixed please comment and this can be reopened if required.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#694