Syntax highlighting no longer exists #343

Closed
opened 2026-02-04 18:46:56 +03:00 by OVERLORD · 4 comments
Owner

Originally created by @domainzero on GitHub (May 31, 2017).

  • BookStack Version: BookStack v0.16.2 (MASTER)
  • PHP Version: PHP 5.6.30 (cli)
  • MySQL Version: mysql Ver 15.1 Distrib 10.1.21-MariaDB
Expected Behavior

Syntax highlighting should be working.
I pulled from master because I was on auto-pilot, wondering if it broke there.

Actual Behavior

No syntax highlighting is present. Code blocks are displayed correctly but I'm not sure if hljs is actually loading. I have an hljs stylesheet loaded into the head of the page and that isn't changing any of the code blocks either.

Originally created by @domainzero on GitHub (May 31, 2017). * BookStack Version: BookStack v0.16.2 (MASTER) * PHP Version: PHP 5.6.30 (cli) * MySQL Version: mysql Ver 15.1 Distrib 10.1.21-MariaDB ##### Expected Behavior Syntax highlighting should be working. I pulled from master because I was on auto-pilot, wondering if it broke there. ##### Actual Behavior No syntax highlighting is present. Code blocks are displayed correctly but I'm not sure if hljs is actually loading. I have an hljs stylesheet loaded into the head of the page and that isn't changing any of the code blocks either.
Author
Owner

@domainzero commented on GitHub (May 31, 2017):

I'm a doofus and should have read the commits.
e5fc6bf5fa - Moved from highlight.js to codemirror

I'll see myself out.

@domainzero commented on GitHub (May 31, 2017): I'm a doofus and should have read the commits. e5fc6bf5faffaa5502155345436469d6c2050510 - Moved from highlight.js to codemirror I'll see myself out.
Author
Owner

@domainzero commented on GitHub (May 31, 2017):

Ehhh, I guess I might have closed this early. Syntax highlighting still isn't working correctly.

@domainzero commented on GitHub (May 31, 2017): Ehhh, I guess I might have closed this early. Syntax highlighting still isn't working correctly.
Author
Owner

@shatteredsword commented on GitHub (Jun 7, 2017):

To clarify, you are talking about the syntax highlighting in the source code view of TinyMCE editor, correct?
I've noticed this was missing as well.

@shatteredsword commented on GitHub (Jun 7, 2017): To clarify, you are talking about the syntax highlighting in the source code view of TinyMCE editor, correct? I've noticed this was missing as well.
Author
Owner

@domainzero commented on GitHub (Jun 11, 2017):

I should have been more clear. I think the only reason I hit this issue was because I pulled from the master branch instead of the release branch.

I wasn't getting any syntax highlighting on saved pages after the removal of hljs. For now I think this is an expected issue as codemirror wasn't 100% implemented at the time that I updated.

I'ma close this and we can sweat the issue if it persists when codemirror is fully implemented.

@domainzero commented on GitHub (Jun 11, 2017): I should have been more clear. I think the only reason I hit this issue was because I pulled from the master branch instead of the release branch. I wasn't getting any syntax highlighting on saved pages after the removal of hljs. For now I think this is an expected issue as codemirror wasn't 100% implemented at the time that I updated. I'ma close this and we can sweat the issue if it persists when codemirror is fully implemented.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#343