Re: [PATCH][FORGOT TO ADD MESSAGE] fdo#56995: SLIDE SHOW: Presenter Console Can't be disabled.Original Message:Presentation Console - Need to turn this off.

The gerrit url
https://gerrit.libreoffice.org/3761
To pull
git pull git://gerrit.libreoffice.org/core refs/changes/61/3761/1


On Fri, May 3, 2013 at 5:13 PM, Manik Malhotra <manik10044@iiitd.ac.in>wrote:

> This is the patch pertaining to bug 56995. I did this as a qualification
> requirement to apply for GSoc in libreoffice. Name : Manik Malhotra
> The message follows:
> fdo#56995 Created global option to disable presenter console in Impress
> Slideshow
>
> Changes made are as Follows
>
>    1. ./officecfg/registry/schema/org/openoffice/Office/Impress.xcs
>       - To add the new global checkbox to Libreoffice
>       Impress>General>Start
>       - A new xml block for EnablePresenterScreen was added
>       - <prop oor:name="EnablePresenterScreen" oor:type="xs:boolean" >
>    2. /sd/source/ui/app/optsitem.cxx
>       - To add the setter/getter for the new checkbox button, namely:
>       - SetEnablePresenterScreen
>          - IsEnablePresenterScreen
>          - EnablePresenterScreen added to index number 26 of:
>       - GetPropNameArray()
>          3. /sd/source/ui/dlg/tpoption.cxx
>       - SdTpOptionsMisc() editted to attach EnablePresenterScreen to
>       CBX_ENABLE_PRESENTER_SCREEN
>       - aCbxEnablePresenterScreen  ( this, SdResId(
>          CBX_ENABLE_PRESENTER_SCREEN))
>          - Also Added to FillItemSet() and Reset() functions
>    4. /sd/source/ui/dlg/tpoption.src b/sd/source/ui/dlg/tpoption.src
>       - enableSdRemote size changed to
>       - Size = MAP_APPFONT ( 116 , 10 ) ;
>       - To allow enableSdRemote(CBX_ENABLE_SDREMOTE) and
>       enablePresenterScreen( CBX_ENABLE_PRESENTER_SCREEN) to be drawn adjacent.
>       - enablePresenterScreen checkbox CBX_ENABLE_PRESENTER_SCREEN added
>    5. /sd/source/ui/inc/optsitem.hxx
>       - Set default value for enablePresenterScreen as 1 or TRUE
>          - sal_Bool    bEnablePresenterScreen : 1;
>       - Added getter/setter named same as above for interface definition
>    6. sd/source/ui/inc/tpoption.hrc
>       - Defined CBX_ENABLE_PRESENTER_SCREEN as 42nd checkbox
>          - #define CBX_ENABLE_PRESENTER_SCREEN 42
>       7. /sd/source/ui/inc/tpoption.hxx
>       -  Added interface declaration of aCbxEnablePresenterScreen which
>       was attached to  CBX_ENABLE_PRESENTER_SCREEN
>    8. /sdext/source/presenter/PresenterScreen.hxx
>       -  Added interface declaration of
>       - bool isPresenterScreenEnabled()
>       -  to PresenterScreen class to allow presenterscreen to get if the
>       option is checked
>    9. /sdext/source/presenter/PresenterScreen.cxx
>       -  Added definition of
>       -  isPresenterScreenEnabled(const
>       css::uno::Reference<css::uno::XComponentContext>& rxContext)
>          - Input : Current Process Context
>          - Output : The 0/1 value of EnablePresenterScreen from
>          presenterconfig
>       - In PresenterScreenListener::notifyEvent( )
>          - Added
>          -
>           if(mpPresenterScreen->isPresenterScreenEnabled(mxComponentContext)
>          - mpPresenterScreen->InitializePresenterScreen()
>       -  To initialize PresenterScreen only when enabled
>
>
>
> --
> Manik Malhotra


-- 
Manik Malhotra
<Space For Ad>