UiCalendar: Fix prev/next month buttons when yearRange is desc

[entioentio]

From UX point of view years should be sorted descending for some cases. When the custom yearRange is specified with descending array, the function breaks.

[JosephusPaye]

Just a small change...

[entioentio]

Perfect! I was a bit concerned about this but didint think of computed as a solution

[JosephusPaye]

Just had a thought for another solution. We can assume that the year range will always be sorted, in ascending or descending order. This will save the cost of sorting a potentially big array that will probably already be sorted in the first place.

So all we need is to pick the earliest and latest years using the first and last elements of the array.

Then this line in previousMonthDisabled() can be replaced with:

const firstYear = Math.min(this.yearRange[0], this.yearRange[this.yearRange.length - 1]);
const outsideYearRange = lastDayOfPreviousMonth.getFullYear() < firstYear;

Similarly this line in nextMonthDisabled() can be replaced with:

const lastYear = Math.max(this.yearRange[0], this.yearRange[this.yearRange.length - 1]);
const outsideYearRange = firstDayOfNextMonth.getFullYear() > lastYear;

[entioentio]

Great. I've added your min/max solution. It's elegant, cool.

[entioentio]

Let me know if you need anything more to be changed

[JosephusPaye]

I'll merge this for the next release. Thanks!

[JosephusPaye]

Thanks!

[JosephusPaye]

Released in v1.3.2