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

6회차 과제 - 권예원 #4

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

Conversation

kyw2790
Copy link

@kyw2790 kyw2790 commented May 8, 2024

  1. 구조 분리
    웹페이지 구조분리
    위에서부터 분리하여서 Header, maincontent,shorts 로 나눠 주었습니다.

  2. 컴포넌트 분리
    image

  3. 아쉬웠던 점
    컴포넌트를 파트별로 분리하였지만 js 부분과 css 부분을 따로 나누는게 더 좋았지 않았을까 싶었습니다.
    파일 분리하는 부분에서 css(style)부분과 컴포넌트 부분을 나눠서 하는게 더 깔끔하고 보기 좋았을거같습니다.

  4. 어려웠던점
    아직 css 부분에서 깔끔하게 만들지 못한다는 것이 조금 어려웠고 파트분리가 되어있으니깐 서로 연결시켜주는 부분에서 오타나 파일 위치 선정을 잘 못 해주면 오류가 나는 부분에서 많이 시간을 써서 그 부분이 힘들었던 것 같습니다.

  5. 그 이외
    계속적으로 컴포넌트 작성을 함수형 컴포넌트를 사용하니 조금은 더 작성하기 편했던것같습니다

@kyw2790 kyw2790 changed the title HW_6 6회차 과제 - 권예원 May 8, 2024
@immms immms mentioned this pull request May 8, 2024
Copy link

@gugitgugit gugitgugit left a comment

Choose a reason for hiding this comment

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

파일을 잘 분리하여 전체적인 코드가 간결하고 가독성이 좋습니다. 디렉토리 구조의 간결화, 직관성 향상 등을 위해 styled-components를 사용해보시는것도 좋을것 같습니다!

src/components/MainContent/MainContent.js Show resolved Hide resolved
@@ -0,0 +1,17 @@
import React from 'react';
import './VideoGrid.css';
import videoImage from './video.png'; // 이미지 파일의 경로를 정확히 지정해야 함

Choose a reason for hiding this comment

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

public 디렉토리에 img 디렉토리를 생성해서 이미지 파일을 저장해두면,

src = "img/imageName"

처럼 쉽게 접근할 수 있더라구요! 필요하시면 참고하시면 될 것 같습니다ㅎㅎ

Copy link
Author

Choose a reason for hiding this comment

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

그런 방벙이 있었군요!!! 이미지 넣는데 고생했는데 public 디렉토리 사용하면 훨씬 쉽게 이미지 넣을 수있겠네요!!! 감사합니당

@kyw2790
Copy link
Author

kyw2790 commented May 8, 2024

@YoonKeumJae 코드 리뷰 마쳤습니다!!

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