GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||
GeezerVison is showing github.com — gh-143754: Add Tkinter methods pack_content(), place_content() and grid_content() by serhiy-storchaka · Pull Request #143845 · python/cpython Conversation
|
|
The new stuff (on top of what is already in #143841) looks okay without closely examining every detail. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I suggest we also:
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
To be nice, we should deprecate the old methods only when the new methods are available in all maintained Python versions, so the user can simply unconditionally switch to using the new methods. Since this change is just cosmetic, there is no need to even deprecate them in code. The code written 20 years ago will continue to work. We still test the |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…and grid_content() They use Tk commands with new name like "pack content instead of old "pack slaves".
92374bb to
999b0e6
Compare
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Perhaps we could soft deprecate until then?
Anyway, thanks for the PR.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
There is no module documentation where the soft deprecation could be written. The docstrins now say "Synonym for ...". This is an equivalent of soft deprecation. You should not use these methods in new code if you do not support Python 3.14, but you are not forced to make mass renaming. I expect that nobody will use the new methods in the next 3 years. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
…and grid_content() (pythonGH-143845) They use Tk commands with new name like "pack content instead of old "pack slaves".
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.