Skip to content

style: [에셋] - 숏로딩 gif 크기 및 케이스 수정#9

Merged
giljihun merged 3 commits into
developfrom
style/에셋---로딩-gif-크기-수정
Jan 21, 2026

Hidden character warning

The head ref may contain hidden characters: "style/\uc5d0\uc14b---\ub85c\ub529-gif-\ud06c\uae30-\uc218\uc815"
Merged

style: [에셋] - 숏로딩 gif 크기 및 케이스 수정#9
giljihun merged 3 commits into
developfrom
style/에셋---로딩-gif-크기-수정

Conversation

@giljihun
Copy link
Copy Markdown
Member

@giljihun giljihun commented Jan 20, 2026

🎯 PR 내용

기존 LoadingAlert의 .short 타입을 .short30과 .short40으로 분리하여 일관된 사이즈 시스템을 적용했습니다.

- short30: 30pt (작은 셀/썸네일용)
- short40: 40pt (일반 로딩용)

기존에 scaleEffect()로 임의 조정하던 방식을 제거하고, 명시적인 타입으로 관리합니다.

실제로 스케일이펙트로 마구잡이로 각 뷰에서 사용되어지고 있었다.
좋은 이슈인 것 같습니다.

수정된 파일

  • LoadingAlert.swift: LoadingType enum 수정
  • 42개 파일(!!!)에서 LoadingAlert 타입 변경 및 scaleEffect 제거

로딩 사용처 전체 목록

🏠 홈

화면 위치 설명
마이페이지 전체 데이터 로딩
이름 변경 전체 업데이트 중 로딩
알림 선물 전체 선물 정보 로딩

🎪 페스티벌

화면 위치 설명
페스티벌 쇼케이스 그리드 셀 다른 사람이 수정 중일 때 표시
페스티벌 쇼케이스 그리드 셀 키링 이미지 로딩
페스티벌 키링 상세 전체 키링 정보 로딩
페스티벌 키링 상세 알럿 키링 복사 중

📦 보관함

화면 위치 설명
보관함 메인 키링 셀 키링 이미지 로딩
키링 상세 전체 키링 정보 로딩
키링 상세 알럿 키링 복사 중
키링 수정 전체 수정 중 로딩
키링 포장 전체 포장 처리 중
포장된 키링 프리뷰 영역 이미지 로딩
키링 받기 전체 키링 정보 로딩
키링 받기 알럿 받기 처리 중
키링 수집 전체 키링 정보 로딩
키링 수집 알럿 수집 처리 중

🎨 공방

화면 위치 설명
공방 아이템 목록 썸네일 파티클 로딩
공방 아이템 상세 프리뷰 구매 중 로딩
공방 아이템 상세 프리뷰 파티클 로딩
아이템 상세 이미지 전체 이미지 로딩
코인 충전 전체 구매 처리 중

✨ 키링 제작

화면 위치 설명
템플릿 프리뷰 전체 구매 중 로딩
템플릿 프리뷰 프리뷰 영역 템플릿 이미지 로딩
프리뷰 가이딩 시트 가이딩 이미지 로딩
키링 커스터마이징 전체 구매 중 / 합성 중 로딩
폴라로이드 프레임 선택 전체 프레임 로딩
말풍선 프레임 선택 전체 프레임 로딩
아크릴포토 크롭 전체 이미지 로딩
스케치 크롭 전체 이미지 로딩

🎁 뭉치

화면 위치 설명
뭉치 만들기 구매 버튼 구매 처리 중
뭉치 만들기 배경 선택 시트 배경 썸네일 로딩
뭉치 만들기 카라비너 선택 시트 카라비너 썸네일 로딩
뭉치 수정 구매 버튼 구매 처리 중
뭉치 수정 키링 선택 시트 키링 목록 로딩
뭉치 아이템 캡처 중 로딩

👋 온보딩

화면 위치 설명
프로필 설정 완료 전체 저장 중 로딩

현재는 가볍게 기존에 0.5 크기 였던 애들은 30 나머지는 40으로 통일시킴.
피그마 참고.

무진장 많은데, 알잘딱해서 30/40 적용하면 될듯

🔗 관련 이슈

✅ 체크리스트

  • 빌드 성공
  • 테스트 완료

테스트를 하긴 했는데,
앞으로 뷰가 바뀌는 곳이 많아서
앞으로 뷰 개발할 때 정밀한 테스트가 다시 필요함!
(큰 문제는 없어보임)

  • Self-review 완료

@giljihun giljihun self-assigned this Jan 20, 2026
@giljihun giljihun linked an issue Jan 20, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Member

@jini-coding jini-coding left a comment

Choose a reason for hiding this comment

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

단순하지만 엄청난 노동이었군여.. 일일이 바꾸느라 정말 고생하셨습니다👍


enum LoadingType {
case short
case short30
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

개인적으로 short30, short40 보다는 피그마에서처럼 shortS, shortM이 좀 더 사용하기엔 직관적인 것 같지만
개발자(?) 입장에서 어떻게 보면 단순하게 크기가 30, 40이니까 이것도 좋네요. 통과ㅎㅎㅎ

@giljihun giljihun merged commit 6b1db6c into develop Jan 21, 2026
@freshfresh22
Copy link
Copy Markdown
Member

short60 추가 가능한가욤??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Style: [에셋] - 로딩 gif 크기 수정

3 participants