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

feat : 분류 컴포넌트, 문서 보드 컴포넌트, 푸터 컴포넌트 퍼블리싱 #32

Merged
merged 6 commits into from
Mar 6, 2024

Conversation

Ubinquitous
Copy link
Member

Issue Number

close #11
close #31
close #12

What

부마위키 문서 보드와 푸터, 분류 컴포넌트를 개발했습니다.

How

  1. 기존에 서브푸터를 따로 직접 렌더링하던 구성방식을 변경하고, 문서보드 컴포넌트에 서브 푸터를 병합시켰습니다.
  2. 인스타그램이 개설되지 않아 기존에 존재하던 인스타그램 로고를 삭제했습니다.
  3. 페이지들의 app 레이아웃은 모두 부마위키 문서 보드 내에서 구성됩니다. 공통적으로 들어가는 분류 컴포넌트와 문서제목 (ex:부마위키:학생) 컴포넌트를 중복되는 코드 없이 어떻게 효율적으로 처리할 수 있는지를 논의해보아야합니다.

기본적으로 page 레이아웃 -> 앱 레이아웃으로 정보를 전달하여 분류 컴포넌트와 제목 컴포넌트가 바뀌는 상황이 제일 이상적이지만,
이를 위해 전역 컴포넌트를 라우팅마다 사용하는 것은 과하게 코드의 양이 늘어날 것이라는 우려를 했습니다.

어떤 방식으로 이 문제점을 해결해야할지에 대해서도 생각해보아야할 것 같습니다.
추가적인 의견 있으시면 comment와 함께 남겨주셔도 매우 도움될 것 같습니다!

ScreenShot

스크린샷 2024-03-06 오후 5 13 34 스크린샷 2024-03-06 오후 5 13 37

@Ubinquitous Ubinquitous added the feat 새로운 기능 label Mar 6, 2024
@Ubinquitous Ubinquitous self-assigned this Mar 6, 2024
@Ubinquitous Ubinquitous changed the title Feat/classify,footer,board feat : 분류 컴포넌트, 문서 보드 컴포넌트, 푸터 컴포넌트 퍼블리싱 Mar 6, 2024
Copy link
Member

@jyh071116 jyh071116 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

Comment on lines 18 to 19
<div className={styles.information}>
<span className={styles.information}>
Copy link
Member

Choose a reason for hiding this comment

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

같은 스타일이 2번 쓰인 이유가 있을까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

div에는 styles.informationBox를 적용하려고했는데 혼동했나보네요!
잘 캐치해주셔서 감사합니다!!

@Ubinquitous Ubinquitous merged commit e26913c into main Mar 6, 2024
1 check passed
@Ubinquitous Ubinquitous deleted the feat/classify,footer,board branch November 15, 2024 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

문서 보드 컴포넌트 개발 분류 컴포넌트 개발 서브푸터, 푸터 컴포넌트 개발
2 participants