<div dir="ltr">Having nothing to do with the code I'd vote for removing QgsColorButton and letting the V2 do all the work (sounds newer to me - probably the original is only used by old dialogues)<div style>Inserting it in the Widget itself sounds also resonable to me .. probably search for some special cases where it would not be needed - but I think it does not hurt to have it implemented in the Widget .. </div>
<div style>Thanks for doing all this work!</div><div style><br></div><div style>kind regards</div><div style>Werner</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 7, 2013 at 4:13 PM, Matthias Kuhn <span dir="ltr"><<a href="mailto:matthias.kuhn@gmx.ch" target="_blank">matthias.kuhn@gmx.ch</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
I've just had a look at QgsColorButton.<br>
<br>
There are two implementations available (QgsColorButton and QgsColorButtonV2). Would it be okay to remove one of them before the 2.0 release? If yes, which one should be removed?<br>
<br>
I also noticed, that in several places, the onClick event is handled to open a QColorDialog and assign the selected color to the button.<br>
Wouldn't it be nice, if this behavior would be part of the widget implementation itself?<br>
<br>
Regards,<br>
Matthias<br>
______________________________<u></u>_________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/qgis-<u></u>developer</a><br>
</blockquote></div><br></div>