Back to Groups
한국어

프론트엔드 PR 리뷰

검색 응답 순서, 폼 제출, 키보드 포커스처럼 프론트엔드 PR에서 반복해서 짚는 동작을 위한 리뷰 문구입니다. 대부분 코드 옆에 바로 남길 수 있고, 재현 조건을 적는 문구에는 입력란이 있습니다. 보내기 전에 대괄호 안의 내용을 실제 정보로 바꾸세요.

workflowskoreanformscode-reviewkorean-frontend-reviewaccessibilityfrontend
SE
seojin.dev
0likes10snippets

Try it here

Loading snippet…

Your typing stays in this tab. Nothing is saved or sent.
Snippets (10)
;kfe-empty
빈 결과와 오류 구분
조회 실패가 ‘결과 없음’으로 보이면 사용자가 검색어를 바꾸게 됩니다. 오류에는 재시도를, 정상 응답이 빈 경우에는 검색 조건 안내를 보여주면 좋겠습니다.
;kfe-race
검색 응답 순서 확인
검색어를 빠르게 바꿨을 때 이전 요청의 응답이 나중에 도착해도 최신 결과를 덮어쓰지 않아야 합니다. 응답 순서를 바꾼 테스트도 추가해 주세요.
;kfe-submit
제출 중 중복 클릭 방지
저장 중에는 같은 요청이 다시 나가지 않게 해 주세요. 실패했을 때 입력값은 남기고 제출 버튼을 다시 활성화해야 사용자가 그대로 재시도할 수 있습니다.
;kfe-overflow
긴 문자열 너비 검수
공백 없는 긴 URL과 긴 이름을 넣어 좁은 화면에서도 확인해 볼까요? 이 영역 때문에 페이지 전체에 가로 스크롤이 생기지 않도록 줄바꿈이나 영역 내부 스크롤 처리가 필요합니다.
;kfe-focus
모달 키보드 포커스
모달을 닫으면 포커스가 열기 버튼으로 돌아오나요? 키보드로 열고 닫는 경로도 확인해 주세요.
;kfe-cleanup
구독과 이벤트 정리
이 effect에서 등록한 리스너나 구독은 cleanup에서 해제하고 있나요? 화면을 나갔다 다시 들어온 뒤에도 콜백이 한 번씩만 실행되는지 확인이 필요합니다.
;kfe-split
기능 수정과 정리 분리
파일 이동과 동작 수정을 별도 커밋으로 나눠주실 수 있을까요? 버그를 고친 코드부터 리뷰하고 싶습니다.
;kfe-space
이미지 로딩 공간 확보
이미지를 받기 전에도 크기나 비율만큼 공간을 잡아두면 어떨까요? 느린 네트워크에서 이미지가 뜨며 아래 버튼이 밀리는지도 함께 확인해 주세요.
;kfe-repro
화면 깨짐 재현 조건
[브라우저·화면 너비·확대 배율]에서 [깨지는 위치와 재현 동작]을 확인했습니다. 같은 조건으로 확인 부탁드립니다. 수정 후에는 인접한 화면 너비에서도 레이아웃이 유지되는지 봐 주
;kfe-keys
목록 항목 식별자 리뷰
이 목록은 삭제나 정렬로 순서가 바뀔 수 있으니 인덱스 대신 항목의 고유 ID를 key로 쓰면 좋겠습니다. 순서를 바꾼 뒤 입력값과 선택 상태가 원래 항목에 남는지도 봐 주세요.