[FE] mock데이터 수정 및 css 수정 #228
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
getTechSkillsWithCategory getRandomTechSkills
hoeun0723
approved these changes
Dec 23, 2022
| @@ -83,16 +94,18 @@ export const TeamName = styled.div` | |||
| font-weight: 700; | |||
| `; | |||
|
|
|||
| export const TeamSlogan = styled.div``; | |||
There was a problem hiding this comment.
team 슬로건 관련 css는 따로 적용해주지 않아도 되는건가요?? 비어 있어서 질문 드립니다
There was a problem hiding this comment.
넵! 아직 텍스트 스타일 관련한 작업이 theme.js에 반영이 안되어 있어서 변수로 묶어두기만 했습니다.
| width: 1180px; // * : 페이지를 줄였을 시 한 행에 4개의 Card가 있게 유지 하려면 % 가 아닌 px로 값을 지정해줘야한다. | ||
| width: 100%; |
There was a problem hiding this comment.
px 값이 아닌 다시 %로 지정해준 이유가 있나요 ??
There was a problem hiding this comment.
px로 지정할 시 페이지 witdh를 줄였을 때, gird의 요소(유저 박스)들이 겹치는 문제가 발생했습니다. ㅠㅠ
%로 설정해서 grid의 요소들이 겹치지 않게 수정했습니다. 대신 페이지의 width를 줄였을 때 한 행에 4개의 유저박스가 보이는게 아닌 최소 넓이에 맞는 숫자로 보여집니다. (ex: 페이지 width를 줄이면 4개 -> 3개 -> 2개 이런식으로 줄어드면서 보입니다.)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About
Description
mock데이터만 수정하고, css도 피그마에 있는 내용을 그대로 반영 한 것 외에 중요한 로직 변경이 없어 세부사항은 생략하겠습니다.
(브랜치명은,,,,, 회원가입 로직 체크하다가 급하게 보이는대로 작업해서 작업한 내용과 맞지 않습니다.😓)