[issue23667] IDLE to provide option for making trailing whitespace visible

Terry J. Reedy added the comment: Given the issues I raised and Raymond's rejection of this, his first suggested option, on the PR, in favor of his second suggested option, #33046, I agree, at least for now. The use case for whitespace tagging would be if someone wants to selectively delete

[issue23667] IDLE to provide option for making trailing whitespace visible

Tal Einat added the comment: See also issue33046 specifically suggesting auto-removal of trailing whitespace on saving a file, as per Raymond's second suggestion here. It has a PR which seems about ready to go. I see that as the simplest approach to this issue, both in terms of implementati

[issue23667] IDLE to provide option for making trailing whitespace visible

Terry J. Reedy added the comment: Raymond, which definition of 'whitespace' do you intent? Git gui also marks trailing spaces in frozen diffs. This is much harder to do while editing as text can be read, keyed, and pasted. Most spaces and blank lines are only temporarily trailing, before s

[issue23667] IDLE to provide option for making trailing whitespace visible

Cheryl Sabella added the comment: Thanks, Terry! I'll take a look at 30290. No need to start with colorizer; it was in this ticket so I referenced it more for convenience than anything. -- ___ Python tracker ___

[issue23667] IDLE to provide option for making trailing whitespace visible

Terry J. Reedy added the comment: Louie: It appears that old PRs might merge as they are but any change, including an update merge, requires that you close and re-open. When you re-open, please add "was pr". It would be easier for me if you redid all the IDLE PRs in a batch or two. Cher

[issue23667] IDLE to provide option for making trailing whitespace visible

Cheryl Sabella added the comment: Terry, Should IDLE library modules get docstrings or is that something you'd rather not have? Putting the question here because ColorDelegator in colorizer.py doesn't have any docstrings. I wanted to read some of the IDLE code and I can try to add missing d

[issue23667] IDLE to provide option for making trailing whitespace visible

Louie Lu added the comment: terry, these close isn't changing anything. But I'm migrating my GitHub account from old one to new one, thus will close the PR and reopen another one, sorry for annoying mailing. -- ___ Python tracker

[issue23667] IDLE to provide option for making trailing whitespace visible

Terry J. Reedy added the comment: Louie, when you close and open PRs, please give at least a brief explanation. In particular, how is the new one different? -- assignee: -> terry.reedy ___ Python tracker ___

[issue23667] IDLE to provide option for making trailing whitespace visible

Changes by Louie Lu : -- pull_requests: +1739 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue23667] IDLE to provide option for making trailing whitespace visible

Changes by Louie Lu : -- pull_requests: +1738 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue23667] IDLE to provide option for making trailing whitespace visible

Changes by Louie Lu : -- pull_requests: +1737 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue23667] IDLE to provide option for making trailing whitespace visible

Changes by Terry J. Reedy : -- stage: needs patch -> patch review versions: +Python 3.6, Python 3.7 -Python 2.7, Python 3.4, Python 3.5 ___ Python tracker ___ ___

[issue23667] IDLE to provide option for making trailing whitespace visible

Louie Lu added the comment: PR 1602 add trailing whitespace highlight, which using color from error. PR 1603 add TrimExtension to provide auto trim while saving the file. -- nosy: +louielu ___ Python tracker _

[issue23667] IDLE to provide option for making trailing whitespace visible

Changes by Louie Lu : -- pull_requests: +1694 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue23667] IDLE to provide option for making trailing whitespace visible

Changes by Louie Lu : -- pull_requests: +1693 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue23667] IDLE to provide option for making trailing whitespace visible

Terry J. Reedy added the comment: I agree that this area needs improvement. 'Strip trailing whitespace' should strip trailing blank lines at the end of a file, as well as trailing blank spaces at the end of each line. Both are required to commit to our repository. I would like to have a conf

[issue23667] IDLE to provide option for making trailing whitespace visible

New submission from Raymond Hettinger: We have an option to clear trailing whitespace (Format / Strip trailing whitespace) but it isn't always clear when to use it (because the trailing whitespace is invisible). One thing the can be done is to colorize trailing white space to make it more vis