-
Notifications
You must be signed in to change notification settings - Fork 9
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
Highlight Only Visible Area (performance improvement) #31
Comments
Can also fix #5 by checking if emoji's are in the tags and removing those areas. |
|
Ok, I have started a dev branch for this and now need to flesh out the idea fully. |
To determine what starts a MLCDS we can take one in the file and lex it character by character until the tag occurs, mark that as a start and then find the end of one (if any) and then subtract characters until the end disappears and then take the removed characters and mark those as the end. Once those are known, it makes the future much simpler. |
Markdown with backsticks and a second language will be difficult. |
This will fix #9 and will fix #11. The idea? Highlight only text that's visible. This will be done by changing over to a method called
.highlight()
that will highlight all the visible text with reference to all the text in the widget. The structure of the method will be roughly as follows:@rdbende What do you think? I have a feeling this will have a performance improvement for large amounts of code and and that small amounts will still be fine.
The text was updated successfully, but these errors were encountered: