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

Review the messages output by pptext #642

Open
windymilla opened this issue Jan 5, 2025 · 3 comments
Open

Review the messages output by pptext #642

windymilla opened this issue Jan 5, 2025 · 3 comments
Labels
future feature New feature or request, but not core

Comments

@windymilla
Copy link
Collaborator

Some of pptext's behavior is based on historical circumstances that may not be so relevant today.

See discussions in #641 for various ideas & comments

@windymilla windymilla added the future feature New feature or request, but not core label Jan 5, 2025
@rtonsing
Copy link

rtonsing commented Jan 5, 2025

Another repeated Character check is for '*'. It would be better to only report any character only once per line, with all of them highlighted.

@windymilla
Copy link
Collaborator Author

Another repeated Character check is for '*'. It would be better to only report any character only once per line, with all of them highlighted.

It would certainly be possible to only report any character once per line. Without a significant change to the checker dialogs, it's not possible to highlight them all individually though, which is why currently the line.col refers to the start of the first occurrence, and it highlights from the start of the first occurrence on the line to the end of the last one.

@rtonsing
Copy link

That is fine. The way some exceptions like '+' are done looks good, just need to extend it to any repeated character. Just saw a file with transcriber's notes set off by lines of 70 '#''s.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
future feature New feature or request, but not core
Projects
None yet
Development

No branches or pull requests

2 participants