Current model is unable to express some languages

[balazsbotond]

I really like small and focused libraries like this one!

I tried to add Hungarian but the words for the time units ("year", "month", etc.) need to be different depending on whether the date is in the past or the future. Looks like disTime doesn't support that yet.

I don't know what your goals are with this library but if you want to support as many languages as possible, I would suggest making each language a function instead of a config object, like this:

'en': function (majorValue, majorUnit, minorValue, minorUnit, isFuture) { ... }

A couple of examples to show how I think this could work:

Example	majorValue	majorUnit	minorValue	minorUnit	isFuture
4 years and 3 months ago	4	'year'	3	'month'	false
4 years ago	4	'year'	undefined	undefined	false
in 4 years	4	'year'	undefined	undefined	true
in 4 years and 3 months	4	'year'	3	'month'	true

There could be a default implementation for the majority of languages which could accept a config object similar to the one currently used:

'en': defaultTranslator({
  words: {
    'preAgo': '',
    'postAgo': 'ago',
    'and': 'and',
    'inFuture': 'in',
    'postInFuture': ''
  },
  year: ['year', 'years'],
  month: ['month', 'months'],
  week: ['week', 'weeks'],
  day: ['day', 'days'],
  hour: ['hour', 'hours'],
  minute: ['minute', 'minutes'],
  second: ['second', 'seconds']
})

And weird languages like Hungarian, Russian and Ukrainian could use their own custom implementation (this would also remove the need for the mode key in the config object).

What do you think? If you like this solution I can put up a PR with the necessary changes.

[SimonWaldherr]

sounds good to me 👍

[balazsbotond]

sounds good to me 👍

Great, I'll start working on it and keep you updated.