Skip to content
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

Documentation issue about textarea set_cursor #753

Closed
massimo79m opened this issue Sep 10, 2023 · 2 comments
Closed

Documentation issue about textarea set_cursor #753

massimo79m opened this issue Sep 10, 2023 · 2 comments
Labels

Comments

@massimo79m
Copy link

massimo79m commented Sep 10, 2023

the documentation says : This method panics if cursor is not the beginning of a character in the content string.
it should report "bytes" in place of "characters".

@massimo79m massimo79m added the bug label Sep 10, 2023
@massimo79m massimo79m changed the title [BUG] textarea setcursor not working on multibytes / multichar grapheme Documentation issue about textarea set_cursor Sep 10, 2023
@massimo79m
Copy link
Author

Please remove the bug label. thank you

@massimo79m massimo79m changed the title Documentation issue about textarea set_cursor lllllllDocumentation issue about textarea set_cursor Sep 10, 2023
@massimo79m massimo79m changed the title lllllllDocumentation issue about textarea set_cursor Documentation issue about textarea set_cursor Sep 10, 2023
@gyscos
Copy link
Owner

gyscos commented Sep 11, 2023

Thanks for the report!

The doc now states:

    /// Moves the cursor to the given byte position.
    ///
    /// # Panics
    ///
    /// This method panics if `cursor` is not the starting byte of a character in
    /// the content string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants