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.

Sign up for GitHub

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

Jump to bottom

[#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가 #1012

Merged
merged 8 commits into from
Jan 11, 2022
Merged

[#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가 #1012

merged 8 commits into from
Jan 11, 2022

Conversation

sej-dev
Copy link
Collaborator

@sej-dev sej-dev commented Jan 7, 2022 •
edited
Loading

  • Window가 여러 개 열려 있을 때 특정 Window를 클릭한 경우, 해당 Window의 z-index 값을 높여 가장 상단에 위치시키는 기능 추가
  • 기능 관련 focusable prop 추가
  • 기능 관련 문서에 예제 코드 추가

구현 내용 정리 notion:
https://www.notion.so/exem-fe/Window-z-index-esc-d6fabfb1bf5c4b178e0057adeb19e881#08b2fb2e19054f5293d006f34f43b89f

@sej-dev
[#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가
16eb52a 
################
- 해당 기능 담당하는 useClickEventForIncreaseZIndex 추가
- useEscKeydownEvent와 같이 사용하는 코드 -> getActiveWindowsOrderByZIndexAsc 함수로 추출
@sej-dev
[#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가
15d6127 
################
- 해당 기능 담당하는 useClickEventForIncreaseZIndex 추가
- increase-z-index-on-click prop 추가
- 문서에 예제 코드 추가
@sej-dev sej-dev changed the title WIP: [#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가 [#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가 Jan 10, 2022
@sej-dev
[#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가
69776db 
################
- 문서 예제 코드 수정
- 문서 z-index 범위 z-index는 [700, 750)로 수정
@sej-dev sej-dev changed the title [#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가 WIP: [#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가 Jan 10, 2022
@sej-dev
[#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가
d1e6d84 
################
- useEscKeydownEvent의 life cycle hook에 주석 추가
- useEscKeydownEvent에서 불필요한 변수 제거
- 기타 문서 예제 코드 수정
@sej-dev
[#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가
ecb45fe 
################
- useEscKeydownEvent + useClickEventForIncreaseZIndex -> useEscCloseAndFocusable
- increase-z-index prop -> focusable prop
- useEscCloseAndFocusable 로직 보강
@sej-dev sej-dev changed the title WIP: [#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가 [#1009][3.0] Window > 클릭 시 z-index 높이는 기능 추가 Jan 11, 2022
@kdeun1 kdeun1 merged commit b56c688 into 3.0 Jan 11, 2022
@kdeun1 kdeun1 deleted the 1009_window_increase_zindex branch January 11, 2022 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kdeun1 kdeun1 kdeun1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sej-dev @kdeun1