feat: use KSyntaxHighlighter for code highlighting#1101
feat: use KSyntaxHighlighter for code highlighting#1101ouuan merged 20 commits intocpeditor:masterfrom
Conversation
|
I have tried running release.yml in my repo and got a lot of errors under Windows. My guess is that |
You may try updating it. It would be preferred if the VS version can be not hardcoded while not adding too much complexity in the scripts.
linuxdeployqt recommends targeting the oldest still-supported Ubuntu LTS release, which is still 18.04. |
|
There are functions like The usage would be when highlighting matching parenthesis we will ignore the parenthesis that comes inside comments, single/double quotes or string literals. Its incomplete here as well. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
I'm trying to use KTextEditor. Currently, basic editing and highlighting work fine. It looks like Kate is studded. |
|
KTextEditor is nice, If we are able to compile on all platforms. Back in the days, it was not fully cross platform. Now it seems a good choice for us. |
|
I recently tried KTextEditor again, referred some code to Kate, and now I can highlight the error line, the text hint is not yet completed, and I guess it will be solved smoothly. But I'm running into some sticky issues.
To use KTextEditor, we need a massive redesign. |
92ebb8e to
5ef969d
Compare
|
I fixed the bracket matching code. Refer to the code of Kate to maintain the Attribute of each row in the highlighter. Now we can directly get the style at a certain position to judge whether it is in the string or comment. So QtFindReplaceDialog waits for cpeditor/QtFindReplaceDialog#9. |
02fee3a to
4b6eb69
Compare
|
I have reviewed the code, and next will test it. Thanks for all your efforts! 😃 |
coder3101
left a comment
There was a problem hiding this comment.
Looks good!
Could you please run the release script in your fork and share the release artifact. We could test if all required dependencies are getting bundled
|
Got it here |
|
@ouuan can you test on Linux? |
|
It generally works. But see the unchecked items in #516. There are some regressions and unfulfilled enhancements, though not very critical. |
|
Minor regression is fine, we can iron it out slowly in subsequent PR, as for small unfulfilling requirements, we can list it out in a single issue and fix it. What do you think @rogeryoungh? |
|
We can merge this and let alpha version users find and report bugs or regression. Hope @rogeryoungh will contribute in fixing and triaging them. |
|
@allcontributors add @rogeryoungh for code |




Use Kate's Syntax highlighter engine for Code Highlighting.
Description
This PR is the same as #516, but without FakeVim.
Related Issues / Pull Requests
#516
Motivation and Context
To make syntax highlighting more better.
How Has This Been Tested?
Tested on Arch Linux.
Screenshots (if appropriate)
Checklist
oldattribute is updated or it is resolved in SettingsUpdater.tr()orQCoreApplication::translate().