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: 음악 검색 기능, 이미지 저장 기능 #11

Merged
merged 13 commits into from
Jun 16, 2022
Merged

Feat: 음악 검색 기능, 이미지 저장 기능 #11

merged 13 commits into from
Jun 16, 2022

Conversation

JIWON1923
Copy link
Member

음악검색기능

  • 검색 결과가 한국어로 뜨도록 변경
  • 코드 리팩토링 (뷰가 너무 길어서 나누어 작성)
  • 주석 추가
    음악검색

이미지 저장 기능

  • 스크린샷 기능 추가 ( 네비게이션 제거 방법은 임의로 크기를 지정해서 크롭했습니다 )
  • 상단 보이는 메뉴 버튼은 수정해야할 것 같습니다. (이슈 올려놓았습니다)

이미지 저장 결과

- linespacing
- font style, color
- padding
* commit '3f178b684d082369dd35768400d59c1f64f9f17d':
  Comment : add comment
  Feat : CD Rotate Animation
  Feat: HomeView
* main:
  Style: UI offset 조정 Feat: DetailView 상에 Menu 추가
  Style: RecordBackground Struct 추가 Comment: 주석 수정
  Feat: UI컴포넌트 별 버튼기능 추가 Comment: 주석 수정 및 추가
  Comment: 주석 세부사항 수정
  Feat: 본문 UI뷰 작성, 본문 결과 뷰 네비바 생성, 네비바 내 메뉴 버튼 추가, List로 이동할 수 있는 네비게이션 링크 추가
Merge branch 'main' into soi

* main:
  Style: RecordUIView 추가 Style: RecordBackground struct를 RecordUIView로 이동

# Conflicts:
#	Recorder.xcodeproj/project.pbxproj
@mxnxxii
Copy link
Collaborator

mxnxxii commented Jun 16, 2022

확인했습니다!

@boskim22
Copy link
Collaborator

확인했습니다

@100seo
Copy link
Collaborator

100seo commented Jun 16, 2022

  • 이미지 저장이 어떻게 되는지 궁금해서 결과이미지 같이 올려주시면 좋을 것 같아요
  • 업로드 하신 하단 이미지가 아이폰13 시뮬레이터가 맞나요?

@YunSeok-Choi
Copy link
Collaborator

확인했습니다.

@Eunbi-Cho
Copy link
Collaborator

확인했습니다

@JIWON1923
Copy link
Member Author

@100seo

  • 이미지 저장이 어떻게 되는지 궁금해서 결과이미지 같이 올려주시면 좋을 것 같아요
  • 업로드 하신 하단 이미지가 아이폰13 시뮬레이터가 맞나요?

위 업로드 된 사진이 아이폰13 시뮬레이터에서 갤러리에 저장된 최종 화면입니다.
기본 갤러리 네비게이션 바를 제외하고 보여주기 위한 화면이었습니다!

갤러리에서 보게 되면 다음과 같이 보이게 됩니다!

스크린샷 2022-06-16 오후 4 45 27

@100seo
Copy link
Collaborator

100seo commented Jun 16, 2022

이해 완

@YunSeok-Choi YunSeok-Choi merged commit dc18508 into main Jun 16, 2022
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.

None yet

6 participants