It is possible close the polygon with dbclick about leaflet.draw

[jacobtoye]

Yeah it totally would be. I've thought about this before but haven't got around to implementing it. Just need to add the handler to the polyline/polygon draw handler.

Feel free to submit a pull request, otherwise I'll look into it after the Christmas break.

[ekaitzht]

I have tried to do in my project but I don´t know how to differ between 'click' and 'dblclick'. This issue is in https://github.com/CloudMade/Leaflet/issues/108

[ekaitzht]

@jacobtoye I have implemented the 'dblclick' but I think that is not very good code, I am very newbie developing. Do you want that I send to you my code?

[jacobtoye]

@ekaitzht we were all newbie devs at one stage, it's great that you are taking the initiative and working in open source :)

You can always commit your code the your Leaflet.draw repo (make a brach first). I can then check it out for you.

[ekaitzht]

@jacobtoye first I am going to read something about branches I hope to commit the code soon.

[ekaitzht]

@jacobtoye I am going to read something about branches and how it works that in github, I hope to send the code soon.

[ekaitzht]

@jacobtoye I haved put a dblclick functionaly, reply me if you want to modify code.

[ekaitzht]

@jacobtoye the problem is that we have to add a event to the same element. In JavaScript doesn´t know to differ when 'dblclick' and 'click' is in the same element.

[ekaitzht]

any news about my problem :(

[jacobtoye]

I see the issue (distinguishing the click and double click), however I don't think it is quite a high priority issue at the moment. The tooltip tells the user to close the shape by clicking the first point.

Right now I am working on the 0.2 branch when I get any spare time. I left this issue open to come back to it at a later date.

[ekaitzht]

@jacobtoye thank you, I would like to help but I don't know how to implement using the dblclick marker event. issue #56

[jacobtoye]

That's fine, I don't think this functionality is super important yet.

The issue with you pull was that you edited the built js files (in the /dist folder). What you want to do is edit the relevant files in the /src directory. When the build script is run it concatenates and minifies the files in the /src directory.

If you absolutely need this functionality I would suggest you create a branch for it in your fork. Make the changes to the relevant files in /src. Then pull any changes to Leaflet.draw to your branch when they are made.