venn.js icon indicating copy to clipboard operation
venn.js copied to clipboard

Use of British English in library

Open JelleBlaauw opened this issue 7 years ago • 3 comments

I love this library very much, but one thing that bothers me is the use of British English in the code, while American English is standardized in coding.

For example, the word colour is used, instead of the American English color:

colourMap = {},

Another example, where inconsistency between the two occurs:

// compute the center of some circles by maximizing the margin of
// the center point relative to the circles (interior) after subtracting
// nearby circles (exterior)
function computeTextCentre(interior, exterior) {}

If HTML, CSS, JavaScript, and basically every other language use American English, then why does this library not follow that standard?

I understand changing this would possibly introduce compatibility issues, but the longer you wait the worse the impact will be.

JelleBlaauw avatar Aug 17 '17 08:08 JelleBlaauw

I'm actually using Canadian English spelling - in British English function names like normalizeSolution would be normaliseSolution instead. Having said that, Canadian English isn't exactly well known and I can understand how you might have gotten it confused =)

I agree about being consistent with CSS and HTML for the public API though - it could be a source of friction for most people using this library. I'm on vacation until September, I'll see about adding support for Americans when I get back.

benfred avatar Aug 23 '17 21:08 benfred

I apologize for getting Canadian English and British English mixed up. I'm not a native English speaker myself. I personally prefer British English over American English on most occasions, however, I feel that - in programming - American English should always be used, for standardization purposes, obviously.

Anyway, thanks for getting back to me. Enjoy your vacation.

JelleBlaauw avatar Aug 23 '17 22:08 JelleBlaauw

in my fork at https://github.com/upsetjs/venn.js. I added support for both: https://github.com/upsetjs/venn.js/blob/eb3e1c2c6a227a485af79a932db056f867e2e3b7/src/index.d.ts#L170-L175

sgratzl avatar Jun 18 '20 17:06 sgratzl