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

[Update] 디테일 뷰, Label -> ActiveLabel 적용으로 URL 클릭 가능 및 태그, 타이틀 표시 기능 구현 완료 #96

Merged
merged 8 commits into from
Nov 14, 2017

Conversation

blackturtle2
Copy link
Collaborator

제목 그대로입니다.

ActiveLabel이라는 오픈소스를 활용하였고, 약간의 커스터마이징을 거쳤습니다.

결과는.. 두구두두구두구.. 아래 움짤을 살펴보세요.

2017-11-10 00_05_32

@fimuxd
Copy link
Collaborator

fimuxd commented Nov 14, 2017

오와 기존 말풍선 이랑 분위기 완전히 바뀌네요. 😮👍🏼

@fimuxd
Copy link
Collaborator

fimuxd commented Nov 14, 2017

왜 매일 들어오면서 재성님 PR은 이제서야 본건지... 죄송합니다 죄송합니다. 제가 머지할게여!

Copy link
Collaborator

@fimuxd fimuxd left a comment

Choose a reason for hiding this comment

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

ActiveLabel 굉장히 깔끔해 보이네요. 덕분에 좋은 오픈소스 줍줍합니당ㅋㅋㅋ 고생하셨어요!

@@ -19,7 +20,7 @@ class BY_DetailTableViewCell: UITableViewCell {

@IBOutlet weak var bgView: UIView!
@IBOutlet weak var characterIconImage: UIImageView!
@IBOutlet weak var explainBubbleText: UILabel!
@IBOutlet weak var explainBubbleText: ActiveLabel!
Copy link
Collaborator

Choose a reason for hiding this comment

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

아하.. 기존의 Label을 대체하는 식으로 들어가는군요. 편하다

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

여담으로 UITextView는 뭔가 손이 너무 많이 갈 것 같더라구요. 공부는 되겠지만, 효율성이..

// label.URLColor = UIColor(red: 20.0/255.0, green: 55.0/255.0, blue: 255.0/255.0, alpha: 1.0) // 진한 파란색
label.URLSelectedColor = UIColor.gray

label.handleMentionTap { self.alert("ㅅention", message: $0) } // 아직 의미 없는 alert가 표시됩니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋ 재성님 "ㅅention" 뭐죸ㅋㅋㅋㅋㅋ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아............. 사실 나중에 조금 졸아버려서.... OTL


label.handleMentionTap { self.alert("ㅅention", message: $0) } // 아직 의미 없는 alert가 표시됩니다.
label.handleHashtagTap { self.alert("Hashtag", message: $0) } // 아직 의미 없는 alert가 표시됩니다.
label.handleURLTap { self.openSafariViewOf(url: $0.absoluteString) } // 인앱 웹뷰를 실행합니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

아.. 그럼 총 Mention, HashTag URL 이렇게 세가지 기능이 들어가는거군요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넹넹, 맞습니다.
자세한 프로토콜은 만나서 말씀드릴게요. 😉

vc.addAction(UIAlertAction(title: "확인", style: .cancel, handler: nil))

UIApplication.shared.keyWindow?.rootViewController?.present(vc, animated: true, completion: nil)
print("///// alert- 8323")
Copy link
Collaborator

Choose a reason for hiding this comment

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

"alert- 8323" 은 어떤 의미가 있는건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

어..어...... 제 로그 스타일인데요.
print 할 때, /////를 붙이고, 로그 제목을 쓰고.. 뒤에 아무 숫자나 붙여버립니당.

이걸 왜 하냐면, 프린트를 엄청 많이 쓰니까, 로그를 복붙해서 다른 에디터에서 확인할 때, 검색하기 쉬우라고 하는 짓(...)이예요.
제목만 쓰면 되는데, 숫자를 쓰는 이유는, 빌드해보면, ///// alert라고 쓰는 프린트도 엄청 많기 때문이랄까요.
그래서 로그에도 일종의 key 값을 주는 겁니당.

결론: 아이고 의미없다.

@fimuxd fimuxd merged commit f39c26b into DEV Nov 14, 2017
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