snapping allows to snap non displayed legend classes

[qgib]

Author Name: Regis Haubourg (@haubourg)
Original Redmine Issue: 16838
Affected QGIS version: 3.0.0
Redmine category:digitising
Assignee: Martin Dobias

---

When unchecking visibility of some classes for categorized or rule based renderer, snapping is still possible even though those features are not visible on map canvas.
This is annoying for some users playing a lot with classes visibility.

Thinking of it, fixing it might not be so direct, as it will imply rebuilding the snapping cache a lot more often.
And currently, it is also possible to keep snapping on an unchecked layer if it is still checked in snapping advanced dock.

Any opinion here?

[qgib]

Author Name: Regis Haubourg (@haubourg)

---

• subject was changed from snapping allons to snap non displayed legend classes to snapping allows to snap non displayed legend classes

[qgib]

Author Name: Martin Dobias (@wonder-sk)

---

Fixing that may not require rebuilding of snapping index - snapping classes allow use of custom filters, so one could create a filter that would skip any matches that belong to invisible features...

[qgib]

Author Name: Regis Haubourg (@haubourg)

---

Good news!

Do you think that could be considered as a bugfix for backporting to 2.18 ?

[qgib]

Author Name: Martin Dobias (@wonder-sk)

---

Not really, I think the amount of required changes would be larger than what can be still considered as a bug fix safe for backporting...

[qgib]

Author Name: Regis Haubourg (@haubourg)

---

Work will start soon on that. Thanks a lot to the funder!

---

• assigned_to_id was configured as Martin Dobias
• version was changed from 2.18.10 to 3.0.0

[qgib]

Author Name: Luigi Pirelli (@luipir)

---

I can't talk from a user point of view, but I feel that snapping should't be related with the fact that the layer is rendered or not (you may thing to snapToGrid, where grid is not a layer and nor is rendered).
If, from the usability point of view, this is the expected behaviour, I would expect that it would be configurable.
IMHO it's not a bug, but a feature.

[qgib]

Author Name: Loïc BARTOLETTI (@lbartoletti)

---

Done in master #6750

[qgib]

Author Name: Regis Haubourg (@haubourg)

---

• resolution was changed from to fixed/implemented
• status_id was changed from Open to Closed