-
Notifications
You must be signed in to change notification settings - Fork 29
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
[js] Does not hide time range window when select date and time #2542
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あー、これdatepickerをえいやっと付けてからUXを特に考えていなかったやつですね。これも 👍 です。
client/static/js/events_view.js
Outdated
@@ -959,7 +959,8 @@ var EventsView = function(userProfile, options) { | |||
function setupTimeRangeFilter() { | |||
$('#begin-time').datetimepicker({ | |||
format: 'Y/m/d H:i:s', | |||
closeOnDateSelect: true, | |||
closeOnDateSelect: false, | |||
closeOnDateSelect: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
自分のコミットで大変申し訳無いんだけど,ここ同じコード2連続で書いていて間違ってます。
正確にはtirgger_view.jsにあるように,closeOnDateSelectとcloseOnTimeSelectです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😢
client/static/js/events_view.js
Outdated
@@ -970,7 +971,8 @@ var EventsView = function(userProfile, options) { | |||
|
|||
$('#end-time').datetimepicker({ | |||
format: 'Y/m/d H:i:s', | |||
closeOnDateSelect: true, | |||
closeOnDateSelect: false, | |||
closeOnDateSelect: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ココも同じ。
7dcf03b
to
7703f2b
Compare
修正しました |
#2523 時間,日付を選択してもウィンドウが閉じないようにした。