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

[PLAY-489] Reinstated Flatpickr Default AM/PM Toggle #2175

Merged
merged 5 commits into from Nov 14, 2022

Conversation

nidaqg
Copy link
Contributor

@nidaqg nidaqg commented Nov 11, 2022

Screens

AM/PM toggle
Screen Shot 2022-11-11 at 11 04 44 AM

AM/PM toggle on hover
Screen Shot 2022-11-11 at 11 05 31 AM

Breaking Changes

No, see runway story for full details.

Runway Ticket URL

Runway Story

How to test this

Test in review env. Scroll down to 'Time Selection' doc example and click on calendar to show Date Picker. Make sure thev Am/PM toggle works to change the Am/PM in the display.

There are 2 Date Pickers in that example, make sure toggle works correctly on each and does not interfere with the other Date Picker's toggle (see story for details)

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY Please add the Milano label when you are ready for a review.
  • SCREENSHOT Please add a screen shot or two.
  • SPECS Please cover your changes with specs.
  • READ DOCS Please make sure you have read and understand the Playbook Release Process

@nidaqg nidaqg added bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels Nov 11, 2022
@nidaqg nidaqg self-assigned this Nov 11, 2022
@nidaqg nidaqg requested a review from a team as a code owner November 11, 2022 16:22
@app-milano app-milano bot temporarily deployed to pr2175 November 11, 2022 17:03 Inactive
@app-milano app-milano bot temporarily deployed to pr2175 November 11, 2022 19:47 Inactive
@app-milano app-milano bot temporarily deployed to pr2175 November 11, 2022 21:20 Inactive
@jasperfurniss jasperfurniss added the Ready for Release merged to master, ready for a versioned released label Nov 14, 2022
@nidaqg nidaqg merged commit 456db74 into master Nov 14, 2022
@nidaqg nidaqg deleted the PLAY-489-Datepicker-ampm-toggle branch November 14, 2022 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants