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

Write Curation View UI 변경사항 #351

Merged
merged 18 commits into from
Dec 1, 2022
Merged

Conversation

HanGyeongjun
Copy link
Member

@HanGyeongjun HanGyeongjun commented Dec 1, 2022

관련 이슈

구현/변경 사항

  • 3개 페이지로 나뉘어져 있던 WriteCurationView를 하나로 통합했습니다.
  • 카테고리 선택 버튼의 UI를 변경했습니다.
  • 단축어 사용에 필요한 앱에 대한 설명을 추가했습니다.
  • 이 외 작은 UI 변경사항 반영 및 수정이 있습니다.

리뷰 시 참고해주세요.

  • 통합되어 사용되지 않는 2개의 파일(WriteShortcutdescriptionView, WriteShortcutTagView)는 코드리뷰 후, 머지 전에 삭제 예정입니다.
  • NavigationStack 관련 필요 없는 코드가 남아있는지 봐 주시면 감사하겠습니다!
  • 더 이상 사용되지 않는 단축어 사용을 위한 요구사항 관련해 추가적으로 처리할 부분은 없는지 확인해주시면 감사하겠습니다.
  • 업로드 버튼의 컬러 확인 부탁드립니다. @mxnxxii

스크린샷

작동 시 동영상 파일로 대체합니다.

Simulator.Screen.Recording.-.iPhone.14.-.2022-12-02.at.04.03.45.mp4

Copy link
Member

@JMM00 JMM00 left a comment

Choose a reason for hiding this comment

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

버튼 이름 변경과 스타일 적용 외에는 시뮬레이터에서 작동확인 했습니다!

Comment on lines 135 to 150
Text("업로드")
.foregroundColor(.Text_Button)
.opacity(0.7)
Text("업로드")
.foregroundColor(.Primary)
.opacity(shortcut.color.isEmpty ||
shortcut.sfSymbol.isEmpty ||
shortcut.title.isEmpty ||
!isNameValid ||
shortcut.downloadLink.isEmpty ||
!isLinkValid ||
shortcut.subtitle.isEmpty ||
!isOneLineValid ||
shortcut.description.isEmpty ||
!isMultiLineValid ||
shortcut.category.isEmpty ? 0.3 : 1)
Copy link
Member

Choose a reason for hiding this comment

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

버튼에 스타일 적용이 안되어있네요..! 업로드는 Headline() 취소는 Body1()으로 적용 부탁드립니다!

Copy link
Collaborator

@JIWON1923 JIWON1923 left a comment

Choose a reason for hiding this comment

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

LGTM 고생하셨습니다

}
}

//MARK: -단축어 사용 필요 앱
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분 편집할 때 클릭이 안 돼요!
이슈 끊어서 추후에 작업이 필요할 것 같아요

Copy link
Member Author

Choose a reason for hiding this comment

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

추후에 작업하겠습니다!

@HanGyeongjun HanGyeongjun merged commit 4e21252 into develop Dec 1, 2022
@HanGyeongjun HanGyeongjun deleted the writeCurationView/updateUI branch December 1, 2022 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type-Feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] WriteShortcutView 디자인 변경
3 participants