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.

Sign up for GitHub

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

Jump to bottom

Refactor/#65: MobileView 375px에서 420px로 확장 #66

Merged
merged 31 commits into from
Mar 18, 2024
Merged

Refactor/#65: MobileView 375px에서 420px로 확장 #66

merged 31 commits into from
Mar 18, 2024

Conversation

semnil5202
Copy link
Member

@semnil5202 semnil5202 commented Mar 15, 2024 •
edited
Loading

📄 Summary

close #65

같이 보면 좋은 이슈: ConceptBe/conceptbe-design-system#23

MobileView 375px에서 420px로 확장 대응합니다.

그 외 자잘한 이슈를 해결합니다.
ImageView 공통 컴포넌트로 img 사용하시면 될 것 같습니다. src 주소가 잘못되었을 경우 따로 에러핸들링 할 필요없이 defaultSrc prop으로 에러가 났을 경우 보여줄 이미지를 전달할 수 있습니다.

🙋🏻 More

해결한 이슈

위 65번 이슈에서 '로직 동작 오류 수정'에 해당되는 부분이 해결한 자잘한 이슈들 입니다. 이를 검수방 목록에서 보면 다음과 같습니다.

  1. 북마크한 글 없는 상태에서 터치하면 피드로 넘어가야 하는데 안 넘어감
  2. 아이디어 적으러가기 눌러도 글쓰기 화면으로 안 넘어감
  3. 피드에서 스킬 셋의 내용이 길어질 경우 피드 작성 시간 및 스킬 셋의 레이아웃이 뭉개짐

검수방에 기록되지 않은 해결한 이슈

  1. 최초 회원가입 시 작성한 자기소개 누락 문제
  2. Agreement, ErrorBoundary 이미지가 특정 기기에서 height이 길게 늘어져 나오는 문제
  3. 로그인 없이 url로 Agreement, sign-up 직접 접근이 가능했던 문제 (실제로 회원가입이 정상적으로 되진 않지만 confirm 경고창 띄우고 라우팅 하도록 변경했습니다.)

semnil5202 added 26 commits March 15, 2024 02:14
@semnil5202
refactor: public 키워드 제거
8b8a30e 
public은 root path으로 자동 설정되어 다음과 같은 경고가 발생하여 제거한다.
files in the public directory are served at the root path.
Instead of /public/assets/favicon.png, use /assets/favicon.png.
@semnil5202
design: 본문 overflow hidden 시 1열로 작성된 본문이 가려지지 않는 오류 수정
59e118a
@semnil5202
refactor: Feed 페이지 420px 대응
84ca3e6
@semnil5202
fix: 본인의 Profile 페이지가 아닌데도 Navbar에 My SVG가 포커스 된 유형으로 변경되던 오류 수정
dbaa157
@semnil5202
refactor: Profile 페이지 420px 대응
9c9bff2
@semnil5202
refactor: IdeaSection key idx 대신 idea.id로 적용
6561d1e
@semnil5202
refactor: Agreement 페이지 420px 대응
6fde181
@semnil5202
refactor: memberInfo가 없을 경우 로그인 이후 진행하지 않은 케이스로 간주하고 경고 후 라우팅 진행
966d9d0
@semnil5202
refactor: userInfo(memberInfo) 없이 url로 직접 sign-up, agreement 페이지 접근 시…
d881e77 
… 경고 및 라우팅하도록 보완

동일한 로직이므로 useValidateUserInfo 훅으로 분리한다.
@semnil5202
fix: 회원가입시 자기소개 누락된 오류 수정
57bccf9
@semnil5202
design: 댓글, 대댓글 좋아요 cursor pointer 추가
218343b
@semnil5202
design: 글쓰기 가능 조건시 확인 버튼 cursor pointer 추가
5ef6fc2
@semnil5202
feat: Feed 페이지 상단 글쓰기 버튼 및 '아이디어 적으러 가기'를 통해서 글쓰기 페이지로 이동토록 추가
23ffed1
@semnil5202
refactor: assets 디자인 시스템 반영 사항 업데이트
8f55f50
@semnil5202
design: 프로필 수정 버튼 폴드 280px에서 단어 단위 개행 및 가운데 정렬 유지
a2c3629
@semnil5202
chore: 디자인 시스템 0.4.6 버전 적용
8162487
@semnil5202
refactor: 프로필 페이지에서 쓴 게시글, 북마크 한 게시글이 없을 시 아이콘 클릭하여 라우팅 하는 기능 추가
395e707
@semnil5202
refactor: 디자인 시스템에서 SVG 컴포넌트 import 하여 적용
3e63379
@semnil5202
refactor: 프로필 사진 부재 시 default image url 지정
c801550
@semnil5202
refactor: img 중복 컴포넌트 ImageView 컴포넌트로 대체 및 불필요한 상수 제거
c35851a
@semnil5202
refactor: img onError default image 지정 일괄 수정
ed4898f
@semnil5202
design: 프로필 수정 뒤로가기 색 흰색으로 변경
0174a59
@semnil5202
refactor: 아이디어 카드 스킬셋 개행 대응
8929672
@semnil5202
fix: 대댓글 작성 시간 미노출 오류 수정 및 삭제된 답글 시 내용 치환
7f828c7
@semnil5202
refactor: 스킬셋 영역 초과 문제 수정
dd0dd9c
@semnil5202
refactor: TabLayout width 반응형 대응
8317d78
@semnil5202 semnil5202 added the feat 기능 구현 label Mar 15, 2024
@semnil5202 semnil5202 requested a review from yogjin March 15, 2024 18:43
@semnil5202 semnil5202 added the refactor 리팩토링 label Mar 15, 2024
@semnil5202 semnil5202 self-assigned this Mar 15, 2024
yogjin
yogjin approved these changes Mar 17, 2024
View reviewed changes
Copy link
Collaborator

@yogjin yogjin left a comment

Choose a reason for hiding this comment

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

확인했습니다. 수고하셨어요~

@semnil5202 semnil5202 merged commit 574e851 into develop Mar 18, 2024
@semnil5202 semnil5202 deleted the refactor/#65 branch April 18, 2024 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yogjin yogjin yogjin approved these changes

Assignees

@semnil5202 semnil5202

Labels
feat 기능 구현 refactor 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

MobileView 375px에서 420px로 확장
2 participants
@semnil5202 @yogjin