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
Date 2012-04-06.22:45:30
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1333752331.34.0.503158251741.issue6167@psf.upfronthosting.co.za>
In-reply-to
Content
(1)  Why did you change the name of the parameter from index to element?  If the underlying engine also accepts indices (e.g., self.activate(4) ) then the name should stay.  If it really is only meaningful to use the name of an element -- maybe it should still stay for backwards compatibility.  Or at least accept the old name too for a release.

Either way, please provide a test case showing that it works under the new name; there may also be doc fixes.  (I'm not sure there is documentation for this widget, though, and providing some in the first place is good, but perhaps a different task.)

FWIW, 
http://docs.python.org/dev/library/tkinter.html#the-index-parameter
suggests that the name should stay index, and can be far more than an element; migrating some of that to the docstring might be helpful.

(2)  It looks like the set command took *args to give some freedom.  There may be extensions that take a third argument.  It may well be valid to call it without any arguments, or with only one.  So the signature may turn into set(first=0, last=1, *args) or some such.  Whatever the answer about what arguments are really needed, there should be test cases to demonstrate this before the API is changed.
History
Date User Action Args
2012-04-06 22:45:31Jim.Jewettsetrecipients: + Jim.Jewett, gpolo, asvetlov
2012-04-06 22:45:31Jim.Jewettsetmessageid: <1333752331.34.0.503158251741.issue6167@psf.upfronthosting.co.za>
2012-04-06 22:45:30Jim.Jewettlinkissue6167 messages
2012-04-06 22:45:30Jim.Jewettcreate