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

Feat/#214 change app icon #215

Merged
merged 7 commits into from
May 19, 2024
Merged

Feat/#214 change app icon #215

merged 7 commits into from
May 19, 2024

Conversation

somarok
Copy link
Contributor

@somarok somarok commented May 19, 2024

:: 최근 작업 주제 (하나 이상의 주제를 선택해주세요.)

  • 기능 추가
  • 리뷰 반영
  • 리팩토링
  • 버그 수정
  • 컨벤션 수정

:: 구현 목표 (해당 브랜치에서 구현하고자 하는 하나의 목표를 설정합니다.)

  • 앱 아이콘을 변경합니다.
  • 반응형으로 사이즈 값을 변환해주는 유틸을 만듭니다.

:: 구현 사항 설명 (작업한 내용을 상세하게 기록합니다.)

  1. 안드로이드/ios 앱 아이콘 파일 추가 및 변경
  2. ReactionUtil 작성.
    -- mediaQuery를 사용하여 각 디바이스 크기에 따라 위젯에 사이즈를 넣는 코드가 여러 곳에서 쓰였기 떄문에 유틸을 만들었습니다.
  3. 홈화면, 피드 검색 화면 드롭다운 버튼 섹션에 reactionUtil 함수 적용
  4. 피드 검색 화면 UI 패딩 수정

:: 성장 포인트 (해당 기능을 구현하며 고민했던 사항이나 새로 알게된 부분, 어려웠던 점 등을 작성합니다.)


:: 기타 질문 및 특이 사항

@somarok somarok self-assigned this May 19, 2024
@somarok somarok linked an issue May 19, 2024 that may be closed by this pull request
2 tasks
@somarok somarok added ✨ Feature 기능 개발 [PR] 리뷰요청 리뷰 요청 드립니다. labels May 19, 2024
Copy link
Collaborator

@NalaJang NalaJang left a comment

Choose a reason for hiding this comment

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

LGTM👍🏻

Copy link
Collaborator

@KimDonghyeok KimDonghyeok left a comment

Choose a reason for hiding this comment

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

LGTM 👍

수고하셨습니다

double widthPercentage = marginWidth / screenWidth;
return screenWidth * widthPercentage;
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

@somarok reactWidget -> rectWidth 오타 같습니다 :D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정하여 반영했습니다 :)

Copy link
Collaborator

@SeongJongHo SeongJongHo left a comment

Choose a reason for hiding this comment

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

LGTM👍

Copy link
Collaborator

@gyubit gyubit left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏼

@somarok somarok merged commit cb93568 into dev May 19, 2024
2 checks passed
@somarok somarok deleted the feat/#214_change_app_icon branch May 19, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 [PR] 리뷰요청 리뷰 요청 드립니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

앱 아이콘 변경 및 반응형 UI Util 추가
5 participants