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

Assignment-02 #Review #2

Open
myeongjunkim opened this issue Apr 6, 2022 · 0 comments
Open

Assignment-02 #Review #2

myeongjunkim opened this issue Apr 6, 2022 · 0 comments

Comments

@myeongjunkim
Copy link

myeongjunkim commented Apr 6, 2022

광민님 안녕하세요! 두번째 과제도 잘 마무리 하셨나요~~
이번주는 과제중에 질문도 주시고 인상깊은 flex과제도 해주셔서 기억에 남네요!
그럼 Assignment-02 리뷰 시작하겠습니다!

그. 전. 에!
앞으로 세션과 과제가 계속 누적 될텐데 폴더링 관리에 신경써주시면 감사드릴게요!

  • image
  • 썰렁한 Assignment-02 폴더와 session 폴더를 채워주세요!!

Q1 remake flex session contents

  • 세션 내용을 복습하는 과제였는데요. 앞으로도 flex 내용을 잘 이해하고 적용하기 위해 다시 찾아보고 돌아오는 베이스캠프 같은 자료로 활용하면 좋을것 같습니다.

  • 한가지 말씀을 드리면 현재 레이아웃은 간격을 맞추기 위해 flex-basis 를 조정해가며 대략적인 길이를 맞춘 한계가 있었는데요. space-between 등을 활용하여 해당 컨텐츠를 위한 일반화된 레이아웃을 고안해보는 것도 좋은 발전방향이 될 것 같아 언급드립니다!

  • image

Q2 Flex-layout

  • 두번째 레이아웃 과제를 자유롭게 주제 설정하여 멋있는 페이지를 만들어주셨네요! 본페이지는 레이아웃을 잡을 때 위에서 언급해드린 일반화 된 레이아웃을 고려해서 만들었다고 볼 수 있을 것 같아요! 전체 wrap의 크기를 줄이더라도 각 요소들의 비율이 무너지지 않음을 확인 할 수 있어 의미가 더 좋은 것 같습니다. 이처럼 맞춰야 할 라인을 계속 의식하고 필요에 따라 wrap을 계속 추가하며 자리를 잡아가는 방향으로 개발 하시면 제일 좋을 것 같아요!
  • 한가지 피드백 드리면 마지막 오른쪽 아래 요소들을 f1, f2, f3, f4 로 네이밍 하셨는데, 실제로 네이밍 자체가 협업시에 중요한 역할을 하기 때문에 앞으로 구분을 조금씩 생각해주시면 좋을 것 같아요! 현재 f1, f2 가 같은 레벨상의 div 이고 f2 안에 자식태그로 f3, f4 가 있는 구조로 확인 되는데 예를 들어 f3, f4의 네임을 "f2__upper", "f2__lower" 로 바꿔 주시면 f2에 포함된 태그임을 확인 할 수 있어 좋을 것 같습니다!
  • image

과제 리뷰 마치도록 하겠습니다. 고생 많으셨습니다!

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

No branches or pull requests

1 participant