-
Notifications
You must be signed in to change notification settings - Fork 23
React 김준석 sprint7 #79
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
Open
junhub98
wants to merge
44
commits into
codeit-bootcamp-frontend:Basic
Choose a base branch
from
junhub98:React-김준석-sprint7
base: Basic
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
React 김준석 sprint7 #79
junhub98
wants to merge
44
commits into
codeit-bootcamp-frontend:Basic
from
junhub98:React-김준석-sprint7
The head ref may contain hidden characters: "React-\uAE40\uC900\uC11D-sprint7"
Conversation
This file contains hidden or 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
1.css 파일이름 변경 2.color 변수이름 변경 3.스타일 속성위치 통합 및 변경
로그인 창에서 인풋요소 조건에 따른 알림문구 설정 회원가입 창에서 인풋요소 조건에 따른 알림 문구 설정 인터렉티브 JS 파일 추가
로그인 및 회원가입의 패스워드 칸 눈모양 클릭 시 암호화 여부 결정기능 추가
sprint5 시작하기에 앞서 기존 프로젝트에서 리액트를 추가하여 연결하는 작업을 진행 ROUTER 활용하여 연결 진행하였음
custom sort필터 구현 및 검색기능 구현
ul을 position:absolute로 변경하여 수정
기존 화살표 버튼에 적용 안되던 문제 해결
이미지 파일 인풋 컴포넌트 컨텐츠 입력 컴포넌트 구현
ALlItemList 안에 있던 개별아이템 출력 컴포넌트를 재사용성을 위해 분리
유지보수 및 재사용성 을 위한 컴포넌트 분리
좀 더 적절한 단어 사용
기존 usesearchParam 커스텀훅이 코드의 난잡함과 이중상태관리 문제성을 가지고 있어 개선한 커스텀훅을 추가함
상태 통합 변수 네이밍 변경 컴포넌트 네이밍 변경 코드리팩터링
검색창 단어 입력 시 600ms 뒤 검색되며 1page로 이동되도록 설계
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.
기본 요구사항
체크리스트 [기본]
상품 상세
=> favoriteCount : 하트 개수
=> images : 상품 이미지
=> tags : 상품태그
=> name : 상품 이름
=> description : 상품 설명
상품 문의 댓글
=> image : 작성자 이미지
=> nickname : 작성자 닉네임
=> content : 작성자가 남긴 문구
=> description : 상품 설명
=> updatedAt : 문의글 마지막 업데이트 시간
체크리스트 [심화]
이전 피드백 반영##
추가구현사항##
질문사항##
최근 릴리즈 된 리액트 컴파일러(자동 메모이제이션)을 사용해서 메모화를 시켜봤는데, 자식 컴포넌트들에 console.log를 추가해 콘솔창에서 리렌더링을 추적할땐 콘솔로그가 출력 되지 않아 불필요한 리렌더링이 없어진것같은데, 리액트개발자 도구로 찍어보면 리렌더링 레이아웃이 떠서 뭐가 맞는건지 모르겠습니다.
EX) pagination에서 페이지 전환 시 관계없는 다른 자식컴포넌트(BestProductsList)에 console.log를 집어 넣었을때 위와 같은 상황이 발생 됩니다.
개별상품 댓글을 입력하는 textarea에 불필요한 리렌더링을 제거하기위해 비제어 인풋 로직을 작성해봤는데, 맞게 한건지 잘모르겠습니다. 관련하여 디자인패턴이 있다면 알려주시면 감사하겠습니다.
배포링크##