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

[#1350] DatePicker > dateRange, dateTimeRange 모드 > 사용자 입력 가능 옵션 추가 #1351

Merged
merged 5 commits into from
Feb 15, 2023

Conversation

h2yeon
Copy link
Collaborator

@h2yeon h2yeon commented Feb 14, 2023

이슈

date, dateTime 모드에선 input에서 바로 입력하여 변경 가능하지만 dateRange, dateTimeRange 모드에선 입력할 수 없음
range 모드에서도 입력 가능할 수 있도록 개선 필요

기대 효과

range 모드도 calendar를 클릭하지 않고 input태그에 입력하여 변경할 수 있음

작업 내용

  1. enableTextInput (텍스트 입력 활성화) 옵션 추가

    • 현재 프로젝트에 적용된 datepicker가 있으므로 적용된 프로젝트의 수정 범위가 적도록 하기 위해 옵션 추가
    • 기본값: false
    • enableTextInput : true일 경우 입력 할 수 없도록 적용(date, dateTime 모드도 포함)
    • enableTextInput : false일 경우 입력 가능
    • 참고 dateMulti 는 항상 readonly: true
    • 입력할 때 disabledDate 함수에 포함되거나 invalid한 날짜 입력했을 때 변경되기 전의 날짜로 변경됨

    datepicker_readonly

  2. 그 외 이슈 또는 개선

    • 함수 안에서만 사용하는 변수들 computed 제거

    • range모드에서 clickDate했을 경우 클릭한 calendar의 page만 업데이트

    • from 달력의 년,월 페이지가 to 달력의 페이지를 넘어서지 못하는 제한 제거(반대도 동일)

      • 이전 from날짜에서 to날짜로 연속된 날짜 표시할 때 사용되었지만 현재 이 기능이 삭제되어 제거
      • 추후 다시 기능을 넣을 때 사용 예정
      • AS-IS
        calendar_bug
      • TO-BE
        calendar_bug_fix
    • range 모드에서 현재 달력에서 이전 달력의 날짜가 선택되었을 때 표시되지 않는 버그 수정

      • AS-IS
        datepicker_bug
      • TO-BE
        datepicker_bug_fix

@h2yeon h2yeon marked this pull request as draft February 14, 2023 05:35
@h2yeon h2yeon marked this pull request as ready for review February 14, 2023 05:57
#######################
enableTextInput 디폴트값 false로 수정
@kdeun1 kdeun1 merged commit 04fe947 into 3.0 Feb 15, 2023
@kdeun1 kdeun1 deleted the 1350_datepicker_range_mode_input branch February 15, 2023 00:53
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.

3 participants