-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#34] 수동 저장 -> 자동 저장으로 변경 #38
Conversation
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.
조금 더 깊은 개념인 web worker까지 학습해서 잘 정리해주신 점이 인상깊네요!
CS적으로 많은 학습을 하신것 같아서 좋네요!
수고하셨습니다 ❤️
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.
자동저장 기능을 통해 사용성을 향상시킨 것 되게 좋네요 ㅎㅎ html2canvas 관련 문제 해결하시느라 고생하셨습니다 <3
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.
기존 코드의 문제점 파악하고 개선하시느라 고생하셨습니다<3
이미지 저장 실패하셨음에도 동작 영상에서는 제대로 동작하는 것 같아 신기하네용👀
충돌하는 파일들이 꽤 많아서 해결 후 머지 부탁드립니다:)
🔗 Linked Issue (이슈)
#34
🙋 Summary (요약)
bandicam.2025-01-24.15-22-45-372.mp4
😎 Description (변경사항)
전역 상태 변경에 따라 저장 로직 호출
html2canvas ignoreElements 옴션을 사용해 불필요한 스타일 계산 축소 + offscreenCanvas와 web worker를 사용해 main thread 블로킹 감소
🔥 Trouble Shooting (해결된 문제 및 해결 과정)
글 작성중입니다.
🤔 Open Problem (미해결된 문제 혹은 고민사항)