To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=36800


User bm changed the following:

                  What    |Old value                 |New value
================================================================================
               Assigned to|bm                        |cl
--------------------------------------------------------------------------------




------- Additional comments from [EMAIL PROTECTED] Wed Mar  2 05:23:17 -0800 
2005 -------
In SlideShowImpl::activate() and SlideShowImpl::deactivate() the slot
SID_SHOWPOPUPS is called without additional argument (like it was done in OOo
1.1 and also in PresentationViewShell::FinishInitialization).  Adding this
additional argument, however, does not help.

Referring to CD, SID_SHOWPOPUPS is defined as property slot without arguments,
but used as an execute slot with arguments.  Thus it will no longer work when
slot handling is globally changed to the new API.  As additionally, it is
unclear how the undocumented argument works, and it obviously doen't work in the
way it did in OOo 1.1, I would suggest to replace the execution of the
SID_SHOWPOPUPS by calling SlideShowImpl::hideChildWindows() and
SlideShowImpl::showChildWindows() in activate()/deactivate(), which does exactly
what we want -- almost.

In hide/showChildWindows nothing is done if the presentation is in FullScreen
mode.  I am not sure why this was done, IMO we could just remove this option.

->CL: Please decide what to do.  As I said, I would change the SID_SHOWPOPUPS to
the internal methods hide/showChildWindows(), while changing the if statement
there.

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to