nhn / tui.date-picker

Component that selects specific date.
https://nhn.github.io/tui.date-picker/latest/
MIT License
94 stars 30 forks source link

tui.DatePicker.createRangePicker does not render a timepicker #49

Closed honggukim0925 closed 5 years ago

honggukim0925 commented 5 years ago

Version

date-picker 4.0

Development Environment

linux ubuntu 18.04

Current Behavior

안녕하세요 아래의 방법으로 RangePicker를 사용하고 있습니다만 api문서에는 timePicker =true 라고 하면 time picker가 아래에 나와야하는데 아무리해도 안나오네요.. 어떻게 하면 좋을까요?

good afternoon I using the "RangePicker" in the following In api document, if i use "timePicker =true", the time picker should be shown at the "rangePicker" I can't make it. How shall I do it?

// Write example code
var today = new Date();//tui.DatePicker.createRangePicker
var picker = tui.DatePicker.createRangePicker({
    usageStatistics: false,
    startpicker: {
        date: today,
        input: '#startpicker-input',
        container: '#startpicker-container'

    },
    endpicker: {    
        date: today,
        input: '#endpicker-input',
        container: '#endpicker-container'

    },
    timePicker:true,
format: 'yyyy-MM-dd HH:mm A',   
    selectableRanges: [
        [new Date(today.getFullYear() - 1, today.getMonth(), today.getDate()) , new Date(today.getFullYear() + 1, today.getMonth(), today.getDate())]
    ]
});

Expected Behavior

heenakwag commented 5 years ago

@honggukim0925

안녕하세요. 리포트해주신 버그 해결하여 v4.0.2(API)로 배포하였습니다. 한 가지 주의하실 점은, 현재 DateRangePicker에서 시간 범위가 고려되지 않고 있습니다. 해당 이슈는 10월 중으로 해결될 예정이며 #51 를 참고해주시길 바랍니다. 감사합니다 :)

Thank you for reporting a bug :) I resolved this issue and released v4.0.2(API) including this patch. The one thing you need to know is DateRangePicker does not consider a time range. This issue will be solved by October. Please refer to #51.

honggukim0925 commented 5 years ago

빠른 피드백 감사 드립니다. Thank you for your quick feedback