Commits

Caolán McNamara  committed a46df95

cmcfixes79: #i114480#: use radio buttons for mutually exclusive options

  • Participants
  • Parent commits 5893d50

Comments (0)

Files changed (10)

File framework/source/classes/menumanager.cxx

 	{
 		USHORT nItemId = FillItemCommand(aItemCommand,pMenu, i );
         bool bShowMenuImages( m_bShowMenuImages );
-        MenuItemBits nBits =  pMenu->GetItemBits( nItemId );
-        // overwrite the default?
-        if ( nBits )
+
+        // overwrite the show icons on menu option?
+        if (!bShowMenuImages)
+        {
+            MenuItemBits nBits = pMenu->GetItemBits( nItemId );
             bShowMenuImages = ( ( nBits & MIB_ICON ) == MIB_ICON );
-
+        }
 
 		PopupMenu* pPopupMenu = pMenu->GetPopupMenu( nItemId );
 		if ( pPopupMenu )
 		if ( _pMenu->GetItemType( nPos ) != MENUITEM_SEPARATOR )
 		{
             bool bTmpShowMenuImages( bShowMenuImages );
-            MenuItemBits nBits =  _pMenu->GetItemBits( nId );
-            // overwrite the default?
-            if ( nBits )
+            // overwrite the show icons on menu option?
+            if (!bTmpShowMenuImages)
+            {
+                MenuItemBits nBits =  _pMenu->GetItemBits( nId );
                 bTmpShowMenuImages = ( ( nBits & MIB_ICON ) == MIB_ICON );
+            }
 
             if ( bTmpShowMenuImages )
 			{

File framework/source/uielement/menubarmanager.cxx

 		Reference< XStatusListener > xStatusListener;
 		PopupMenu* pPopup = pMenu->GetPopupMenu( nItemId );
         bool bItemShowMenuImages = m_bShowMenuImages;
-        MenuItemBits nBits =  pMenu->GetItemBits( nItemId );
         // overwrite the show icons on menu option?
-        if ( nBits )
+        if (!bItemShowMenuImages)
+        {
+            MenuItemBits nBits =  pMenu->GetItemBits( nItemId );
             bItemShowMenuImages = ( ( nBits & MIB_ICON ) == MIB_ICON );
+        }
 		if ( pPopup )
 		{
             // Retrieve module identifier from Help Command entry

File officecfg/registry/data/org/openoffice/Office/UI/CalcCommands.xcu

 					<value xml:lang="en-US">~Trace Dependents</value>
 				</prop>
 			</node>
+			<node oor:name=".uno:SinglePane" oor:op="replace">
+				<prop oor:name="Label" oor:type="xs:string">
+					<value xml:lang="en-US">~Normal Pane</value>
+				</prop>
+				<prop oor:name="ContextLabel" oor:type="xs:string">
+					<value xml:lang="en-US">~Normal</value>
+				</prop>
+			</node>
 			<node oor:name=".uno:SplitWindow" oor:op="replace">
 				<prop oor:name="Label" oor:type="xs:string">
 					<value xml:lang="en-US">~Split Window</value>

File sd/uiconfig/simpress/menubar/menubar.xml

 			<menu:menuitem menu:id=".uno:NormalMultiPaneGUI" menu:style="radio"/>
             <menu:menuitem menu:id=".uno:OutlineMode" menu:style="radio"/>
 			<menu:menuitem menu:id=".uno:DiaMode" menu:style="radio"/>
-			<menu:menuitem menu:id=".uno:Presentation"/>
+			<menu:menuitem menu:id=".uno:Presentation" menu:style="radio"/>
 			<menu:menuitem menu:id=".uno:NotesMode" menu:style="radio"/>
 			<menu:menuitem menu:id=".uno:HandoutMode" menu:style="radio"/>
    			<menu:menuseparator/>
 			<menu:menu menu:id=".uno:MasterPageMenu">
 				<menu:menupopup>
-					<menu:menuitem menu:id=".uno:SlideMasterPage"/>
-					<menu:menuitem menu:id=".uno:NotesMasterPage"/>
+					<menu:menuitem menu:id=".uno:SlideMasterPage" menu:style="radio"//>
+					<menu:menuitem menu:id=".uno:NotesMasterPage" menu:style="radio"//>
         			<menu:menuseparator/>
                		<menu:menuitem menu:id=".uno:MasterLayouts"/>
 				</menu:menupopup>

File sw/uiconfig/sglobal/menubar/menubar.xml

   </menu:menu>
   <menu:menu menu:id=".uno:ViewMenu">
     <menu:menupopup>
-      <menu:menuitem menu:id=".uno:PrintLayout"/>
-      <menu:menuitem menu:id=".uno:BrowseView"/>
+      <menu:menuitem menu:id=".uno:PrintLayout" menu:style="radio"/>
+      <menu:menuitem menu:id=".uno:BrowseView" menu:style="radio"/>
       <menu:menuseparator/>
       <menu:menuitem menu:id=".uno:AvailableToolbars"/>
       <menu:menuitem menu:id=".uno:StatusBarVisible"/>

File sw/uiconfig/sweb/menubar/menubar.xml

   </menu:menu>
   <menu:menu menu:id=".uno:ViewMenu">
     <menu:menupopup>
-      <menu:menuitem menu:id=".uno:PrintLayout"/>
-      <menu:menuitem menu:id=".uno:BrowseView"/>
-      <menu:menuitem menu:id=".uno:SourceView"/>
+      <menu:menuitem menu:id=".uno:PrintLayout" menu:style="radio"/>
+      <menu:menuitem menu:id=".uno:BrowseView" menu:style="radio"/>
+      <menu:menuitem menu:id=".uno:SourceView" menu:style="radio"/>
       <menu:menuseparator/>
       <menu:menuitem menu:id=".uno:AvailableToolbars"/>
       <menu:menuitem menu:id=".uno:StatusBarVisible"/>

File sw/uiconfig/swform/menubar/menubar.xml

   </menu:menu>
   <menu:menu menu:id=".uno:ViewMenu">
     <menu:menupopup>
-      <menu:menuitem menu:id=".uno:PrintLayout"/>
-      <menu:menuitem menu:id=".uno:BrowseView"/>
+      <menu:menuitem menu:id=".uno:PrintLayout" menu:style="radio"/>
+      <menu:menuitem menu:id=".uno:BrowseView" menu:style="radio"/>
       <menu:menuseparator/>
       <menu:menuitem menu:id=".uno:AvailableToolbars"/>
       <menu:menuitem menu:id=".uno:StatusBarVisible"/>

File sw/uiconfig/swreport/menubar/menubar.xml

   </menu:menu>
   <menu:menu menu:id=".uno:ViewMenu">
     <menu:menupopup>
-      <menu:menuitem menu:id=".uno:PrintLayout"/>
-      <menu:menuitem menu:id=".uno:BrowseView"/>
+      <menu:menuitem menu:id=".uno:PrintLayout" menu:style="radio"/>
+      <menu:menuitem menu:id=".uno:BrowseView" menu:style="radio"/>
       <menu:menuseparator/>
       <menu:menuitem menu:id=".uno:AvailableToolbars"/>
       <menu:menuitem menu:id=".uno:StatusBarVisible"/>

File sw/uiconfig/swriter/menubar/menubar.xml

   </menu:menu>
   <menu:menu menu:id=".uno:ViewMenu">
     <menu:menupopup>
-      <menu:menuitem menu:id=".uno:PrintLayout"/>
-      <menu:menuitem menu:id=".uno:BrowseView"/>
+      <menu:menuitem menu:id=".uno:PrintLayout" menu:style="radio"/>
+      <menu:menuitem menu:id=".uno:BrowseView" menu:style="radio"/>
       <menu:menuseparator/>
       <menu:menuitem menu:id=".uno:AvailableToolbars"/>
       <menu:menuitem menu:id=".uno:StatusBarVisible"/>

File sw/uiconfig/swxform/menubar/menubar.xml

   </menu:menu>
   <menu:menu menu:id=".uno:ViewMenu">
     <menu:menupopup>
-      <menu:menuitem menu:id=".uno:PrintLayout"/>
-      <menu:menuitem menu:id=".uno:BrowseView"/>
+      <menu:menuitem menu:id=".uno:PrintLayout" menu:style="radio"/>
+      <menu:menuitem menu:id=".uno:BrowseView" menu:style="radio"/>
       <menu:menuseparator/>
       <menu:menuitem menu:id=".uno:AvailableToolbars"/>
       <menu:menuitem menu:id=".uno:StatusBarVisible"/>