<div dir="ltr">The gerrit url<div><a href="https://gerrit.libreoffice.org/3761">https://gerrit.libreoffice.org/3761</a><br></div><div style>To pull</div><div style>git pull git://<a href="http://gerrit.libreoffice.org/core">gerrit.libreoffice.org/core</a> refs/changes/61/3761/1<br>

</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 3, 2013 at 5:13 PM, Manik Malhotra <span dir="ltr"><<a href="mailto:manik10044@iiitd.ac.in" target="_blank">manik10044@iiitd.ac.in</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">This is the patch pertaining to bug 56995. I did this as a qualification requirement to apply for GSoc in libreoffice. Name : Manik Malhotra<div>

The message follows:<div><div>fdo#56995 Created global option to disable presenter console in Impress Slideshow</div>
<div><br></div><div>Changes made are as Follows</div><div><ol><li>./officecfg/registry/schema/org/openoffice/Office/Impress.xcs</li><ul><li>To add the new global checkbox to Libreoffice Impress>General>Start</li>
<li>A new xml block for EnablePresenterScreen was added</li><li><prop oor:name="EnablePresenterScreen" oor:type="xs:boolean" ></li></ul><li>/sd/source/ui/app/optsitem.cxx</li><ul>
<li>To add the setter/getter for the new checkbox button, namely:<br></li></ul><ol><ul><li>SetEnablePresenterScreen<br></li><li>IsEnablePresenterScreen<br></li></ul></ol><ul><li>EnablePresenterScreen added to index number 26 of:<br>


</li></ul><ol><ul><li>GetPropNameArray()<br></li></ul></ol><li>/sd/source/ui/dlg/tpoption.cxx</li><ul><li>SdTpOptionsMisc() editted to attach EnablePresenterScreen to CBX_ENABLE_PRESENTER_SCREEN<br>
</li></ul><ol><ul><li>aCbxEnablePresenterScreen  ( this, SdResId( CBX_ENABLE_PRESENTER_SCREEN))<br></li></ul></ol><ul><li>Also Added to FillItemSet() and Reset() functions</li></ul><li>
/sd/source/ui/dlg/tpoption.src b/sd/source/ui/dlg/tpoption.src</li><ul><li>enableSdRemote size changed to</li><li>Size = MAP_APPFONT ( 116 , 10 ) ;</li><li>To allow enableSdRemote(CBX_ENABLE_SDREMOTE) and enablePresenterScreen( CBX_ENABLE_PRESENTER_SCREEN) to be drawn adjacent.</li>


<li>enablePresenterScreen checkbox CBX_ENABLE_PRESENTER_SCREEN added</li></ul><li>/sd/source/ui/inc/optsitem.hxx</li><ul><li>Set default value for enablePresenterScreen as 1 or TRUE</li><ul><li>
sal_Bool    bEnablePresenterScreen : 1;</li></ul><li>Added getter/setter named same as above for interface definition</li></ul><li>sd/source/ui/inc/tpoption.hrc</li><ul><li>Defined CBX_ENABLE_PRESENTER_SCREEN as 42nd checkbox</li>


<ul><li>#define CBX_ENABLE_PRESENTER_SCREEN 42</li></ul></ul><li>/sd/source/ui/inc/tpoption.hxx</li><ul><li> Added interface declaration of aCbxEnablePresenterScreen which was attached to  CBX_ENABLE_PRESENTER_SCREEN</li>


</ul><li>/sdext/source/presenter/PresenterScreen.hxx</li><ul><li> Added interface declaration of</li><li>bool isPresenterScreenEnabled()</li><li> to PresenterScreen class to allow presenterscreen to get if the option is checked</li>


</ul><li>/sdext/source/presenter/PresenterScreen.cxx</li><ul><li> Added definition of<br></li><li> isPresenterScreenEnabled(const css::uno::Reference<css::uno::XComponentContext>& rxContext)</li>
<ul><li>Input : Current Process Context</li><li>Output : The 0/1 value of EnablePresenterScreen from presenterconfig</li></ul><li>In PresenterScreenListener::notifyEvent( )</li><ul><li>
Added</li><li> if(mpPresenterScreen->isPresenterScreenEnabled(mxComponentContext)</li><li>mpPresenterScreen->InitializePresenterScreen()</li></ul><li> To initialize PresenterScreen only when enabled</li><span class="HOEnZb"><font color="#888888">
</font></span></ul></ol></div><span class="HOEnZb"><font color="#888888"><div><br></div><br>-- <br>Manik Malhotra<br><div><br></div>
</font></span></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Manik Malhotra<br><div><font color="#ff0000"><Space For Ad><br></font><br></div>
</div>