good point. so mainly we are talking prohibiting CursorTools to edit invisible layers. maybe we should think smaller and just change the way CursorTools determine the layer to edit.
..ede On 19.10.2016 11:20, Rahkonen Jukka (MML) wrote: > Hi, > > Trying to think how we actually edit layers: > > - from the map > - from attribute window > - with tools which edit attributes and geometries in place > > Isn't it only when doing edits from the map when the layer really must be > visible? > > -Jukka- > > edgar.soldin wrote: > >> Mike, > >> had a quick look. simplest approach might be just editing >> AbstractLayerable.isEditable() to always return boolean sum of editable && >> visible, no? all layerables i know are based on AbstractLayerable. > >> plugins probably all use TreeLayerNamePanel.chooseEditableLayer() *but* do >> not have to. so there is a slight chance that code would edit the layer >> anyway, because it still sees it as editable and didn't use the >> chooseEditableLayer() to retrieve it. > >> ..ede > > On 18.10.2016 21:23, Michaël Michaud wrote: >>> Hi all, >>> >>> Thank you for your feedback, >>> >>> I made a second attempt to solve the problem in a more conservative way >>> (r5083+). >>> >>> Instead of making invisible layers uneditable, I changed the method >>> chooseEditableLayer used by most editing tools so that it does not choose >>> invisible layers any more. >>> Now, you can let invisible layer editable, but editing tools will never use >>> it until you make it visible again. >>> >>> Let me know what you think, >>> >>> Michaël >>> >> >> Le 18/10/2016 à 15:06, Stefan Steiniger a écrit : >>> >>> I think I somewhat agree with Jukka. I also look once in while what is >>> underneath by shortly making it invisible... >>> >>> stefan >>> >>> >>> On 10/18/16 09:56, Rahkonen Jukka (MML) wrote: >>>> >>>> Hi, >>>> >>>> >>>> >>>> I agree that it is logically good that non-visible layer is not editable >>>> because I can't imagine a real world use case where it would be better to >>>> create features without seeing the other features of the layer. However, >>>> sometimes it is useful to switch the layer off and have a look under the >>>> existing lines and polygons which are covering the background. Now turning >>>> layer non-visible turns it also into non-editable and user must >>>> re-activate the edit mode. I am not sure if this automatic switch to >>>> non-editable is an improvement or not. With the current default "only one >>>> editable layer at a time" the risk to digitize features into a non-visible >>>> layer by accident is not so big because of the radio button behavior. If >>>> user starts to edit some other layer the non-visible layer turns into >>>> non-editable anyway. >>>> >>>> >>>> >>>> -Jukka Rahkonen- >>>> >>>> >>>> >>>> >>>> >>>> >>>> >>>> -------------------------------------------------------------------- >>>> ---------- Check out the vibrant tech community on one of the >>>> world's most engaging tech sites, SlashDot.org! >>>> http://sdm.link/slashdot >>>> >>>> >>>> _______________________________________________ >>>> Jump-pilot-devel mailing list >>>> Jump-pilot-devel@lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >>> >>> >>> >>> --------------------------------------------------------------------- >>> --------- Check out the vibrant tech community on one of the world's >>> most engaging tech sites, SlashDot.org! http://sdm.link/slashdot >>> >>> >>> _______________________________________________ >>> Jump-pilot-devel mailing list >>> Jump-pilot-devel@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >> >> >> >> ---------------------------------------------------------------------- >> -------- Check out the vibrant tech community on one of the world's >> most engaging tech sites, SlashDot.org! http://sdm.link/slashdot >> >> >> >> _______________________________________________ >> Jump-pilot-devel mailing list >> Jump-pilot-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >> > > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most engaging tech > sites, SlashDot.org! http://sdm.link/slashdot > _______________________________________________ > Jump-pilot-devel mailing list > Jump-pilot-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, SlashDot.org! http://sdm.link/slashdot > _______________________________________________ > Jump-pilot-devel mailing list > Jump-pilot-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
