This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author terry.reedy
Recipients THRlWiTi, ajaksu2, drhok, gpolo, josiahcarlson, kbk, rhettinger, roger.serwy, terry.reedy
Date 2016-08-15.17:49:55
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1471283396.21.0.543806409518.issue1442493@psf.upfronthosting.co.za>
In-reply-to
Content
manual_wrap.patch patches OutputWindow (2.7) to arbitrarily and blindly wrap at 80 chars.  OutputWindow is used directly for grep output, as well as the base for PyShell.  The grep output format is currently "'{}: {}: {}'.format(filepath, linenum, codeline)".  Output lines are typically (for me) > 80 chars and I would not want a fixed-column wrap.  With a right click, one can go to the file and line and this should not be disabled.  Ditto for tracebacks, where code lines are pre-wrapped (with an added indent) onto a second physical line. Wrap at 80 would wrap lines that were originally 80 before having the traceback indent added.  Autowrap should only be applied to user stdout sent to Shell.

Perhaps wrapping should have a window (80-100?) within which we look for a space.  I have about concluded that we should add horizontal scrollbars anyway, since Python and IDLE output lines longer than 80 chars.

To evaluate the patch further, I want to look at how the socket stream is being read.

As long as we are modifying user output before inserting into the text widget, astral chars should be expanded into their unicode escapes. (There are multiple issue for astral chars.  Tk 8.7 reportedly will handle them.) The replacement text should be tagged and colored as such.  Wrapping should not break replacements.  The same could be done for control chars to make them visible.  (Astral char handling is needed for paths also!).
History
Date User Action Args
2016-08-15 17:49:56terry.reedysetrecipients: + terry.reedy, rhettinger, kbk, josiahcarlson, drhok, ajaksu2, gpolo, roger.serwy, THRlWiTi
2016-08-15 17:49:56terry.reedysetmessageid: <1471283396.21.0.543806409518.issue1442493@psf.upfronthosting.co.za>
2016-08-15 17:49:56terry.reedylinkissue1442493 messages
2016-08-15 17:49:55terry.reedycreate