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

[FE] Main Page 1차 css 완성 #229

Merged
merged 27 commits into from
Dec 29, 2022
Merged

[FE] Main Page 1차 css 완성 #229

merged 27 commits into from
Dec 29, 2022

Conversation

hoeun0723
Copy link
Collaborator

About

Main Page css 구현

Description

총 8개의 Section으로 나눴고, Section 당 해당하는 Icon이나 Text 값들을 배치 했습니다.

화면을 줄였을 때나 키웠을 때 깨지는 부분이나 아이콘이 변경되어야 하는 부분이 있기에 _추후 수정이 필요_할 거 같습니다.

주의사항

git 오류로 인해 새롭게 clone을 받아와, commit 기록이 날라갔습니다.
주석으로 총 8개의 Section을 나눠 놓고 commit 기록을 새롭게 남겼으니 참고 하시면 좋을 거 같습니다.

Result

전체적인 화면 모습

bandicam.2022-12-26.12-45-48-482.mp4

배경 이미지 비율 오류 모습

bandicam.2022-12-26.12-47-44-507.mp4

사진의 비율을 줄이면 사진 그대로의 모습이 아닌 잘라서 나타내어집니다 😥

Ref

#204

@hoeun0723 hoeun0723 added FE 프론트 관련 Design css 등 사용자 ui 디자인 변경 labels Dec 26, 2022
@hoeun0723 hoeun0723 added this to the 1차 배포 milestone Dec 26, 2022
@hoeun0723 hoeun0723 requested a review from kimyouknow December 26, 2022 03:50
@hoeun0723 hoeun0723 self-assigned this Dec 26, 2022
@hoeun0723
Copy link
Collaborator Author

onClick이 안되네요 ㅠ

Copy link
Contributor

@kimyouknow kimyouknow left a comment

Choose a reason for hiding this comment

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

css작업 역시 어렵네요 ㅠㅠ

</S.FinishSection>
<S.BottomBox>
<S.InformationBox>
<span>커넥트 프로젝트</span>
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분도 위에 처럼 미리 배열 객체를 만들면 좋을 것 같아요!

iconBehind: <S.ThirdCardIcon2 />,
},
];

Copy link
Contributor

Choose a reason for hiding this comment

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

컴포넌트 내부가 아닌 constant폴더에서 관리해야 컴포넌트가 가벼워질 것 같아요. ㅎㅎ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

component가 있는 배열들을 constant 폴더에서 관리 하는 것에 실패했습니다 ㅠ
component가 있는 배열들 때문에 main page 자체의 폴더 내부에 constant 파일을 만드는 건 어떻게 생각하시나요??(아직 적용하진 않았습니다.)

Comment on lines 52 to 49
<span>코넥트는 사람과 사람의 연결을 의미합니다.</span>
맞잡은 손을 이용해 코넥트가 표현하고자 하는 연결의 의미를 나타내었습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

(질문) css속성을 주는 text만 span으로 감싸고, css속성이 필요 없는 text는 span으로 감싸지 않은 건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

네 맞습니다. 사실 이게 맞는 방법인지 모르겠지만, 특정 문자에만 파란색 색상 효과를 줘야 해서 span태그로 묶고 span 태그로 묶인 곳에 부분적으로 효과를 주었습니다. 혹시 더 나은 방법이 있을까요 !?

Comment on lines 199 to 235
export const FirstCardIcon1 = styled.img.attrs({ src: `${GhostIcon}` })`
position: absolute;
display: flex;
left: 10%;
width: 300px;
height: 300px;
`;
export const FirstCardIcon2 = styled.img.attrs({ src: `${CloudIcon}` })`
position: absolute;
display: flex;
top: 8%;
width: 300px;
height: 300px;
`;
export const SecondCardIcon1 = styled.img.attrs({ src: `${EarthIcon}` })`
position: absolute;
display: flex;
left: 10%;
width: 240px;
height: 300px;
`;
export const SecondCardIcon2 = styled.img.attrs({ src: `${HeartIcon}` })`
position: absolute;
display: flex;
left: 4%;
top: 10%;
width: 183px;
height: 188px;
`;
export const ThirdCardIcon1 = styled.img.attrs({ src: `${CellPhoneIcon}` })`
position: absolute;
display: flex;
left: 14%;
width: 170px;
height: 300px;
`;
export const ThirdCardIcon2 = styled.img.attrs({ src: `${MessageIcon}` })`
Copy link
Contributor

Choose a reason for hiding this comment

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

first, second, third 같은 순서를 나타내는 변수명 보다는 icon명(MessageIcon, CellPhoneIcon등등) 그대로 사용하는게 알아보기 쉽다고 생각해요!

font-weight: ${FONT.WEIGHT.BOLD};
font-size: ${FONT.SIZE.MEDIUM};
line-height: ${FONT.HEIGHT.LARGE2};
`,
Copy link
Contributor

Choose a reason for hiding this comment

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

👍👏

@@ -112,6 +126,43 @@ const fonts = {
font-size: ${FONT.SIZE.BASE};
line-height: ${FONT.HEIGHT.MEDIUM};
`,
default2: css`
Copy link
Contributor

Choose a reason for hiding this comment

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

매직넘버! default1, default2보다 알아보기 좋은 변수명이 뭐가 있을까요?🧐 같이 고민해 봐요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

앗 theme 부분 변수명 수정을 안했군요. 다른 자료 저도 찾아보겠습니다 !!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

https://velog.io/@velopert/create-your-own-design-system-with-storybook#%EB%B2%84%ED%8A%BC%EC%9D%98-theme-%EB%A7%8C%EB%93%A4%EA%B8%B0

찾아봤는데 버튼 관련 theme밖에 못 찾아서 그냥 이름을 default_no_bold로 지었습니다 ㅠㅠ 더 좋은 네이밍이 무엇일지 더 고민해보겠습니다.

@hoeun0723
Copy link
Collaborator Author

마지막 부분을 map으로 처리 하면 커넥트 프로젝트라는 문구만 따로 bold처리가 안 되는 트러블이 발생했습니다.(현재 span태그로 묶어서 따로 효과를 적용하고 있음.)

윗 내용과 겹치는 부분이어서 혹시 부분적으로 효과를 줘야할 때 좋은 방법을 알고 계신다면 조언 구합니다😥

@kimyouknow kimyouknow merged commit ff7ea2c into develop Dec 29, 2022
@kimyouknow kimyouknow deleted the FE/#204-main_css branch December 29, 2022 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design css 등 사용자 ui 디자인 변경 FE 프론트 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants