feat: implement syntax highlighting for preview text

[cartercanedy]

Tree-sitter is a permissively-licensed, open-source parser framework, well known for its integrations in many popular editors (i use nvim, btw). Luckily, they provide Rust bindings that are reasonably straightforward to use. This PR implements syntax highlighting for the source preview, making it much easier to read for people wanting to understand the code behind commands that they are running.

Type of Change

• New feature
• Bug fix
• Documentation update
• Refactoring
• Hotfix
• Security patch
• UI/UX improvement

Description

• Implement tree-sitter-based syntax highlighting

Testing

Manual testing by perusing the previews of various commands

Impact

This should make it much easier to understand the source while being presented in the preview

Checklist

• My code adheres to the coding and style guidelines of the project.
• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation.
• My changes generate no errors/warnings/merge conflicts.

[adamperkowski]

Awesome.
Just remember to run cargo fmt --all.

[cartercanedy]

Just remember to run cargo fmt --all.

and remove Cargo.lock
:))

[adamperkowski]

and remove Cargo.lock

Don't remove it if you're adding deps.
If it doesn't conflict, then it's fine.

[cartercanedy]

and remove Cargo.lock

Don't remove it if you're adding deps.
If it doesn't conflict, then it's fine.

Did the Cargo.lock ci check get removed?

[adamperkowski]

Did the Cargo.lock ci check get removed?

It did because Cargo.lock exists for some reason.
Just don't push a deleated and re-cleanbuilded Cargo.lock if not neccessary. - That's the idea here.

[cartercanedy]

Just don't push a deleated and re-cleanbuilded Cargo.lock if not neccessary

makes sense

[Real-MullaC]

Please take a look at the check.

[cartercanedy]

Please take a look at the check.

already on it

[lj3954]

This causes a reversion by removing functionality added in #202.

[cartercanedy]

Yeah, that'll have to get sorted out. I'll have to take a deeper look into breaking up the lines on overflow, since it gets a little complicated with the unrendered ansi sequences in the raw strings

[cartercanedy]

Just perusing the docs a bit, this shouldn't be too hard. I'd probably still use a flat map, just over the constructed lines rather than the raw text to preserve the styling and accurately determine the width of the actually rendered text

[cartercanedy]

@lj3954 why are we looking at wrapping rather than side scrolling?

[ChrisTitusTech]

@lj3954 why are we looking at wrapping rather than side scrolling?

Oh I like side scrolling a lot better.

[cartercanedy]

Oh I like side scrolling a lot better.

Me 2, but there's also the issue of altered semantics of a mid line break without also inserting a trailing* backslash, and even then I'm not sure that backslashes are valid at any arbitrary location*

[cartercanedy]

@ChrisTitusTech @lj3954 just implemented side scrolling to fix regression

[adamperkowski]

v_scroll and h_scroll could be renamed to scroll_v and scroll_h but it's not that big.

[lj3954]

get_lines should not take in an owned String, the split method is implemented for str instead. The lines() method also is more idiomatic than split("\n").

[cartercanedy]

get_lines should not take in an owned String, the split method is implemented for str instead. The lines() method also is more idiomatic than split("\n").

good catch, that was pretty sloppy