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/set default end time #994

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

mathsuky
Copy link
Contributor

イベント作成における日時のautocompleteのロジックを改善しました。
具体的には,デフォルトでは終了日時が開始日時+0日1時間となるように自動で値を埋めるようにしました。また,その後ユーザーが終了日時の方をいじったあとに開始日時を変更した場合,変更する直前の終了日時と開始日時の差を保つようにして終了日時を自動で変更するようにしました。

既知の問題として,開始日→開始時刻以外の順番で入力されるとautocompleteがうまくいかないというものがあります。これについては,何らかの方法によって日付の初期値を設定できると良いかと思っています。
(今日の日付を入れる?あるいはカレンダーから日付を選択してイベントを入れられるようにする?)

Copy link
Member

@itt828 itt828 left a comment

Choose a reason for hiding this comment

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

PRありがとうございます!
初期値の問題については、今の日時に設定してしまってもいいと思います。

値の更新毎にautocompleteを発火させられるので、@blurのかわりに@inputを使うのも個人的にはアリだと思いました。これは変えなくてもよいです。

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.

2 participants