RF launcher demo - Githubissues

aroberge / easygui_qt

Inspired by easygui, designed for PyQt

BSD 3-Clause "New" or "Revised" License

60 stars 18 forks source link

RF launcher demo #14

Closed jeremygray closed 9 years ago

jeremygray commented 9 years ago

provide a 'quit' button
add place-holder tool-tips; could add more documentation here
parameterize button creation: simplifies code, uses eval() but only for known values

aroberge commented 9 years ago

Seeing the removal of the repetitive code (I should have noticed as I started by copying-pasting code each time I added a widget), I feel greatly humbled (to not say dumb...).

jeremygray commented 9 years ago

Seeing the removal of the repetitive code (I should have noticed as I started by copying-pasting code each time I added a widget), I feel greatly humbled (to not say dumb...).

I feel happy to have my little code snippets included in your project! Some people are deeply averse to using eval(), but I think it makes life more pythonic here.

By reading your code, I'm learning about pyqt and python 3. Thanks for putting this out there, I'm learning a lot from you, André.

aroberge commented 9 years ago

eval() is there for a reason ... and your use seems like an excellent reason to use it! (although, I just thought of a possible way to remove it ... if only to please the "purists"). And, your idea to have tooltips is excellent (I never used them with PyQt before) ... I may just have abused them in the latest commit!

aroberge commented 9 years ago

Yes, it seems possible to use eval. Since you used it, I suspect you did not know of the method I used.