Compare method should handle null | undefined PlainDate

[mukesh-msci]

I am using Temporal.PlainDate in column of Grid in my project, which may have null or undefined values. I am using Temporal.PlainDate.compare method as comparator for sorting rows which is failing while comparing with null and undefined values.

[ptomato]

Thanks for the feedback! This is not something that we're able to change at this point, but you could propose it for a future iteration at https://github.com/js-temporal/proposal-temporal-v2.

That said, I would personally suggest a different way than making compare() handle null out of the box, because undoubtedly some applications will need the nulls sorted first, some will need the nulls sorted last, some will need to handle null but not undefined, etc. So it seems kind of arbitrary if we were to choose one or the other for all JavaScript developers.

Here's an example snippet that you could use to sort nulls first:

list.sort((a, b) => {
  if (a === null) return -1;
  if (b === null) return 1;
  return Temporal.PlainDate.compare(a, b);
});