이미지 압축 패키지를 Browser Image Compression에서 upload-images-converter로 변경 #684
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
upload-image-converter은 이미지를 비율대로가 아닌 지정한 값으로 자르는 옵션만 설정할 수 있어서 직접 비율대로 자르기 위해 해당 함수를 구현
|
⚡️ Lighthouse report!
|
inyeong-kang
approved these changes
Sep 22, 2023
| @@ -7,6 +7,19 @@ import { handlers } from './src/mocks/handlers'; | |||
|
|
|||
| dotenv.config({ path: './.env.test' }); | |||
|
|
|||
| /** | |||
| * 이 코드가 없다면 jest에서 upload-images-converter 패키지에 관한 에러가 발생합니다. | |||
There was a problem hiding this comment.
jest의 신, jest의 자랑, jest의 빛
우스
| https://github.com/nrwl/nx/issues/7844#issuecomment-1220559108 | ||
| */ | ||
| jest.mock('upload-images-converter', () => ({ | ||
| __esModule: true, |
There was a problem hiding this comment.
| height, | ||
| }); | ||
|
|
||
| const outputWebpFile = new File([compressedBlob[0]], `${Date.now().toString()}.webp`); |
There was a problem hiding this comment.
[compressedBlob[0] 은 무슨 값인가요??
There was a problem hiding this comment.
이미지 파일입니다 패키지를 바꾸면서 기존에는 이미지 파일을 사용하다가 바꾼 패키지에서는 이미지 파일 배열을 반환해서 [0]으로 사용하게 되었어요
| const maxSize = Math.max(originWidth, originHeight); | ||
|
|
||
| if (maxSize <= maxWidthOrHeight) { | ||
| return { width: originWidth, height: originHeight }; |
There was a problem hiding this comment.
오 이미지 원본의 너비/높이가 너비/높이 제한값 이하이면, 원본의 너비/높이를 그대로 사용하는 로직 맞나요?
|
|
||
| if (originWidth === maxSize) { | ||
| const width = maxWidthOrHeight; | ||
| const height = Number(((originHeight * maxWidthOrHeight) / originWidth).toFixed(1)); |
There was a problem hiding this comment.
이거 혹시 무슨 계산인지 한번에 안 와닿아서 그러는데 예시로 설명해주실 수 있나요..?🤔😂
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔥 연관 이슈
close: #683
📝 작업 요약
⏰ 소요 시간
1시간
🔎 작업 상세 설명
처리하는 시간
네트워크 부하 (자바스크립트 코드로 처리하기 때문에 네트워크 속도와는 관련이 없음)
처리 속도 : 6ms
맥북 에어(cpu 6배 부하)
처리 속도: 20ms
웹팩 번들 크기 비교
####Browser Image Compression
크기: 55kb
upload-images-converter
크기: 5kb
🌟 논의 사항
크루들과 이야기 해보고 싶은 부분을 적어주세요.