Change message timestamp format

[Haarolean]

US locale: mm/dd/yyyy
any other locale: dd-mm-yyyy

[hyzyla]

I will try to fix that issue ✋

[Haarolean]

I will try to fix that issue ✋

hey, thanks for the interest, I've assigned the issue to you

[germanosin]

I think this should be formated locale-based. You could use Date.toLocaleDateString(). @workshur any suggestions?

[Haarolean]

I think this should be formated locale-based. You could use Date.toLocaleDateString(). @workshur any suggestions?

that's the next step (#2163). At this point it'd be nice to see dd.MM

[germanosin]

I think we could use localestring for now and then switch to configurable version

[Haarolean]

@hyzyla as far as I remember, the timestamp is being formatted on frontend. Can you format it with taking user browser's preferences into consideration?

[hyzyla]

@hyzyla as far as I remember, the timestamp is being formatted on frontend. Can you format it with taking user browser's preferences into consideration?

Yes, sure

[hyzyla]

So, after some investigation, I found that formatting to user locale is challenging, because dayjs doesn't support autodetecting locale. For that reason, I decided to just change locale to DD.MM.YYYY.

Let me know if you think that snippet is OK to use for detecting locale

[Haarolean]

So, after some investigation, I found that formatting to user locale is challenging, because dayjs doesn't support autodetecting locale. For that reason, I decided to just change locale to DD.MM.YYYY.

Let me know if you think that snippet is OK to use for detecting locale

@provectus/kafka-frontend need opinions

[workshur]

I would suggest getting rid of dayjs. We only use it in two places. So, let's create an helper function to parse the server date and convert it into a localized format.

try {
  date = new Date('2022-06-06T12:44:42.118Z');
  date.toLocaleString();
} catch ....

[hyzyla]

I would suggest getting rid of dayjs. We only use it in two places. So, let's create an helper function to parse the server date and convert it into a localized format.

try {

  date = new Date('2022-06-06T12:44:42.118Z');

  date.toLocaleString();

} catch ....

Problem with toLocaleString is that function isn't not consistent within different browsers [1]. For example, I have "uk" locale, but Firefox decide to format in en-US, while Chrome format in some weird format 18/06/2022

1. https://stackoverflow.com/a/66578542

[workshur]

From my point of view, there is no right solution. Let's wait until BE for the custom locale is ready. Let the user decide which format is more convenient for him

[Haarolean]

@hyzyla actually are you sure your browser locales are 'uk'? It seems like both are us