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 Jim.Jewett
Recipients Jim.Jewett, asvetlov, gpolo, serhiy.storchaka, terry.reedy
Date 2014-06-02.16:04:24
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1401725065.05.0.728331512821.issue6167@psf.upfronthosting.co.za>
In-reply-to
Content
I'm still not seeing why these changes are sufficiently desirable to justify the code churn.  Nor am I seeing test or doc changes that would explain the advantages of the new way, and prevent future regressions.

I agree that the changes would make the signatures better for the typical use cases for this particular widget -- but wouldn't they also break the common interface for the "set" and "activate" methods across several types of tkinter widget?  

If so, then instead of changing or restricting the method, it would be better to add examples (and maybe even an explanation) to the documentation (including the docstring).




In particular:

(1)  Why change actrivate's parameter from "index" to "element"?  I agree that "element" is a better name for the normal case, but 
https://docs.python.org/dev/library/tkinter.html#the-index-parameter
strongly suggests that "index" is more consistent with the rest of tkinter, and that there are use cases wehre "index" is the right name.  If that is not true, please say so explicitly, at least in comments.

(2)  Why change the "set" method?  I understand that a more specific signature is desirable, and I assume that other values would be ignored (or raise an exception), but the set method seems to have an existing API across several widgets -- and that shouldn't be broken lightly.
History
Date User Action Args
2014-06-02 16:04:25Jim.Jewettsetrecipients: + Jim.Jewett, terry.reedy, gpolo, asvetlov, serhiy.storchaka
2014-06-02 16:04:25Jim.Jewettsetmessageid: <1401725065.05.0.728331512821.issue6167@psf.upfronthosting.co.za>
2014-06-02 16:04:25Jim.Jewettlinkissue6167 messages
2014-06-02 16:04:24Jim.Jewettcreate