fix: Use correct offset when trimming unicode char

[Muscraft]

#330 showed a few cases where annotate-snippets would annotate a line incorrectly when it tried to trim unicode characters. Two things caused this:

• Not adjusting the left padding if it didn't match skipped
• Not skipping multi-byte chars when their width + skipped exceeded left.

This PR addresses both of those issues.

Before:

After:

Test Script

use annotate_snippets::{AnnotationKind, Group, Level, Renderer, Snippet};

fn main() {
    /***** (A): ASCII only *****/
    let ascii_only_end = (
        "(A): ASCII only",
        "/*aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa*/?",
        "?",
    );

    /***** (B): non-ASCII prefix, ASCII highlight *****/
    let non_ascii_pre_ascii_highlight_end = (
        "(B): non-ASCII prefix, ASCII highlight",
        "/*这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。*/?",
        "?",
    );

    /***** (C): non-ASCII everything *****/
    let non_ascii_end = (
        "(C): non-ASCII everything",
        "/*这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。*/好",
        "好",
    );

    /***** (D): non-ASCII everything, annotate middle *****/
    let non_ascii_middle = (
        "(D): non-ASCII everything, annotate middle",
        "/*这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽好。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。这是宽的。*/?",
        "好",
    );
    for (desc, source, highlight) in [
        ascii_only_end,
        non_ascii_pre_ascii_highlight_end,
        non_ascii_end,
        non_ascii_middle,
    ] {
        //println!("\n");
        let renderer = Renderer::styled();

        let start = source.find(highlight).unwrap();
        let range = start..start + highlight.len();
        let annotation = AnnotationKind::Primary.span(range.clone());

        let group_with_label = Group::with_level(Level::ERROR)
            .element(Snippet::source(source).annotation(annotation.clone().label("label")));
        println!("{desc}\n{}\n", renderer.render(&[group_with_label]));

        let group_no_label =
            Group::with_level(Level::ERROR).element(Snippet::source(source).annotation(annotation));

        println!("{}\n", renderer.render(&[group_no_label]));
    }
}

fixes #330

[epage]

does this commit pass on its own? I'm not seeing the edit that would cause this output change (and ascii is unchanged)

[Muscraft]

That change shouldn't have been there. I'm not sure how I missed it.

[epage]

then reverted here

[epage]

Feel free to merge once the commits are fixed