Opened 15 years ago
Closed 15 years ago
#1805 closed patch (fixed)
Unique Value Renderer multiple class selection issue
Reported by: | gcarrillo | Owned by: | gcarrillo |
---|---|---|---|
Priority: | minor: annoyance | Milestone: | Version 1.2.0 |
Component: | Symbology | Version: | Trunk |
Keywords: | unique value renderer class multiple selection spinbox | Cc: | jef |
Must Fix for Release: | No | Platform: | All |
Platform Version: | Awaiting user input: | no |
Description
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.
Attachments (1)
Change History (9)
by , 15 years ago
Attachment: | patch_for_bug_1805.txt added |
---|
comment:2 by , 15 years ago
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.
comment:3 by , 15 years ago
Cc: | added |
---|
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.
follow-up: 5 comment:4 by , 15 years ago
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.
comment:5 by , 15 years ago
Replying to lutra:
I'll give it a try and I will report back.
the patch works for me under Ubuntu 9.04
follow-up: 7 comment:6 by , 15 years ago
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.
follow-up: 8 comment:7 by , 15 years ago
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.
comment:8 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Patch for this minor issue.