bpo-30495: IDLE: refactor textview by csabella · Pull Request #2283 · python/cpython (original) (raw)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I said, "lets try TextviewWindow and TextviewFrame." Having tried them, I don't like them. ViewWindow is enough. Also, TextviewFrame should be split into a ViewFrame containing a TextFrame.

The close button should be in ViewFrame. It is currently redundant but will be needed if a Viewframe is tiled in a window, as Mark proposed for About IDLE.

I will give these a try.