-
Notifications
You must be signed in to change notification settings - Fork 79
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.
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
exclude en-US localization from sources to allow flexibility #204
Comments
Hmmm will think on this. There are other knock on issues that this would cause. Can you elaborate on the fixes? I'm trying to not diverge from the original DateJS formats for v1. It sounds like this might end up being a later milestone fix |
When parsed at 2014-05-02 13:45:00 I get this
I have no idea where it picks up the second one. These are the ones I discovered that bother me most because they are common words. The first one is an abbreviation of yesterday, but I think the second one is a bug. I will keep a list and write them here as I find more. I can fix the first one and other abbreviations easily if I can edit the localization files, I just change the regex. Most of this is customizing it to fit my needs. E.g., I want to disable short forms by editing the regex instead of writing some crazy configuration for this :) |
wow, that "say" one is... weird. One of the features I want to keep of the current system is that en-US is the default (and always present). I think it's becoming clear that the solution I had of making the keys the en-US ones is not very maintainable right now though. Will have to rework that. |
I would use the keys from line 255 in This way adding new keys and changing old values is easily done without maintenance. |
Right, that's a feature that having the key be the original key allows with a lot of simplicity. Would have to change the logic a bunch to fix that. In general though I think it requires a rethink of how that is organized... |
It would be nice to be able to customize the English like other languages. I would like to remove the short forms for yesterday
yes
and do other fixes, but I have to edit the source files which is a maintenance nightmare.The text was updated successfully, but these errors were encountered: