Unique Value Renderer multiple class selection issue

[gcarrillo]

On Unique Value Renderer, when I select multiple classes on the class list and edit a double setting (lineWidth or pointSize) I get bad behavior of the enable/disable spinBoxes. Even when the selected classes has the same setting (lineWidth or pointSize) the spinBox is disabled.

[gcarrillo]

Patch for this minor issue.

[lutra]

Hi,

thanks for the patch.

Have you seen #1703? Is this related?

[gcarrillo]

The #1703 ask for new color buttons behavior (and others) when multiple classes are selected.

The code for this class set to disabled each symbology setting when the classes has different value for this setting. As jef says, it is intentional. I think the #1703 enhancement could be useful.

So, this is related with #1703 but only fix a little thing and doesn't make nothing for that.

[lutra]

Hi,

I gave a better look/try to #1703 and it seems to me that it also fixes the small problem that lead you to create this patch. In fact if you leave the "restrict changes to common properties" check-box unchecked, you can change the lineWidth or pointSize of multiple classes selection regardless if the lineWidth or pointSize is the same or if it is different.

[lutra]

I gave another look, and at the end it seems I was wrong, in fact I noticed now that if I leave the "restrict changes to common properties" check-box unchecked, then obviously all the classes will take the same style options.

As a matter of fact if I select a few classes, change the outline width, unselect them and then select them again, the outline width selector is disabled, so your patch will correct indeed this issue.

I'll give it a try and I will report back.

[lutra]

Replying to lutra:

I'll give it a try and I will report back.

the patch works for me under Ubuntu 9.04

[gcarrillo]

Thanks lutra, just a thing: The patch contains the function isEqual, which can be useful to other classes in spite of be a very small function. I don't know if QGis has something like Utils class or something like that where it could be better placed.

[jef]

Replying to gcarrillo:

Thanks lutra, just a thing: The patch contains the function isEqual, which can be useful to other classes in spite of be a very small function. I don't know if QGis has something like Utils class or something like that where it could be better placed.

I introduced a function to compare doubles yesterday. r11244 makes use of that now.

[lutra]

I introduced a function to compare doubles yesterday. r11244 makes use of that now.

works for me