kishikawakatsumi / SECoreTextView

SECoreTextView is multi style text view.
MIT License
943 stars 143 forks source link

textViewDidEndSelecting of delegate is not called. #41

Open leak4mk0 opened 10 years ago

leak4mk0 commented 10 years ago

初めまして。 CocoaPodsやiOS5への対応がありがたく、 利用させていただいています。

件名で書かせていただいた通り、 DelegateのtextViewDidEndSelectionが呼び出されません。 一方で、textViewDidChangeSelectionは呼び出されています。

恐らく、finishSelectingメソッドにおいて、 自クラスにてrespondsToSelectorを呼び出していることが問題だと思われます。 File: SETextView.h, Line: 529~543

- (void)finishSelecting
{
#if TARGET_OS_IPHONE
    if (!self.editing) {
        if (self.showsEditingMenuAutomatically) {
            [self hideEditingMenu];
            [self showEditingMenu];
        }
    }
#endif

    if ([self respondsToSelector:@selector(textViewDidEndSelecting:)]) {
        [self.delegate textViewDidEndSelecting:self];
    }
}

勝手ながら、改善案を示させていただきますと、 次のようになるのではないでしょうか。 File: SETextView.h, Line: 540

    if ([self.delegate respondsToSelector:@selector(textViewDidEndSelecting:)]) {
kishikawakatsumi commented 10 years ago

う、確かにおかしいですね。直します。 ありがとうございます。