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

feat: Multiple language support and 24-hour format support #82

Merged
merged 8 commits into from
Jun 19, 2021

Conversation

MasterHiei
Copy link
Contributor

@MasterHiei MasterHiei commented Jun 15, 2021

Closes: #61

  • Add language support for Chinese/Japanese
  • Add a theming parameter for toggle 12-hour/24-hour format
  • Remove unnecessary null checks to avoid app crashes
  • Add a parameter to handle event happens when ther user canceled drag gesture
    This change will fix an issue: user cannot remove the created time overlay if drag gesture canceled without any movement.

Checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Copy link
Owner

@JonasWanke JonasWanke left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! I've added some minor comments, but overall, it looks great :)

lib/src/components/multi_date_content.dart Outdated Show resolved Hide resolved
lib/src/components/time_indicator.dart Outdated Show resolved Hide resolved
example/lib/main.dart Outdated Show resolved Hide resolved
@MasterHiei
Copy link
Contributor Author

@JonasWanke Hi, Thanks for the review :) I pushed the fixes on 7ee1a28, could you check it please?

@JonasWanke
Copy link
Owner

JonasWanke commented Jun 19, 2021

Thanks :) I'll let you know when I publish this to pub.dev, which I'll hopefully get around to in the next few days (after fixing a few things)

@JonasWanke JonasWanke merged commit 13ce1ee into JonasWanke:main Jun 19, 2021
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.

allow formatting using something other than 24 hour time?
2 participants