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

기존 예제 템플릿 페이지를 서비스 내부로 옮겨옵니다. #66

Merged
merged 9 commits into from
Sep 9, 2023

Conversation

hhhminme
Copy link
Collaborator

@hhhminme hhhminme commented Sep 6, 2023

📌 이슈 링크


📖 작업 배경

  • 서비스 개발을 위해 예제 템플릿에 있는 페이지들을 추가합니다.

🛠️ 구현 내용

  • 공통으로 쓰이는 gnbfooter를 만들어주었습니다.

  • 유사하게 서비스되는 도메인을 나눠 라우트를 시켜주었습니다. 나눠준 이유는 공통으로 쓰이는 API의 관심사가 유사하고 함께 쓰일 수 있는 UI가 많아서 묶어주었습니다.

    • (article) : 글, 글쓰기 페이지
    • (auth) : 로그인, 회원가입
    • (user) : 프로필, 설정 페이지

💡 참고사항

  • 현재 기능이 연동되어 있지 않아 모든 페이지가 라우팅 되지 않습니다.
  • 차후 추가기능을 붙여 개발 완료하도록 하겠습니다.
  • pr이 너무 커져버렸네요. 기능구현시엔 다이어트를 시켜서 기능별로 올리도록 할게요

🖼️ 스크린샷

image image image image image

@hhhminme hhhminme self-assigned this Sep 6, 2023
Comment on lines +1 to +7
import { Titillium_Web as titilliumWeb } from "@next/font/google";

export const titillium = titilliumWeb({
subsets: ["latin"],
weight: "400",
variable: "--titillium-web-font",
});
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

navbar-brand 라는 좌측 상단 로고의 폰트가 해당 폰트가 적용되어 있어 의존성을 통해 추가해줍니다.

"next": "13.4.19",
"react": "18.2.0",
"react-dom": "18.2.0",
"@tanstack/query-core": "^4.29.17",
"@tanstack/react-query": "^4.29.17",
"react-ionicons": "^4.2.1",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

템플릿 css에서 쓰이고 있는 icon은 ionicons 라는 것인데 이것의 내부 스타일링은 styeld-component 로 되어 있어 icon임에도 rsc에서 사용하지 못합니다.

svg로 사용할 수 있는 다른 방안을 찾아보고자 합니다.

Comment on lines -12 to -22
@font-face {
font-family: "Ionicons";
src: url("../fonts/ionicons.eot?v=2.0.1");
src:
url("../fonts/ionicons.eot?v=2.0.1#iefix") format("embedded-opentype"),
url("../fonts/ionicons.ttf?v=2.0.1") format("truetype"),
url("../fonts/ionicons.woff?v=2.0.1") format("woff"),
url("../fonts/ionicons.svg?v=2.0.1#Ionicons") format("svg");
font-weight: normal;
font-style: normal;
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아이콘의 사용 쓰임새를 보니 웹폰트를 가져와 before에 관련 특수문자를 꽃아주고 있었습니다.

현재는 불필요하여 삭제해둡니다.

* @hhhminme @brgndyy @InSeong-So @Siihyun @0uizi0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

시현님이 write 권한이 없는 것 같습니다 확인 부탁드려요 @InSeong-So

@hhhminme hhhminme merged commit 2febb0c into pagers-org:team9/hhhminme Sep 9, 2023
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.

1 participant