Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Date formats #118

Open
siatstcolumb opened this issue Jan 7, 2018 · 0 comments
Open

Date formats #118

siatstcolumb opened this issue Jan 7, 2018 · 0 comments

Comments

@siatstcolumb
Copy link

siatstcolumb commented Jan 7, 2018 •
edited
Loading

Suggestion: make a note in the docs that the moment.js short date format specifier differs from the .NET date format:

  • moment.js - DD/MM/YYYY
  • .NET - dd/MM/yyyy

I used System.Globalization.CultureInfo.CurrentUICulture to supply the data-dateformat value in the table header, and it broke bootstrap-sortable's date sorting. Fixed by applying .ToUpper() to the date format string returned by CurrentUICulture.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@siatstcolumb