Szilveszter Farkas has voted tweak.
Status is now: Conditionally approved
Comment:
Some suggestions:

1. Please use os.path.join() instead of iconpath+"iconname.png".

2. I had to tweak setup.py a bit to get all the needed icons installed 
into the right place (most of the icons of Olive get installed to 
"share/olive/icons", but the code searches for them in 
"share/bzr-gtk/icons")

3. I miss the separators from the main toolbar.

4. Branch/Tags menu icons seems to be unknown.

If you fix these issues, I'd be happy to merge your branch into trunk.

Thanks,
Szilveszter

For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C4880613C.2040905%40xs4all.nl%3E

-- 
bzr-gtk mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.canonical.com/mailman/listinfo/bzr-gtk

Reply via email to