Skip to content

Feat/key board button#290

Merged
ksiomng merged 7 commits intoDevelopfrom
Feat/KeyBoardButton
Jan 22, 2025
Merged

Feat/key board button#290
ksiomng merged 7 commits intoDevelopfrom
Feat/KeyBoardButton

Conversation

@ksiomng
Copy link
Collaborator

@ksiomng ksiomng commented Jan 21, 2025

📕 Issue Number

Close #

📙 작업 내역

구현 내용 및 작업 했던 내역

  • 필요없는 코드삭제
  • 보기어려운 이름 수정
  • 키보드 닫는 버튼 추가
  • 폰트 바꾸는 부분에 생긴 문제 해결

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

📋 체크리스트

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 테스트하다가 이상한 오류를 발견해 수정해줬습니다

  • 키보드 버튼 아이콘이 피그마랑은 달라서 이건 어떤지 바꿔야하면 아이콘 이미지가 필요할꺼같습니다 !!



@ksiomng ksiomng self-assigned this Jan 21, 2025
}
}
.onAppear{
NotificationCenter.default.addObserver(
Copy link
Collaborator

Choose a reason for hiding this comment

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

오오 NotificationCenter를 토스트 말고도 키보드 hide show에도 사용할 수 있군여 ..?!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

https://gun-oo.tistory.com/13

이걸 컨닝했는데 쓰면서 신기방기 햇서요 .

@ksiomng ksiomng merged commit 13afc37 into Develop Jan 22, 2025
1 check passed
@ksiomng ksiomng deleted the Feat/KeyBoardButton branch February 10, 2025 01:13
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.

2 participants

Comments