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.

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

[Fix] 상품 리스트 깨짐 해결 + dependency 수정 #116

Merged
merged 3 commits into from
Feb 12, 2022
Merged

Conversation

LeeMir
Copy link
Contributor

@LeeMir LeeMir commented Feb 12, 2022

작업 내용

  • 상품 리스트 깨지는 거 해결
  • 레이아웃 해결하는 김에 스피너도 가운데로 배치 완료
  • dependency 수정

전달 사항

react-scripts를 devDependency로 옮겼습니다.
CRA로 구성하면 기본적으로 dependency에 있는데 취약점이 거슬려서...
node_modules 한 번 삭제 후 yarn install 해주세용

스크린샷 (선택)

image
image

고정된 %값이 아니라 알아서 크기에 맞게 배치되도록해서 깨지지 않게 함
relative, absolute를 이용해 스피너도 가운데로 배치
개발자 도구에서 작업하다가 옮기다가 빼먹은듯
@LeeMir LeeMir added 🌞 FE Frontend 🎨 Design Design 🛠️ Refactor 리팩토링 labels Feb 12, 2022
@LeeMir LeeMir requested a review from poiu694 February 12, 2022 05:07
@LeeMir LeeMir self-assigned this Feb 12, 2022
npm audit (--production)을 조용히 하게 하기 위해서
@LeeMir LeeMir changed the title [Fix] 상품 리스트 깨짐 해결 [Fix] 상품 리스트 깨짐 해결 + dependency 수정 Feb 12, 2022
@poiu694
Copy link
Contributor

poiu694 commented Feb 12, 2022

확인했습니다!

@poiu694 poiu694 merged commit 682f7a0 into draft Feb 12, 2022
@poiu694
Copy link
Contributor

poiu694 commented Feb 12, 2022

image

왜 제 로컬은 이렇죠 ㅠㅠ...

해결해보겠습니다

@poiu694
Copy link
Contributor

poiu694 commented Feb 12, 2022

위에 이슈는 다시 해보니까 잘 해결이 되었는데,

dilanx/craco#23
facebook/create-react-app#4342

같은 얘기가 있더라고요!

간단하게 말하면 react-scriptspollyfill 을 내장하고 있어서 production단계에서 필요하다 라는 얘기인 것 같습니다!

나중에 혹시 문제 생기면 다시 바꾸면 될 것 같습니다!

@LeeMir
Copy link
Contributor Author

LeeMir commented Feb 12, 2022

@poiu694

facebook/create-react-app#11102

제가 확인했던 링큰데 요게 좀 더 최신이에요!
아무튼 개발자도 손쓸 수 없는 상황인듯하네요..!

@poiu694
Copy link
Contributor

poiu694 commented Feb 12, 2022

쉽지 않네요...ㅎㅎㅎ

이래서 다들 직접 웹팩설정하고 하나봐요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Design Design 🛠️ Refactor 리팩토링 🌞 FE Frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants