-
Notifications
You must be signed in to change notification settings - Fork 0
Refactor(#217): 화이트보드-backspace로 그래픽 요소 지우기 구현 #226
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
Refactor(#217): 화이트보드-backspace로 그래픽 요소 지우기 구현 #226
Conversation
✅ Deploy Preview for boarlog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생 많으셨습니다 😊
@@ -68,7 +68,8 @@ const CanvasSection = () => { | |||
window.addEventListener("resize", handleResize); | |||
|
|||
// delete 키를 이용해서 선택된 객체 삭제 | |||
const handleDelete = () => { | |||
const handleDelete = ({ key, code }: { key: string; code: string }) => { | |||
if (!(code === "Delete" || key === "Delete" || code === "Backspace" || key === "Backspace")) return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
의견 반영해주셔서 감사합니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
헉,..! 의견 반영 감사합니다 :)
@@ -95,6 +92,7 @@ const CanvasSection = () => { | |||
return () => { | |||
newCanvas.dispose(); | |||
window.removeEventListener("resize", handleResize); | |||
window.removeEventListener("keyup", handleDelete); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이벤트 잊지 않고 제거해주는 것 좋습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM입니다!! 로그인으로 많은 작업을 못해서... ㅠㅠㅠ 리뷰만 해드리는 것 같네요 죄송합니다 ㅠㅠ
@@ -68,7 +68,8 @@ const CanvasSection = () => { | |||
window.addEventListener("resize", handleResize); | |||
|
|||
// delete 키를 이용해서 선택된 객체 삭제 | |||
const handleDelete = () => { | |||
const handleDelete = ({ key, code }: { key: string; code: string }) => { | |||
if (!(code === "Delete" || key === "Delete" || code === "Backspace" || key === "Backspace")) return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
헉,..! 의견 반영 감사합니다 :)
작업 개요
화이트보드-backspace로 그래픽 요소 지우기 구현 close #217
작업 사항
고민한 점들(필수 X)
스크린샷(필수 X)