Hi everyone, <div><br></div><div>I'm new to this list. I've made a simple fix for <a href="https://bugs.freedesktop.org/show_bug.cgi?id=38276">Bug 38276</a> . The diff is below if anyone wants to have a look (I wasn't sure if I should add it as an attachment or not?). Bear in mind that I'm completely new to LibreOffice development and open source development in general.</div>
<div><br></div><div><div><div>diff --git a/framework/source/uielement/toolbarmanager.cxx b/framework/source/uielement/toolbarmanager.cxx</div><div>index 80adbd7..ecce13f 100644</div><div>--- a/framework/source/uielement/toolbarmanager.cxx</div>
<div>+++ b/framework/source/uielement/toolbarmanager.cxx</div><div>@@ -1817,6 +1817,22 @@ PopupMenu * ToolBarManager::GetToolBarCustomMenu(ToolBox* pToolBar)</div><div> aPopupMenu.RemoveItem( nPos );</div><div>
}</div><div> </div><div>+ sal_uInt16 nObsoleteItems = 6;</div><div>+ sal_uInt16 positionInMenu;</div><div>+ sal_uInt32 obsoleteItems[] = { MENUITEM_TOOLBAR_CLOSE , </div><div>+ MENUITEM_TOOLBAR_VISIBLEBUTTON , </div>
<div>+ MENUITEM_TOOLBAR_CUSTOMIZETOOLBAR,</div><div>+ MENUITEM_TOOLBAR_LOCKTOOLBARPOSITION , </div><div>+ MENUITEM_TOOLBAR_DOCKTOOLBAR , </div>
<div>+ MENUITEM_TOOLBAR_DOCKALLTOOLBAR </div><div>+ };</div><div>+ for( int i = 0 ; i < nObsoleteItems ; i++ )</div><div>
+ {</div><div>+ positionInMenu = aPopupMenu.GetItemPos( obsoleteItems[i] );</div><div>+ if ( positionInMenu != MENU_ITEM_NOTFOUND )</div><div>+ aPopupMenu.RemoveItem( positionInMenu );</div><div>
+ }</div><div>+ </div><div> // copy all menu items to the toolbar menu</div><div> if( pMenu->GetItemCount() )</div><div> pMenu->InsertSeparator();</div></div></div><div><br></div><div>Thanks, </div>
<div><br></div><div>Iain.</div>