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: 둘러보기 페이지 개발 #45

Merged
merged 9 commits into from
Mar 1, 2023
Merged

feat: 둘러보기 페이지 개발 #45

merged 9 commits into from
Mar 1, 2023

Conversation

endmoseung
Copy link
Contributor

작업 내용

-관련 api생성
-페이지 레이아웃 개발
-추후 갯수 많아지면 무한스크롤 구현 예정

참고 이미지(선택)

화면 기록 2023-02-27 오후 12 10 30

어떤 점을 리뷰 받고 싶으신가요?

-관련 api생성
-페이지 레이아웃 개발
-추후 갯수 많아지면 무한스크롤 구현 예정
@endmoseung endmoseung added the ✨ Feature 기능 개발 label Feb 27, 2023
@endmoseung endmoseung self-assigned this Feb 27, 2023
@netlify
Copy link

netlify bot commented Feb 27, 2023

Deploy Preview for hollang ready!

Name Link
🔨 Latest commit 9483e34
🔍 Latest deploy log https://app.netlify.com/sites/hollang/deploys/63ff606b98824700085c50c7
😎 Deploy Preview https://deploy-preview-45--hollang.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Contributor

@guesung guesung left a comment

Choose a reason for hiding this comment

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

확인했습니다 ~ 코멘트 확인 부탁해요 😄

next.config.js Outdated Show resolved Hide resolved
src/pages/browsing.tsx Outdated Show resolved Hide resolved
-config 간결하게 수정
-불필요한 여는 태그 수정
-반응형 실험 작업
-styledComponent로 전역상태 관리
Copy link
Contributor

@guesung guesung left a comment

Choose a reason for hiding this comment

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

코멘트 확인 부탁해요 ~

package.json Outdated
Comment on lines 27 to 28
"styled-component": "^2.8.0",
"styled-components": "^5.3.6",
Copy link
Contributor

Choose a reason for hiding this comment

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

중복입니다. 근데 global style때문에 styled-component도입하신건가요 ,, ?
그렇다면 저희 이미 src>global.css 파일 있어서 거기서 손 보면 될 거 같아요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 뀨 이게 제가 height에 따라 동적으로 fontsize도입해야되는데 tailiwind에 방법이 없어서 styledcomponents썼어요!

Comment on lines 9 to 11
export const GlobalStyle = createGlobalStyle<GlobalStyleProps>`
html{
font-size:${(props) => `${((props.height * 0.01) / 84.4) * 100}px`}};
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분은 globals.css에서 font-size:x%로도 바꿀 수 있습니다.

Copy link
Contributor

@guesung guesung Feb 27, 2023

Choose a reason for hiding this comment

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

https://canoe726.tistory.com/67 링크 참고해주세용

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저 부분 너무 좋은데 이게 innerHeight에 따라 진행해야돼서 이렇게 진행했습니다 .ㅠㅠ
일단 머지할게요

Copy link
Contributor

@guesung guesung Feb 27, 2023

Choose a reason for hiding this comment

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

아뇨 근데 이미 저희가 16px = 1rem으로 하기로 결정을 해서 다 작업을 진행했는데, 저와 상의 전혀 없이 10px = 1rem으로 바꾸고, styled component를 도입하는 건 좀 아닌 거 같아요
저도 단위 변환 관련해서 몽과 이야기를 나눠 결정을 해야 하는데 이야기를 못나눠 작업 진행을 못하고 있는 상태입니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

반응형 웹 -> globalcss에서 font-size: x vh로 지정
1. Vh : 높이값의 100분의 1 단위. 즉, 높이값이 900px일 때 9px
2. 우리의 경우 높이가 900px. 900:16 = 100:x
font-size:calc(16/9)로 지정하면 저희가 해오던 방식으로 진행할 수 있을 것 같습니다.

-반응형 실험 작업
-styledComponent로 전역상태 관리
-관련 라이브러리 삭제
-단위1/10> 1/16으로 수정
Copy link
Contributor

@guesung guesung left a comment

Choose a reason for hiding this comment

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

코멘트만 확인 후 머지해도 될 거 같아요 ~~ 고생하셨어요 😄

@@ -22,6 +21,7 @@ export default function App({ Component, pageProps }: AppProps) {
<Head>
<link rel="shortcut icon" href="/static/favicon.ico" />
<title>Hollang</title>
<style></style>
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분 제거해도 될 거 같아요 !

Comment on lines 106 to 109
html {
font-size: calc();
}

Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분도 제거해도 될 거 같아요 !

</div>
))}
</section>
<div className={`sticky left-[50%] top-[85%]`}>
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분 템플릿 안써도 될 거 같아요 !

Comment on lines 20 to 24
const [innerHeight, setInnerHeight] = useState<number>(0);

useEffect(() => {
setInnerHeight((window.innerHeight / 84.4) * 100);
}, []);
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분도 제거해도 될 거 같아요 !

@endmoseung endmoseung merged commit d8a4c7f into develop Mar 1, 2023
@guesung guesung deleted the feature/42 branch March 15, 2023 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants