Hello, Graeme Geldenhuys.

I've made small corrections to your package:
- buttons should be freed after configure dialog (this will free
unneeded memory and aslo solves problem with thir doubling after
configure dialog)
- error after buttons destruction caused by configure button (it was
clicked and destroyed before click finished). I moved it declaration
to class namespace (CfgButton) and skip it destruction (we can skip,
because it will be created in any case)
- setted property cancel to true to cancel button in config dialog
- moved recreate toolbar code from config dialog to toolbar object

Patch attached.

If you have no objections, I will apply it.


ps:
 sorry, if my english is not so good :)

-- 
Best regargs,
  Paul Ishenin                           mailto:[EMAIL PROTECTED]

Attachment: editor_toolbar.patch
Description: Binary data

Reply via email to