Code review comment for lp:~chipaca/ubuntuone-client/nautilus-ppage

Revision history for this message
Rodrigo Moya (rodrigo-moya) wrote :

yay! But a few things:

ubuntuone-nautilus.c is too crowded, so could you move the property page creation/code to its own file, like location-widget.c does?

If the data is not editable, why not just use a label? See the 'Audio' property page for MP3's, it has sections (in bold) and then the different data in labels, with nice italic strings for the "field" names. Something like that would look much better.

Also, the 'Ubuntu One' page shows for every folder/file, whether it's a UDF/inside a UDF or not.

review: Needs Fixing

« Back to merge proposal