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.

Sign up for GitHub

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

Jump to bottom

[Merge] #86 long press 햅틱 추가 #88

Merged
merged 1 commit into from Jun 14, 2022
Merged

[Merge] #86 long press 햅틱 추가 #88

merged 1 commit into from Jun 14, 2022

Conversation

Minkyeong-Ko
Copy link
Collaborator

@Minkyeong-Ko Minkyeong-Ko commented Jun 14, 2022 •
edited

작업 내용

  • Long Press 했을 때 햅틱이 동작하도록 코드를 추가하였습니다

리뷰 포인트

  • 상의 결과 햅틱 종류를 impact의 medium으로 설정했습니다

다음으로 진행될 작업

  • 안건 종료 시 햅틱(success) 및 사운드 추가

질문

  • HapticManager 파일을 우선 ViewModel 디렉토리에 추가했는데, 저는 MVVM..?이 익숙하지 않아서 맞는 지 헷갈립니다. 잘 아시는 분이 있으시다면 확인 및 설명 주실 수 있을까요?

References

https://seons-dev.tistory.com/entry/SwiftUI-Haptic-Feedback-haptics-vibrations#recentComments

@Minkyeong-Ko Minkyeong-Ko added feature New feature or request 다니 labels Jun 14, 2022
@imparang
Copy link
Collaborator

ViewModel은 말그대로 View 그리는데 사용되는 데이터 모델과 관련된 것들을 모아두는 곳이라고 생각합니다.
HapticManager 같은 경우는 View를 그리는거랑은 상관이 없어서 Utils 쪽에 빼는 것이 좋아보여요..!

@Minkyeong-Ko Minkyeong-Ko added this to In progress in MC2 via automation Jun 14, 2022
@Minkyeong-Ko
Copy link
Collaborator Author

Minkyeong-Ko commented Jun 14, 2022 •
edited

ViewModel은 말그대로 View 그리는데 사용되는 데이터 모델과 관련된 것들을 모아두는 곳이라고 생각합니다.
HapticManager 같은 경우는 View를 그리는거랑은 상관이 없어서 Utils 쪽에 빼는 것이 좋아보여요..!

오 감사합니다!
파일 위치 변경 완료 하였습니당

@Minkyeong-Ko
Feat: #86 롱프레스 시 햅틱 추가
b1eee41 
Long Press 시 한 번만 햅틱이 동작하게 코드를 추가했습니다

Ref: #86
imparang
imparang approved these changes Jun 14, 2022
View reviewed changes
Copy link
Collaborator

@imparang imparang left a comment

Choose a reason for hiding this comment

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

확인이용

@imparang imparang merged commit 5275a9f into develop Jun 14, 2022
MC2 automation moved this from In progress to Done Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@imparang imparang imparang approved these changes

Assignees
No one assigned
Labels
feature New feature or request 다니
Projects
MC2
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Minkyeong-Ko @imparang