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] 단축어 글쓰기 시 자동 타이틀 채우기 #482

Merged
merged 10 commits into from
Oct 17, 2023

Conversation

HanGyeongjun
Copy link
Member

관련 이슈

구현/변경 사항

  • LinkPresentation을 사용하여 단축어 작성 시 URL 작성 후 Submit 시 자동으로 title이 작성되도록 했습니다. LPLinkMetadata의 Title (URL 타이틀) 값을 사용했습니다.
  • Main App과 Share Extention 둘 모두에서 동작합니다.
  • 사용자가 여러 번 URL을 Submit 할 경우를 대비해 Async를 사용했습니다.
  • title 값이 20자가 넘게 되면 모두 봍여넣어지지만, 작성은 막힙니다. 사용자가 직접 20자 미만으로 줄여야 작성이 가능합니다.

@HanGyeongjun HanGyeongjun added the Type-Feature New feature label Jul 26, 2023
@HanGyeongjun HanGyeongjun requested review from JMM00 and jim4020key July 26, 2023 16:41
@HanGyeongjun HanGyeongjun self-assigned this Jul 26, 2023
@HanGyeongjun HanGyeongjun added this to the 14th Sprint milestone Jul 28, 2023
@JMM00
Copy link
Member

JMM00 commented Aug 11, 2023

링크의 경우 붙여넣기 후 키보드의 엔터를 누르기보다는 키보드 외의 영역을 터치해 키보드를 내린 후 다음 영역을 채우는 경우가 많다고 느껴지는데 이때에는 제목이 자동으로 채워지지 않는 것 같아요. 혹시 링크 입력창이 focus 되었다가 unfocus된 상태를 감지해서 타이틀을 채우도록 하는 것은 어려울까요?

Copy link
Collaborator

@jim4020key jim4020key left a comment

Choose a reason for hiding this comment

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

LinkPresentation 배워갑니다 로겐시 👍

  • 엘리 의견에 동의합니다 ~ 키보드 외의 영역 터치로도 동작하면 좋을 것 같아요!!
  • ShareExtension 실행했을 때 단축어 이름 길이 경고가 안뜨는데 제한할 수 있을까요 ??

@@ -42,6 +48,28 @@ struct ShareExtensionWriteShortcutView: View {
}
.background(Color.shortcutsZipBackground)

Copy link
Collaborator

Choose a reason for hiding this comment

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

단축어 이름이 fetch 되는 중임을 알려주면 좋을 것 같아요 !!

Suggested change
.overlay {
if isFetchingMetadata {
ProgressView()
}
}

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

This comment was marked as resolved.

@HanGyeongjun
Copy link
Member Author

@JMM00 @jim4020key 아래 두 가지 기능 추가했습니다. 참고해서 리뷰 부탁드려요!

  • downloadLink 가 입력될 경우 onChange로 감지해 자동으로 metadata fetch하기 -> 기존 onSubmit 시 fetch될 때보다 사용성이 향상되었습니다.
  • metadata에서 가져온 title이 20자가 넘을 경우 자동으로 뒷부분 잘라 20자 이내로 유지하기

@HanGyeongjun HanGyeongjun changed the title 단축어 글쓰기 시 자동 타이틀 채우기 [Feat] 단축어 글쓰기 시 자동 타이틀 채우기 Oct 9, 2023
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.

포커스 사라졌을 때 자동완성 기능 및 shareExtension에서 20자 제한 기능 확인완료했습니다!
고생하셨어요 로겐 ~~

Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
@HanGyeongjun HanGyeongjun merged commit 8755d79 into develop Oct 17, 2023
@HanGyeongjun HanGyeongjun deleted the feature/481-TitleAutoFill branch October 17, 2023 09:36
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] 단축어 글쓰기 시 자동 제목 채우기
4 participants