Code review comment for lp:~gtg-user/gtg/plugins-preferences-separation
Revision history for this message
| Bertrand Rousseau (bertrand-rousseau) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
It runs fine. I haven't checked the code yet.
Small remarks:
- I would add some padding around the widget (maybe 5-10 pixels) in order to give a better look to the dialogs
- "Preferences" is usually the last item of the Edit menu, so I would move it there. Also there is usually a separator item, I would add one too. That would give:
* Plugins
* Synchronization Services
* --------- (separator)
* Preferences