-
Notifications
You must be signed in to change notification settings - Fork 11
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
[Feature]: useResizeObserver dimension 추가 #460
Comments
좋은 훅이라고 생각합니다!! 👍 |
@Sangminnn 해당 훅에 대해 고민을 조금 해봤습니다! 개인적으로 훅을 신규로 추가하기보다 useResizeObserver에 내에서 dimension 상태를 추가하면 좋을 것 같은데 어떻게 생각하실까요!? useResizeObserver 훅이 ResizeObserver를 사용하기 때문에 현재의 return { ref, contentRect, dimension } |
추가적으로 dimension은 특정 요소의 아래 모던 자바스크립트 문서의 경우에는 좌표를 coordinates로 사용하네요! |
@ssi02014 좋은 방향으로 같이 고민해주셔서 감사합니다! 사실 기존에 진행하려던 방식 역시 useResizeObserver로부터 파생된 훅을 만들려고 했었고, 그러다보니 이미 기존의 방향에서도contentRect도 반환해주고있다보니 현 ResizeObserver의 입장에서 해당 값(dimensions)을 제공해주는 방향으로도 괜찮다면 저는 좋은 방향이라고 생각합니다! ( 최초 ResizeObserver를 제가 만들지 않았기때문에 혹시 기존 의도를 해치는건 아닐까 생각이 들었었네요! ) 추가적으로 제안해주신 네이밍 변경건 역시 저도 최초에는 어떤 단어로 표현하는 것이 좋을지 고민하던 중 클로드와 이야기하며 받은 의견을 채택했었는데요! 아래에서 실제 단어에 대한 의미와 제안해주신 것을 종합해보았을때 클로드가 제안해준 dimensions 역시도 좋은 표현이나, 비교적 영어적 관점으로 느껴지기도해서 한글로 해석한 상태에서의 의미를 본다면 저도 제안해주신 두 단어가 더 좋아보입니다! 👍 제안해주신 두 단어 중에서는 |
@Sangminnn 아! 정정하겠습니다. 현재 contentRect는 실제로 getBoudingClientRect와 동일한 데이터를 다루는 것으로 확인됐습니다.
https://developer.mozilla.org/ko/docs/Web/API/Element/getBoundingClientRect#%EA%B0%92
사실상 ContentRect는 요소의 크기를 포함한 viewport와 관련된 좌표 데이터를 이미 다루고 있기 때문에 해당 좌표에서 다루지 않는 값들을 추가 상태로 관리되면 좋을 것 같은데요! .예를 들어 다음과 같습니다 .
추가적으로, offsetHeigth/Width, clientHeight/Width가 getBoudingClientRect에서 다루는 width/height와 동일한 경우도 있지만, 100% 동일하지 않기 때문에 따로 관리되는게 적절하다고 생각합니다. 결론결론적으로 위 |
@ssi02014 좋은 레퍼런스와 함께 다각도로 고민해주셔서 감사합니다! 🙇 위의 내용을 확인해보니 dimension도 충분히 좋은 표현으로 보이네요! 👍 괜찮으시다면 기존의 resizeObserver에 dimension을 추가하여 반환할 수 있도록 제가 작업해도 될까요 ?? 😄 |
@Sangminnn 좋습니다!!! 감사합니다👍👍 |
Package Scope
Overview
타겟 엘리먼트의 dimension 정보에 대해 접근이 가능한 훅입니다.
The text was updated successfully, but these errors were encountered: