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
Closed
WooGi1020
approved these changes
Jan 20, 2025
src/components/home/Navbar.tsx
Outdated
| </div> | ||
| <span className={`text-[10px] text-textDarkGray`}>프로필</span> | ||
| </Button> | ||
| <Link to="/mypage"> |
There was a problem hiding this comment.
제가 알기론 아마 리액트에서 사용하는 Link 태그가 a 태그일텐데 보통 a 태그가 자체적으로 클릭 이벤트를 가지고 있다보니 내부에 button 같은 중복되는 역할의 태그를 넣으면 접근성에 좋지 않은걸로 알고 있어요!! 요 부분은 그냥 Link 만 사용해주셔도 좋을 것 같습니다!
There was a problem hiding this comment.
아항 그럼
<Link
to="/mypage"
className="flex flex-col items-center justify-center gap-1"
>
<div className="flex items-center rounded-full bg-textDarkGray w-[24px] h-[24px]">
<BasicProfileIcon className="text-[#AEAEAE]" />
</div>
<span className="text-[10px] text-textDarkGray">프로필</span>
</Link>
이런식의 수정 말씀하시는거죠..??
src/pages/landing/FirstLanding.tsx
Outdated
Comment on lines
9
to
11
| title="지금 바로 매칭하기" | ||
| subtitle={`자동 매칭 기능을 통해\n현재 위치에서 매칭하기`} | ||
| /> |
There was a problem hiding this comment.
subTitle props에서 직접 스트링 문자열을 내려주는 부분이 \n을 사용하시다 보니 Header에서 부득이하게 split 메서드가 사용되고 있는 것 같은데 혹시 여기를 h1, p 태그로 직접 내려주고 Header에서 children으로 받아 사용하는 느낌은 어떨까요?? 그리고 변경될 일 없는 문자열은 상수 폴더에 정리해도 좋을 것 같습니다!!
There was a problem hiding this comment.
아항 split 메서드 사용하면 조금 더 간결해보일까 싶어서 사용했었는데 우기님 말씀처럼 하는게 더 좋을듯 하네용! 수정하겠습니다!!
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.
1. 무슨 이유로 코드를 변경했나요?
2. 어떤 위험이나 장애를 발견했나요?
3. 관련 스크린샷을 첨부해주세요.
2025-01-20.2.01.07.mov
4. 완료 사항
랜딩페이지 UI를 수정을 완성하였습니다
5. 추가 사항
위 영상은 모바일 뷰 입니다! 옆으로 드래그하는 방법, 그리고 저 페이지네이션 버튼을 눌러도 넘어갑니다!