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

Date picker for Date and DateTime field input #4981

Merged
merged 17 commits into from
Apr 16, 2024
Merged

Date picker for Date and DateTime field input #4981

merged 17 commits into from
Apr 16, 2024

Conversation

lucasbordeau
Copy link
Contributor

@lucasbordeau lucasbordeau commented Apr 15, 2024

  • Implemented correct mask for Date and DateTime field in InternalDatePicker
  • Use only keyDown event and click outside in InternalDatePicker and DateInput
  • Refactored InternalDatePicker UI to have month and year displayed
  • Fixed bug and synchronized date value between the different inputs that can change it

gitstart-twenty and others added 12 commits April 10, 2024 10:03
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: Matheus <matheus_benini@hotmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: Matheus <matheus_benini@hotmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: Matheus <matheus_benini@hotmail.com>
Copy link

github-actions bot commented Apr 15, 2024

TODOs/FIXMEs:

  • // TODO: implement keyboard events here: packages/twenty-front/src/modules/ui/input/components/internal/date/components/InternalDatePicker.tsx

Generated by 🚫 dangerJS against 0bd0aa2

@lucasbordeau lucasbordeau changed the title Twnty 4376 Date picker for Date and DateTime field input Apr 15, 2024
@lucasbordeau
Copy link
Contributor Author

@charlesBochet I'm fixing some minor bugs

@lucasbordeau
Copy link
Contributor Author

@charlesBochet Ready

Copy link
Member

@Weiko Weiko left a comment

Choose a reason for hiding this comment

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

LGTM but you probably want a second 👀!

@lucasbordeau
Copy link
Contributor Author

I'll add the time in the date time display component too

@lucasbordeau lucasbordeau merged commit 19a3be7 into main Apr 16, 2024
8 checks passed
@lucasbordeau lucasbordeau deleted the TWNTY-4376 branch April 16, 2024 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants