Skip to content

Commit 538856d

Browse files
committed
Use methods instead of inline expressions
1 parent f23e45b commit 538856d

File tree

2 files changed

+15
-3
lines changed

2 files changed

+15
-3
lines changed

packages/date-picker/src/panel/date-range.vue

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
@input.native="handleDateInput($event, 'min')"
3232
@change.native="handleDateChange($event, 'min')" />
3333
</span>
34-
<span class="el-date-range-picker__time-picker-wrap" v-clickoutside="() => minTimePickerVisible = false">
34+
<span class="el-date-range-picker__time-picker-wrap" v-clickoutside="handleMinTimeClose">
3535
<el-input
3636
size="small"
3737
:disabled="rangeState.selecting"
@@ -62,7 +62,7 @@
6262
@input.native="handleDateInput($event, 'max')"
6363
@change.native="handleDateChange($event, 'max')" />
6464
</span>
65-
<span class="el-date-range-picker__time-picker-wrap" v-clickoutside="() => maxTimePickerVisible = false">
65+
<span class="el-date-range-picker__time-picker-wrap" v-clickoutside="handleMaxTimeClose">
6666
<el-input
6767
size="small"
6868
:disabled="rangeState.selecting"
@@ -539,6 +539,10 @@
539539
}
540540
},
541541
542+
handleMinTimeClose() {
543+
this.minTimePickerVisible = false;
544+
},
545+
542546
handleMaxTimePick(value, visible, first) {
543547
if (this.maxDate && value) {
544548
this.maxDate = modifyTime(this.maxDate, value.getHours(), value.getMinutes(), value.getSeconds());
@@ -553,6 +557,10 @@
553557
}
554558
},
555559
560+
handleMaxTimeClose() {
561+
this.maxTimePickerVisible = false;
562+
},
563+
556564
// leftPrev*, rightNext* need to take care of `unlinkPanels`
557565
leftPrevYear() {
558566
this.leftDate = prevYear(this.leftDate);

packages/date-picker/src/panel/date.vue

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
@input="val => userInputDate = val"
2828
@change="handleVisibleDateChange" />
2929
</span>
30-
<span class="el-date-picker__editor-wrap" v-clickoutside="() => timePickerVisible = false">
30+
<span class="el-date-picker__editor-wrap" v-clickoutside="handleTimePickClose">
3131
<el-input
3232
ref="input"
3333
@focus="timePickerVisible = true"
@@ -317,6 +317,10 @@
317317
}
318318
},
319319
320+
handleTimePickClose() {
321+
this.timePickerVisible = false;
322+
},
323+
320324
handleMonthPick(month) {
321325
if (this.selectionMode === 'month') {
322326
this.date = modifyDate(this.date, this.year, month, 1);

0 commit comments

Comments
 (0)