Skip to content

Conversation

@two678
Copy link
Contributor

@two678 two678 commented Apr 26, 2025

📝 Summary

🔧 Changes

✅ Checklist

  • 컨벤션을 준수하였습니다.
  • 변경 사항을 테스트하였습니다.
  • 설명을 충분히 작성하였습니다.
  • 올바른 브랜치에 PR을 보냈습니다.
  • 🤞 리뷰어의 마음을 사로잡았습니다.

🚀 Test Plan

🖼️ Screenshots (UI 변경 시)

변경 전 변경 후
이미지 이미지

📚 Additional

  • 아마도 이 PR은 다크호스일지도... 🎩✨

🚨 "모든 PR에는 커피가 필요하다!"

two678 and others added 30 commits April 18, 2025 15:09
컴포넌트 import 경로를 간결하게 하기 위해 Header, Footer, Modal 디렉토리에 index.js 추가
구조 정리에 따른 재사용성과 가독성 개선 목적

Resolves: #34
Header 컴포넌트의 레이아웃을 개선하여 mainGrid(className)을 이용하여
반응형이 자동으로 적용되게 수정하고 로고가 항상 중앙에 위치하게
고정하고, 프로필 이미지는 오른쪽 끝에 위치하도록 스타일 수정

Resolves: #34
footer의 직접 스타일을 제거하고 .mainGrid 내부에서 레이아웃 정렬
높이와 정렬, 패딩 등의 속성을 mainGrid에 위임하여 일관성 향상
Header와 통일감을 위한 구조 리팩토링

Resolves: #34
Header.styles.js, Footer.styles.js로 emotion 스타일 분리
컴포넌트 파일에서 스타일 import하도록 구조 정리
코드 가독성과 유지보수성 향상
…pdate

🔨 Footer 레이아웃 구조를 mainGrid 클래스 기반으로 리팩토링
크레딧 충전 컴포넌트 애니메이션 수정 Resolves: #35
🔧 Fix: 크레딧 충전 컴포넌트 수정
- 전체적인 ui 레이아웃 구조를 잡았습니다.
- IdolList.jsx 를 컴포넌트로 만들어서 넣을 예정입니다.
- Card 컴포넌트를 반복 렌더링하는 구조를 추가했습니다.

- 슬라이더 구조 안에 배치했으며, 버튼을 좌우로 배치했습니다.

- 추후 케러셀 라이브러리 기능 과의 연계를 고려중입니다.

- Resolves: #28
API 통합 구조 설계 및 공통 모듈 분리 Resolve: #38
api 파일명 변경 Related to: #38
✨ Feat: API 통합 구조 설계 및 공통 모듈 분리
- div 기반에서 label + input[type="radio"] 구조로 리팩토링 했습니다

- input 태그 활용에 맞게 css 스타일을 변경했습니다

- Testpage에서 임시 배열을 생성해 테스트를 했습니다

Resolves: #41
…ntic

♻️ Refactor: RadioButton 컴포넌트 시맨틱태그 적용
피그마에 맞춰서 ui 구현했습니다
우선 mock데이터로 연결해보았습니다.
[내가 선택한 아이돌] 부분은 스크롤 할 예정입니다.
[관심있는 아이돌을 추가] 부분은 데스크톱에서 클릭/ 태블릿& 모바일에서는 스크롤 구현 예정입니다.
- 반복되는 요소로 idolList를 컴포넌트화 하였습니다.
- 캐러셀 라이브러리 사용 고려중입니다.
주석을 추가하였고, 파일명을 index로 변경하면서 파일구조를 짜고 경로를 수정하였음
- credit 상태를 전역에서 관리할 수 있도록 CreditContext 생성

- 초기 크레딧을 localStorage에서 불러오거나 기본값 10000으로 설정

- 크레딧  변경 시 localStorage에 자동 저장되도록 useEffect 설정

- 크레딧 충전 (addCredit), 차감(deductCredit)함수 정의

- useCredit 커스텀 훅을 통해 context에 접근할 수 있도록 구성

Resolves: #44
- CreditProvider로 전체 라우트를 감싸 전역에서 크레딧 상태 접근
  가능하도록 설정

 - 후원 및 크레딧 관련 기능에 필요한 Context 구조 구축 완료

 Resolves: #44
- App.jsx에서 Testpage 경로 인식 오류로 인해 Testpage.jsx를 index.jsx로 변경

- CreditContext에서 credit, addCredit,deductCredit을 불러와 상태 변화
테스트 진행

- 버튼 클릭으로 크레딧 충전 및 사용 기능 테스트 구현

- 현재 크레딧 상태를 화면에 표시하면서 확인
페이지 공통 레이아웃 제외를 위한 라우팅 구조 수정
✨ Feat: 크레딧 상태 관리를 위한 CreditContext 및 Provider 구현
commonStyle.jsx 파일 중복되는 부분 삭제
- Donation 컴포넌트를 List 페이지에 연결해 UI 확인 예정
- 후원을 기다리는 조공 타이틀 태그 absoulute 속성 제거

- 버튼과 슬라이더 영역을 하나로 묶었던 slider div 태그 제거

- 버튼과 슬라이더 내부 영역을 독립적으로 분리해 button에 absolute를 적용
  할 예정
Seon-K and others added 29 commits April 25, 2025 16:15
✨ Feat: Donation 컴포넌트 스켈레톤UI를 구현했습니다.
meta태그 유효하지 않은 속성으로 인한 에러 해결
초기에 카운트업 애니메이션이 동작하지 않는 이슈 해결
- 아이돌 데이터, 로딩, 에러 상태 관리
- 탭 변경, 더보기, 투표 기능 포함
Resolves: #123
- 기존 상태 및 로직을 훅으로 대체하여 컴포넌트 간결화 Resolves: #123
스켈레톤 UI를 추가하여 로딩중을 명시화 Resolves: #123
투표 모달창에 스켈레톤 ui 를 추가하고 모바일 일때는 길이를 9개로 다르게 구현 Resolves: #123
- Vite 설정 파일에 '@'를 src로 매핑
- import 구문을 절대경로로 변경하여 가독성 향상
Resolves: #139
✨ Feat:  경로 별칭(@) 설정으로 import 경로 개선
모바일(425px)부터 title의 font-size를 조정하여 레이아웃 무너지는 현상을 수정했습니다. Button 컴포넌트 내부 글씨가 모바일부터 좌측으로 쏠리는 현상이 발생해 font-size를 10px로 조정하여 해결했습니다. Resolves: #138
- 로컬 스토리지 관련 함수 (laodData, savaData)를 별도 함수로 분리해
  재사용성 및 가독성을 향상시켰습니다.

- 매직 넘버 사용을 줄이고자 상수(LOCAL_STORAGE_KEY,
  SKELETON_DISPLAY_MIN_TIEM. ERROR_DISPLAY_DELAY)를 선언해 두었습니다.

-  로컬 스토리지 초기화(loadInitialData)와 API 데이터
   호출(getDonations)을 분리하여 useEffect 내부 로직을 단순화했습니다.

Resolves: #138
…provement

Refactor/credit recharge improvement
♻️ Refactor: Donation 컴포넌트 리팩토링
@two678 two678 closed this Apr 26, 2025
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.

6 participants