[UX] Date in Overview section doesn't show date

[chetsangnil]

"Date" in overview section only shows time. Please take a look into this.

[philliphoff]

@chetsangnil What should be shown? (E.g. just the date, date and time, long or short form of date/time, etc.)

[chetsangnil]

Date and time yep. I remember we used to have this before as well?

[philliphoff]

Yes, but there's a lot of ways we can display a date/time (e.g. long form with day of week, short form, do we include the year, etc.).

[chetsangnil]

I tried to put both date and time in the mock and it becomes quite long in the overview section. Therefore, we can put only "Date" in the overview section since we already show time in the request list.

You can use the first example of "Long date" style according to this w3school link below: http://www.w3schools.com/js/js_date_formats.asp

Ex. Mar 25 2015

The reason I picked this format is because, US dating format is abit different than other places in the world where month comes before date. I thought that using the long date would eliminate that confusion.

[nikmd23]

I second the long date format, though we may want to consider adding the time as a separate entry in the overview section.

I know it's already in the History, but if the user has scrolled away, then they lose that context.

@chetsangnil, if you agree with that, might I suggest we use the time format from the .toLocaleTimeString() method? It will spit out the time in a format that matches the culture of the user. It also has pretty great support, though we should probably heed the performance warning on it.

[chetsangnil]

That sounds perfect Nik. Let's do that! Here's an updated screen shot for this issue:

[avanderhoorn]

Ok I've updated things so that any times we show are localized using Date.prototype.toLocaleTimeString() and added the date time overview section and format using Date.prototype.toLocaleDateString().