QWAT icon indicating copy to clipboard operation
QWAT copied to clipboard

[QGIS] Snapping on invisible feature

Open dsavary opened this issue 7 years ago • 23 comments

In QGIS, we can snap on invisible features even if the snapping options have been defined as "snap on all visible layers". The snapping options don't take into accoount the layer style.
Will this be fixed for QGIS 3.0 ?

Démo GIF : https://share.oslandia.net/public/b91c8e

dsavary avatar Jun 01 '17 14:06 dsavary

Hi @dsavary let's split it in two phases:

  • do you have qgis 2.99 installed? That could let you test directly in qgis nightlies. You can have them using the osgeo4w installer, and updating is only a 5-6 click job (just run the osgeo4w/bin/setup.bat and choose advanced installer)

  • before fixing what appears to you as a bug, it must be discussed and see if it really is a bug or a feature for all users. When it appears strange to you, it might be a useful feature to other. When you say layer style is not taken into account, are you talking about invisible feature that are:

    • unchecked classes ?
    • unclassified object (ie object not in any category or range class and not falling in a Else Filter)
    • fully transparent ?
    • not visible because of a blending mode effect ?
    • classified but set with "no symbol" style

For unchecked classes, that would make sense not to have a snapping index. For the others, we'll need to discuss this largely with the community.

haubourg avatar Jun 01 '17 15:06 haubourg

I tested with QGIS 2.99 3 week ago. The bug is still not fixed. I talk about invisible feature that are unchecked classes . The snapping options check only if the layer are checked. Perhaps, it's not a bug, but a new feature request .

dsavary avatar Jun 01 '17 15:06 dsavary

Ok, this case is probably an issue. Let's raise the issue on the mailing list.

haubourg avatar Jun 01 '17 15:06 haubourg

thread opened here http://osgeo-org.1560.x6.nabble.com/issue-with-snapping-on-unchecked-legend-classes-td5322685.html

haubourg avatar Jun 01 '17 16:06 haubourg

About snapping in QGIS 3 https://github.com/QGEP/QGEP/issues/330

ponceta avatar Jun 12 '17 13:06 ponceta

For Lausanne, this issue is important. In the priority, it should be fix for QGIS 3.0 , ideally for QGIS 2.18 LTR if it's still possible. I don't understand the use cases where you would snap the new feature to invisible features on the map ...

dsavary avatar Jun 21 '17 16:06 dsavary

ticket created upstream https://issues.qgis.org/issues/16838

My guess is that the whole snapping workflow deserves something consistent. We have quite a bunch of good issues and feature requests in that area

haubourg avatar Jul 10 '17 14:07 haubourg

@dsavary backporting a change to 2.18 seems not possible according to @wonder-sk, it's a little bit more than only a bugfix. @qwat-psc: what is the priority of this issue when compared to all other issues with snapping? I have in mind the following needs:

  • improve and fix the new snapping dialog (help to display the dock, intensive test ) #228
  • add snapping priority and interactive selection (visual snapping - part 2)
  • allow select all and deselect all in advanced snapping options
  • layer finder filter (for projects with lots of layers)

haubourg avatar Jul 11 '17 06:07 haubourg

IMHO, Snapping on invisible features is a feature.

Removing it completely would be wrong. I suggest to set it optionnal in the same way that snapping on intersections is optionnal.

Default could be set to visible only.

ponceta avatar Oct 02 '17 13:10 ponceta

@wonder-sk Hi Martin, I have 2 days available in the end of 2017 budget to fix that in QGIS 3 or 3.2. Would you be available to work on that?

haubourg avatar Nov 24 '17 08:11 haubourg

@wonder-sk Hi in case you had some time left, We are out of budget for 2017. Would you be interested in that for 2018 ?

haubourg avatar Dec 06 '17 11:12 haubourg

Sorry for the delayed response... Sure we can do it in 2018 if you are interested in this feature.

wonder-sk avatar Dec 22 '17 11:12 wonder-sk

@wonder-sk well, let's keep that in standby that until I get a clearance for the 2018 actions. Any time estimate on that task?

haubourg avatar Dec 22 '17 15:12 haubourg

Cool... My estimate for this is ~2 days

wonder-sk avatar Dec 22 '17 15:12 wonder-sk

We have an external contributor funding that fix. Work starting soon, thanks!

haubourg avatar Mar 15 '18 12:03 haubourg

Good news ! planned for QGIS 3.1 (2018) ? I think not for QGIS 2.18 :) ...

dsavary avatar Apr 11 '18 14:04 dsavary

Planned for 3.2. 3.1 is the current master development version.

2.18 was investigated by Martin and needed API changes >> NoGO

haubourg avatar Apr 12 '18 07:04 haubourg

@lbartoletti hub cleanup, I assign you to the task you are already working on

haubourg avatar Apr 12 '18 12:04 haubourg

PR Merged https://github.com/qgis/QGIS/pull/6750

lbartoletti avatar May 02 '18 07:05 lbartoletti

Please keep it opened until we have a green light from the testers :)

haubourg avatar May 02 '18 07:05 haubourg

Could be closed now?

lbartoletti avatar Nov 12 '18 07:11 lbartoletti

The main bug was fixed one year ago. I fixed a related bug for tracing.

FYI, there is a last related bug https://github.com/qgis/QGIS/issues/26733

Do you want me to take care of it?

lbartoletti avatar Jun 26 '19 06:06 lbartoletti

Do you want me to take care of it?

are you talking to me? ;-) Please go ahaed and close qgis/QGIS#26733 and https://github.com/qwat/QWAT/issues/193 if everything is OK:

haubourg avatar Jun 28 '19 09:06 haubourg