-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[POC][DO NOT MERGE] Added improved diff lexer #7806
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@wouterj This is a DoNotMerge... but I want us to merge it :). I know this was a quick experiment - what's missing? Thanks! |
Ping @wouterj! I had another idea since I can see how powerful this is... Could we not use the A) Does what your lexer does now - looks for a Wouldn't that allow us to have non-copiable diffs AND highlighting? |
The code block added in #7868 is a good example for this diff and could be adapted accordingly here. |
Hey @bellisk! You should totally check this out ;) |
@wouterj just asking: what's the plan for this PR? Are we going to finish it to replace the existing |
I'm closing this PR as "won't merge" ... because we're working on a new doc builder which will be completely incompatible with this. But the idea is very nice ... so we'll implement it in the new builder. Thanks. |
What syntax highlighter is used in the new builder? |
I've replied to you via Slack with some more details about the upcoming doc builder. Thanks. |
Just to play with something discussed this afternoon: Allowing to show diff, but making
+
/-
not copyable.View the page on platform.sh:
And make sure you inject the following CSS when viewing:
/cc @weaverryan