[fix/#152]:랜딩페이지 #154
[fix/#152]:랜딩페이지 #154
Conversation
…ngshi/frontend into 152-feature-landing-page
Kong-E
requested review from
Kong-E and
riverkite0708
and removed request for
Kong-E
|
|
||
| useEffect(() => { | ||
| if (isRootPath) { | ||
| const header = document.querySelector(`.${styles.header_landing}`); |
There was a problem hiding this comment.
document.querySelector는 안티패턴일 것 가타요!
There was a problem hiding this comment.
useRef() 를 써보는건 어떨까요!!
There was a problem hiding this comment.
useRef() 써서 수정했습니다!
components/landing/Landing.tsx
Outdated
| <div className={styles.intro_text_wrap}> | ||
| <Text className={styles.intro_text}> | ||
| 우행시 | ||
| <br /> |
There was a problem hiding this comment.
여긴 <br /> 이 없어도 될 것 같아요~
| <img src="/imgs/landing/inquiry.png" alt="문의하기아이콘" className={styles.inquiry_icon} /> | ||
| 문의하기 | ||
| </button> | ||
| </a> |
There was a problem hiding this comment.
<a> 태그로 감싸져 있으니까 아래에 또 <button>은 빼도 작동에 문제가 없을것 같습니다!!
| <img src="/imgs/landing/clock.png" alt="타이머아이콘" className={styles.timer_title_icon} /> | ||
| 공부시작 | ||
| <br /> | ||
| </Text> |
There was a problem hiding this comment.
여기도 굳이 <br />이 필요하지 않겠죠?!
| </Text> | ||
| <Text size="5" className={styles.timer_detail}> | ||
| <br /> | ||
| 내가 선택한 과목으로 공부 시간을 실시간으로 측정할 수 있어요. <br /> |
There was a problem hiding this comment.
상단 여백을 <br />로 주기보다는 padding이나 margin이 좋을것 같습니다~ (<br />은 자간행간에 영향을 받음)
| alt="트로피아이콘" | ||
| className={styles.ranking_title_icon} | ||
| /> | ||
| 순위조회 |
There was a problem hiding this comment.
별개로 사이트를 띄워봤을때 제목에 들어가있는 아이콘들이 미묘...하게 비율이 깨지는것 같은데 기분탓일까요...? 뭔가 찌그러진 느낌..?
| src="/imgs/landing/landing_cal_record.png" | ||
| alt="랜딩캘린더기록확인" | ||
| className={styles.cal_record_img} | ||
| /> |
There was a problem hiding this comment.
img들이 겹쳐져 있는 부분은 flex랑 transform을 쓰기 보다는 position으로 겹쳐놓는것도 좋을것 같아요~
📋 연관된 이슈 번호
🛠 구현 사항