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.

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

Introduce the DateFmtInfo object. #202

Open
wants to merge 52 commits into
base: development
Choose a base branch
from
Open

Conversation

ehoogerbeets
Copy link
Contributor

NOT READY TO MERGE YET.

Gives information about the date format without cluttering up the methods on the date format object itself. The methods on the date format object are now deprecated and will be removed in ilib 16.X

Checklist

  • At least one test case is included for this feature or bug fix.
  • ReleaseNotes has added or is not needed.
  • This PR has passed all of the test cases on Nodejs and Chrome Browser.
  • This PR has passed all of the test cases on QT/QML.
  • This is an API breaking change.
  • Requires a major version change.

Issue Resolved / Feature Added

Resolution

Links

ehoogerbeets and others added 30 commits December 6, 2018 11:01
Add the idea of a value property, the value of which is a
function that calculates the value of this field based on
the value of other fields. For example, the day of week is
not something that a UI typically displays in a date input
form. Instead, you pick the year, month, and day and the
day of week is determined automatically.

More work is needed for time zones and to translate stuff.
Add the idea of a value property, the value of which is a
function that calculates the value of this field based on
the value of other fields. For example, the day of week is
not something that a UI typically displays in a date input
form. Instead, you pick the year, month, and day and the
day of week is determined automatically.

More work is needed for time zones and to translate stuff.
(They don't pass though.)
- use "placeholder" instead of "template" property
- use an options param instead of separate params to the getFormatInfo
Also updated the jsdocs to reflect the options param.
Things don't work yet...
Generated from the cldr 34 data.
Gleaned from the names of the field. For Arabic script
or RTL languages or short names, just use the full name.
For everything else, use abbreviations.
Much more work needed
Still need to fix more though
@ehoogerbeets ehoogerbeets self-assigned this Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant