search

junmer / clipper-lib

Boolean operations and offsetting library in Javascript
175 stars 32 forks source link

uppercase versus lowercase x,y #4

Closed jeromew closed 7 years ago

jeromew commented 7 years ago

I understand that this lib needs points to be represented as { X:, Y: }. It seems to me that many npm modules rely on the fact that points are defined as {x:, y:} because of this specification https://github.com/colinmeinke/points.

would it make sense to change this library to use the lowercase version, or to accept both versions ?

it would avoid a systematic impedance transformation when integrating several libs.

junmer commented 7 years ago

hi @jeromew ,this repo is only fork from https://sourceforge.net/projects/jsclipper/ . please open a issue at https://sourceforge.net/p/jsclipper/tickets/ thank you

jeromew commented 7 years ago

ok thanks. I'll close it here and look on jsclipper if it was already discussed.