Feat: 일정 만들기 페이지 #40
Feat: 일정 만들기 페이지 #40
Conversation
|
추가로, 컴플릭트도 발생하고있어서 develop pull 한번 받고 충돌관리 한번 해주시면 좋겠습니다! |
| return ( | ||
| <> | ||
| <div className="flex flex-nowrap gap-[8px] w-full"> | ||
| {content.map((_content, index) => ( |
There was a problem hiding this comment.
[Lv1] {content.map((_content, index) => ( 이 부분에서 content 배열에서 받은 각 요소를 _content 라고 네이밍 하셨는데, 이 부분은 요소의 이름은 배열의 변수명 앞에 _ 를 붙인다던지 혹시 따로 컨벤션이 있는걸까요!? 잘 몰라서 여쭤봅니다!
There was a problem hiding this comment.
저도 컨벤션이 있다거나 유래는 몰랐는데 이번에 찾아보니까 과거에 C, C++에서 사용하던 스코프 구분을 위한 표기법이라 합니다!
지역변수를 구분짓고 싶을 때 사용하는데 보통 변수의 네이밍이 겹칠 때 사용합니다!
// 공통 input이라 가정
const [value, setValue] = useState("")
const onChange = (_value: string) => {
setValue(_value)
}간단한 예시로 이런게 있습니다!
There was a problem hiding this comment.
추가로 첨언 드리자면, js에서 변수명에 _를 붙이는 경우가 하나 더 있습니다! 이전에는 클래스의 형태로 컴포넌트를 구성했는데, 내부 private variable이라는것을 명시하기 위해 변수명 앞에 _ 를 붙이곤 했습니다. (물론 접근 자체는 가능하기때문에 암묵적인 표기에 가까웠습니다). 요즘은 클래스로 사용하지않기 때문에 굳이 사용하진않지만 private를 명시하기위해 종종 사용하기도 하는것같아요.
진욱님이 말씀하신 의도로도 사용됩니다! 그래서 언더바가 들어가는 경우는 보통 이 두가지로 유추하곤 합니다. 이외에 사용한다면 오히려 의도를 알수없어서 헷갈리기때문에 코멘트가 달릴수도있으니 참고해주세요!
|
요청사항 수정 완료했습니다! 확인 한번씩 부탁드릴게요! |
There was a problem hiding this comment.
이제 보니 RemainChar의 title이 input으로 받는다면 state를 굳이 사용하지않아도될것같습니다!
interface RemainCharProps {
title: string;
}
const LENGTH_LIMIT = 40; // PlanTitleInput 에 두고 input maxLength에 같이 쓰면 관리포인트가 줄어들어서 좋을거같습니다
const RemainChar = ({ title }: RemainCharProps) => {
const remainChar = LENGTH_LIMIT - title.length;
return (
<div className="text-[12px] text-[#8D8D8D] font-medium -tracking-[0.01em]">
{remainChar < 0 ? 0 : remainChar}자 남음
</div>
);
};
export default RemainChar;쓰다보니 추가로 생각이 드는데, input에 입력될때마다 렌더링이 될테니 아예 RemainChar를 빼고 PlanTitleInput에 합쳐도 좋을거같습니다! 이건 급한 수정사항은 아니니 지금 반영해주셔도되고, 다음 PR에 반영하셔도됩니다!
There was a problem hiding this comment.
[Lv1] 선택된 이미지를 렌더링하는 부분은 기획에 있을까요?? TODO로 선택한 이미지도 렌더링하도록 개선 부탁드립니다!
There was a problem hiding this comment.
[Lv2] 리코일 이 값은 어디에서 사용되는건가요?? 찾아봤을때 호출하는곳은 안보였는데, atom이 PlanTitle 형식이라면, create-schedule/constants 에 있는 TITLE과 SUBTITLE만 import해서 사용해도될거같습니다!
There was a problem hiding this comment.
원래 Title 컴포넌트 안에서 제목을 관리하려 했었으나 작업을 진행하면서 굳이 그럴 필요는 없다고 생각이 들어 만들어놓고 사용을 하지 않았습니다! 삭제하도록 하겠습니다
변경사항
테스트
Notice
TODO