1199126 - DateTime column sorting displaying wrong order in Storage Inspector

Status: RESOLVED DUPLICATE of bug 1073967

(DevTools :: Storage Inspector, defect)

Description

[magicp]

Attachment: datetime-column.png

User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:43.0) Gecko/20100101 Firefox/43.0
Build ID: 20150826030211

Steps to reproduce:

1. Open Storage Inspector
2. Sort on DateTime column (e.g. Expires on, Last accessed on)


Actual results:

DateTime column sorting displaying wrong order, because DateTime format is "yyyy/m/d H:mm:ss". It is not windows setting.


Expected results:

DateTime format should be "yyyy/mm/dd HH:mm:ss"

Comment 1

[Tom Tromey :tromey]

I'm not sure changing the formatting is best.
It's using toLocaleString to format the date, which seems best.

Given that, the issue seems to be that TableWidget doesn't allow sorting
columns based on the model -- only on the displayed content.

TableWidget refactoring is going on in bug 1201742.

Comment 2

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Making the cookie table editable has meant switching to the proper cookie date format Date.toISOString():
Mon, 02 Jun 2025 12:37:37 GMT

Of course the sorting is still nuked... I should point out that the current XUL-based widget is not good to work with and it's warts have taken by far the majority of my time working on making things editable.

Comment 3

[Brian Grinstead [:bgrins]]

(In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #2)
> Of course the sorting is still nuked... I should point out that the current
> XUL-based widget is not good to work with and it's warts have taken by far
> the majority of my time working on making things editable.

I'm not sure if you've been needing to modify that widget, but just a heads up that there's a tree/table widget in flight at bug 1247065 that should be able to handle this use case.