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 Todd.Rovito, r.david.murray, roger.serwy, terry.reedy
Date 2013-06-01.03:30:02
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1370057404.17.0.81353710638.issue18104@psf.upfronthosting.co.za>
In-reply-to
Content
While improving the self-test for confixSectionNameDialog.py after fixing the error that stopped it from running, I discovered an error in the NameOk method: names were not really stripped before being tested. so duplicate section names were erroneously accepted if initially surrounded by whitespace. After fixing that, I removed unneeded code, including 5 pairs of parentheses in one statement, and added recommended spaces. I also centered the action buttons in the dialog.

Attached is a drop-in replacement for the file (produced with Idle on my non-developement  Win7 machine, so probably with the extra line end char). I think it is good enough to commit, but I am curious if the instructions added to the test are clear enough. (The dialog is the one seen if Save as New Custom Theme (Highlighting tab) or Save as New Custom Key Set (Key tab) are selected on the Options / Preferences dialog.

For new custom test files, the root window should be open farther from the edge of the screen.
History
Date User Action Args
2013-06-01 03:30:04terry.reedysetrecipients: + terry.reedy, roger.serwy, r.david.murray, Todd.Rovito
2013-06-01 03:30:04terry.reedysetmessageid: <1370057404.17.0.81353710638.issue18104@psf.upfronthosting.co.za>
2013-06-01 03:30:04terry.reedylinkissue18104 messages
2013-06-01 03:30:03terry.reedycreate