https://bugs.freedesktop.org/show_bug.cgi?id=56995

Manik Malhotra <manikmalhotr...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #29 from Manik Malhotra <manikmalhotr...@gmail.com> ---
(In reply to comment #28)
> Hi Manik,
> good to see you working on this.
> can you send a 'git diff' of what you've done together with your question to
> the Mailing List?
> There you get more attention than here.

I hope I'm not disturbing you, I just wanted to confirm before disturbing a
whole lot of more people if my approach was acceptable.
I had earlier removed the Job which meant that once Listener was activated (On
startup) it looked if the Enable Presentation Screen was checked and
accordingly added or ignored the Job.
Now I have moved the condition to the actual event listener function 

void SAL_CALL PresenterScreenListener::notifyEvent( const
css::document::EventObject& Event ) throw (css::uno::RuntimeException);

And only initialised the presenterScreen object if object is enabled which now
works flawlessly. I followed the norms and the function which checks if the
option is enabled is defined in presenterScreen interface and implemented in
presenterScreen implementation and uses presentationConfiguration and existing
context.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to