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

[#646, #663][3.0] Select, DatePicker 컴포넌트 드랍다운 로직 수정 #705

Merged
merged 3 commits into from
Nov 10, 2020

Conversation

kdeun1
Copy link
Contributor

@kdeun1 kdeun1 commented Nov 6, 2020

################

  • Select 컴포넌트의 드랍박스의 스펙 변경
    • Vue3의 teleport 로직으로 인풋박스의 움직임을 감지하지못하여 dom nested 구조로 변경
    • 기본적으로 인풋박스 위치 아래로 드랍다운됨
    • 브라우저 높이 기준으로 드랍박스가 드랍다운되었을 때, 브라우저 영역을 벗어나는 경우는 인풋박스의 상단으로 드랍업되도록 로직 수정
    • 셀렉스 컴포넌트의 여러 기능으로 인하여 드랍박스의 위치가 변경되는 경우 nextTick 이후에 렌더링 위치를 재조정하는 로직 추가
  • 컴포넌트 싱글, 멀티인 경우 레이아웃, css 개선
  • 드랍박스의 리스트에 텍스트뿐만 아니라 아이콘이 위치할 수 있도록 기능 개선
  • EVUI DOCS 설명 수정 및 아이콘 부분 추가

################
- Select 컴포넌트의 드랍박스의 스펙 변경
  - Vue3의 teleport 로직으로 인풋박스의 움직임을 감지하지못하여 dom nested 구조로 변경
  - 기본적으로 인풋박스 위치 아래로 드랍다운됨
  - 브라우저 높이 기준으로 드랍박스가 드랍다운되었을 때, 브라우저 영역을 벗어나는 경우는 인풋박스의 상단으로 드랍업되도록 로직 수정
  - 셀렉스 컴포넌트의 여러 기능으로 인하여 드랍박스의 위치가 변경되는 경우 nextTick 이후에 렌더링 위치를 재조정하는 로직 추가
- 컴포넌트 싱글, 멀티인 경우 레이아웃, css 개선
- 드랍박스의 리스트에 텍스트뿐만 아니라 아이콘이 위치할 수 있도록 기능 개선
- EVUI DOCS 설명 수정 및 아이콘 부분 추가
@kdeun1 kdeun1 changed the title [#646][3.0] Select 컴포넌트 버그 수정 및 리펙토링 [#646, #663][3.0] Select, DatePicker 컴포넌트 드랍다운 로직 수정 Nov 9, 2020
kimdoeun added 2 commits November 9, 2020 15:17
################
- DatePicker 컴포넌트의 드랍박스의 스펙 변경
  - 기본적으로 인풋박스 위치 아래로 드랍다운됨
  - 브라우저 높이 기준으로 드랍박스가 드랍다운되었을 때, 브라우저 영역을 벗어나는 경우는 인풋박스의 상단으로 드랍업되도록 로직 수정
  - 셀렉스 컴포넌트의 여러 기능으로 인하여 드랍박스의 위치가 변경되는 경우 nextTick 이후에 렌더링 위치를 재조정하는 로직 추가
- 컴포넌트 레이아웃, css 개선
- CloseAll 시에 드랍박스 보이지 않게 로직 수정
################
- DatePicker css stylelint order error 수정
@mmindy mmindy merged commit f29a430 into 3.0 Nov 10, 2020
@mmindy mmindy deleted the 646_refactoring_select branch November 10, 2020 02:36
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