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

1189 - IdsTimepicker Fix formatting when day period goes first in the time format #1254

Merged
merged 9 commits into from
Apr 25, 2023

Conversation

andriivarhanov
Copy link
Contributor

Explain the details for making this change. What existing problem does the pull request solve?
This PR adds adjustments to the time picker and formatDate function to be able to format day period correctly

Related github/jira issue (required):
Closes #1189

Steps necessary to review your pull request (required):

  • pull and build
  • go to http://localhost:4300/ids-time-picker/example.html
  • change the locale to ko-KR in the browser's console document.querySelector('ids-container').locale = 'ko-KR'
  • select a time
  • see the day period goes before the numeric time according to the format a h:mm
  • select a time in the Time Picker - required example
  • see the required error doesn't trigger
  • test zh-Hans locale
  • test other locales

Included in this Pull Request:
- [ ] An e2e or functional test for the bug or feature.

  • A note to the change log.

* main: (75 commits)
  fix menu button icon
  Hide menu items in IdsMasthead
  Revert aria-description change, ignore in tests
  Update snapshots/fix test conditions
  Fix scrollable IdsPopup-related aXe violations
  Add fine grain control over flip logic in IdsPopup to account for label sizes
  Add fine grain control over flip logic in IdsPopup to account for label sizes
  Fix bad spacing when IdsDatePicker popup changes axes
  Fix input field cutoff issue in some layouts
  Fix IdsPager input field size
  Remove unused code
  Remove unavailable Popup example
  Allow IdsPopupMenu resize to fix/unfix menu height
  Add changelog note for #1205
  Fix lint issues
  Fix docs/tests
  Fix submenu navigation issue
  Normalize IdsToolbarMoreActions menu structure for light DOM, fix menu behavior
  Remove extra tweak
  Add IdsPopupMenu scrolling demo, better sharing of maxHeight property
  ...
* main: (63 commits)
  Wizard contrast test
  Small change in wizard test
  Skip problematic tests
  Add fix for order setting
  Fix import issue in examples
  Fix some examples that were missed on the first pass
  Fix virtual scroll and wizard tests
  Fix draggable example
  Fix color picker lint error
  Fix readme
  Fix lint errors
  Minor update to readme
  Update button setting
  Remove old grid directory
  Update more examples
  More example fixes
  Add more e2e tests
  Add align settings to grid cell items
  Use attributes instead classes on the host element
  Minor adjustment to documentation
  ...
Copy link
Collaborator

@jdilag90 jdilag90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look good!

@tmcconechy tmcconechy merged commit 6aeaba5 into main Apr 25, 2023
1 check passed
@tmcconechy tmcconechy deleted the 1189-ids-timepicker-formatting branch April 25, 2023 18:53
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.

IdsTimePicker: Invalid value when day period goes first in the time format
4 participants