Skip to content

Conversation

@kangtayie
Copy link
Contributor

1. 무엇을?

  • React Router를 사용하여 Todo, 로그인, 회원가입 페이지를 분리
  • 루트 페이지에서 로그인 ,회원가입 버튼 제공
  • 로그인 페이지에서 회원가입 페이지로 이동할 수 있는 버튼 제공

2. 스크린샷

image
image
image

@kangtayie kangtayie self-assigned this May 7, 2025
Copy link
Member

@yezzan9 yezzan9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

5주차 과제도 수고하셨습니다!
전체적으로 잘 구현해주셨네요🙌 다만, prettier 설정이 계속 안 되어있는 것 같아 해당 부분만 확인해주시면 좋을 것 같아요!

Comment on lines 33 to 56
const AddTodo = ({ onAddTodo }) => {
const [inputValue, setInputValue] = useState("");

const handleInputChange = (e) => {
setInputValue(e.target.value);
};

const handleAddClick = () => {
onAddTodo(inputValue);
setInputValue("");
};

return(
<AddTodoContainer>
<StyledText>What needs to be done?</StyledText>
<StyledInput type="text" placeholder="Add a task" />
<StyledButton>Add</StyledButton>
<StyledInput
type="text"
placeholder="Add a task"
value={inputValue}
onChange={handleInputChange}
/>
<StyledButton onClick={handleAddClick}>Add</StyledButton>
</AddTodoContainer>
)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분 들여쓰기 2칸으로 수정해주세요 !

Comment on lines +34 to +43
const styles = {
container: {
backgroundColor: "#f0f0f0",
height: "100vh",
display: "flex",
flexDirection: "column",
justifyContent: "center",
alignItems: "center",
fontFamily: "sans-serif",
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

styled component가 아니라 이런 식으로 구현하신 이유가 궁금해용 !!

@yezzan9
Copy link
Member

yezzan9 commented May 7, 2025

그리고 base branch가 Leets-official:main으로 되어있어요 ! 수정부탁드립니당

@kangtayie kangtayie changed the base branch from main to 강태이/main May 27, 2025 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants