[prev in list] [next in list] [prev in thread] [next in thread] 

List:       ktexteditor-devel
Subject:    Re: SmartRangeNofifier::caretExitedRange
From:       Hamish Rodda <rodda () kde ! org>
Date:       2007-10-10 14:56:07
Message-ID: 200710110056.09451.rodda () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Wed, 10 Oct 2007 05:29:34 am Anders Lund wrote:
> Who is responsible for emitting this signal?
>
> Because I connected to it, but never receive it.
>
> Code (kate docwordcompletion plugin):
>
> connect( d->liRange->primaryNotifier(),
> SIGNAL(caretExitedRange(KTextEditor::SmartRange*, KTextEditor::View*)),
> this, SLOT(lastInsertedRangeExited(KTextEditor::SmartRange*,
> KTextEditor::View*)) );
>
> When a new directional completion is started, I call
>
>   KTextEditor::SmartInterface *si =
>      qobject_cast<KTextEditor::SmartInterface*>( m_view->document() );
>   if ( si )
>     si->addHighlightToView( m_view, d->liRange, true );
>
> lastInsertedRangeExited does:
>
>   KTextEditor::SmartInterface *si =
>      qobject_cast<KTextEditor::SmartInterface*>( m_view->document() );
>   if ( si )
>     si->removeHighlightFromView( m_view, d->liRange );
>
> But that never happens, as the signal is never received.
>
> Shouldn't the View implementation emit the signal? Grepping around in the
> KTextEditor folder doesn't reveal it being submitted from anywhere (nor
> does grepping in kdelibs/kate)...

That sounds like something I planned but didn't implement yet.  It should be 
triggered by the view, I think you'll find I did it indirectly (via methods 
in the KateSmartCursor) for other signals.

I'm working on getting my dev environment back online but it's taking some 
time.

Cheers,
Hamish

["signature.asc" (application/pgp-signature)]

_______________________________________________
KTextEditor-Devel mailing list
KTextEditor-Devel@kde.org
https://mail.kde.org/mailman/listinfo/ktexteditor-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic