<div>Hi Nyall<br></div><div><br></div><div>It's okay for me, when you merge it to be save and we remove it, when the issue is resolved.<br></div><div><br></div><div>But I'll remove the part that the duplicate functions toggles the edit mode anyway. <br></div><div>So in case there will be still the action to duplicate on a not editable layer somewhere (what it shouldn't) it won't damage the layer.<br></div><div><br></div><div>Thanks and regards<br></div><div>Dave<br></div><div><br></div><div><i>21 February 2018 21:25 Nyall Dawson <nyall.dawson@gmail.com> wrote:</i><br></div><blockquote class="infmailquote"><div>On 22 February 2018 at 06:20, Matthias Kuhn  wrote: <br></div><div> > On 02/21/2018 08:55 PM, Nyall Dawson wrote: <br></div><div> >> On 22 February 2018 at 05:24, Matthias Kuhn  wrote: <br></div><div> >>> Hi Dave, hi Nyall, <br></div><div> >>> <br></div><div> >>> Thanks for all your thoughts and work on this. The approach that is <br></div><div> >>> currently being developed sounds like a very reasonable one given the <br></div><div> >>> remaining time for 3.0. <br></div><div> >>> <br></div><div> >> In the meantime, I'd like to merge that PR. 3.0 is only a couple of <br></div><div> >> hours away and it makes me very nervous having this feature exposed <br></div><div> >> still. Can I merge, and then we can address whether to remove the flag <br></div><div> >> when the changes have landed? <br></div><div> > <br></div><div> > I don't know enough about it myself, I only followed this topic marginally. <br></div><div> > <br></div><div> > It would be good to get the green flag for it from Dave and Andreas if <br></div><div> > somehow possible. <br></div><div> <br></div><div> Yes, ideally... but also (with full respect to all involved), in its <br></div><div> current form this feature is just too dangerous to have in the <br></div><div> release. I would not feel comfortable at all in rolling out 3.0 in my <br></div><div> customer's organizations with the knowledge that a single accidental <br></div><div> slip of the mouse can permanently damage a non-editable layer. <br></div><div> <br></div><div> That's why I would greatly prefer to hide it now, and then un-hide if <br></div><div> the issues are resolved before release. <br></div><div> <br></div><div> "Better safe than sorry" and all that :) <br></div><div> <br></div><div> Nyall <br></div><div> _______________________________________________ <br></div><div> QGIS-Developer mailing list <br></div><div> QGIS-Developer@lists.osgeo.org <br></div><div> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer <br></div><div> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer <br></div><div> <br></div></blockquote><div><br></div><div><br></div><div id="editorUserSignature"><p><span class="font" style="font-family:monospace">---------------------</span><br></p><p><span class="font" style="font-family:monospace">David Signer</span><br></p><p><span class="font" style="font-family:monospace">Programmer OPENGIS.CH</span><br></p><p><span class="font" style="font-family:monospace">Wülflingerstrasse 213</span><br></p><p><span class="font" style="font-family:monospace">CH - 8408  Winterthur</span><br></p><p><span class="font" style="font-family:monospace">+ 41 (0) 78 766 13 03</span><br></p><p><span class="font" style="font-family:monospace">---------------------</span><br></p></div><div><br></div>