Message294298
| Author |
terry.reedy |
| Recipients |
cheryl.sabella, louielu, terry.reedy |
| Date |
2017年05月23日.23:56:19 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1495583779.93.0.14604024229.issue30290@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
At first glance, 1714 looks great. I hope to merge it tomorrow.
On the issues you raised:
1. I want to leave existing constants alone for now.
2. There are two 'create widgets' styles: create, place, create, place, ...; and create, create, ..., place, place, ... . Both have their logic, both are used in IDLE. The first makes sure that everything created gets placed. The second makes it easier to arrange things in relation to each other and to rearrange. I naively started with the first, but it seems that most experts advocate the second.
3. If the dialog were left alone, three helper functions could be a sensible refactoring.
However, a alternate design could have links with standard blue tagging instead of buttons. For example, the current About Firefox box has these 8 links: 'What's new', 'Mozilla', 'a global community', 'Make a donation', 'get involved', 'Licensing Information, 'End-User Rights', and 'Privacy Policy'. This would make code much shorter and the result would not have the issues above. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2017年05月23日 23:56:20 | terry.reedy | set | recipients:
+ terry.reedy, louielu, cheryl.sabella |
| 2017年05月23日 23:56:19 | terry.reedy | set | messageid: <1495583779.93.0.14604024229.issue30290@psf.upfronthosting.co.za> |
| 2017年05月23日 23:56:19 | terry.reedy | link | issue30290 messages |
| 2017年05月23日 23:56:19 | terry.reedy | create |
|