How good a fit is this for tklib ? Coding standards ?
Completeness of the submission (Documentation present ?
Testsuite present ?). Does the API make sense ? Is that
something you would find useful for yourself and use as well ?
A meta-topic to consider: There is not much in tklib yet, so
anything we add now will at least partially set the tone for
future packages.
I placed the burden on you as I am not that much of a Tk
person, whereas I believe you are such :) I hope that is ok.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
menu management package
Logged In: YES
user_id=75003
Bryan, do you have time for a review ?
Logged In: YES
user_id=493507
Hello,
I have done some bug / style fix since I have used it in
real application.
Here is the latest version.
Logged In: YES
user_id=249075
Yes, I'll try to find time to review it, though it might
take me a couple days. What sort of review are you looking for ?
Logged In: YES
user_id=75003
How good a fit is this for tklib ? Coding standards ?
Completeness of the submission (Documentation present ?
Testsuite present ?). Does the API make sense ? Is that
something you would find useful for yourself and use as well ?
A meta-topic to consider: There is not much in tklib yet, so
anything we add now will at least partially set the tone for
future packages.
I placed the burden on you as I am not that much of a Tk
person, whereas I believe you are such :) I hope that is ok.