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

9회차 과제 - 정수안 #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

petitmatinz
Copy link

👉🏻 회사 소개 페이지 만들기

본인 마음대로 회사 소개 페이지를 만들어보세요~

가상의 회사든, 실존하는 회사이든 아래 조건만 만족하면 됩니다.

  • 네비게이션 바 존재
  • 요구 페이지 목록
    • 회사 소개
    • 회사 목표
    • 공지사항 (CRUD 가능해야함!)
    • …etc
  • 외부 라이브러리 사용 금지 (styled-components, react-router-dom 제외)
    애플컴퍼니란 이름의 회사 사이트를 만들어 보았습니다
    깃에 올릴때 프로젝트 구조를 분리해서 올리지 못한점이 아쉽습니다.

@petitmatinz
Copy link
Author

또 과제를 급하게 하느라 디자인을 신경쓰지 못한점이 아쉽습니다..

Copy link

@jihyun132 jihyun132 left a comment

Choose a reason for hiding this comment

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

전반적으로 이해하기 쉽고 깔끔하게 기능 구현하신 것 같아요! 많이 배우고 갑니다!! 수고하셨어욤~

};

const handleDeleteNotice = (index) => {
setNotices(notices.filter((_, i) => i !== index));

Choose a reason for hiding this comment

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

여기서 언더바는 무엇일까요??

</NoticeItem>
))}
</NoticeList>
</NoticeContainer>

Choose a reason for hiding this comment

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

styled-component를 활용하는 각 태그의 이름을 이해하기 쉽게 잘 지으신 것 같아요! 저도 다음엔 이 기능을 적극적으로 활용해봐야겠네요!!

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