Pavel Sanda wrote:
> yes i understood your plan :) this way when you create new group and then
> click on another image without applying the group is gone. but i dont care
> much about it, if you like it this way, we can use it.

Yes, I prefer it that way. On the contrary, I think the current behaviour, if 
I understand correctly, is wrong: in the current approach, the graphics params 
are changed without the user pressing apply (simply if the group widget's 
content is changes). We should never ever change parameters without explicit 
user confirmation (after all, there's no auto-apply checkbox in this dialog).

> there is one serious bug with your patch though - you need to load group
> settings into the dialog when selected in the combobox, otherwise you
> destroy group settings by the current one after apply.

I don't understand. Can you provide a recipe?

Jürgen

Reply via email to