diff --git a/apps/presentationeditor/main/app/view/DocumentHolder.js b/apps/presentationeditor/main/app/view/DocumentHolder.js index f694feb8c..64694a9a9 100644 --- a/apps/presentationeditor/main/app/view/DocumentHolder.js +++ b/apps/presentationeditor/main/app/view/DocumentHolder.js @@ -2564,11 +2564,20 @@ define([ menu : (function(){ function onItemClick(item) { if (me.api) { - me.api.put_ShapesAlign(item.value); + var value = me.api.asc_getSelectedDrawingObjectsCount()<2 || Common.Utils.InternalSettings.get("pe-align-to-slide"); + value = value ? Asc.c_oAscObjectsAlignType.Slide : Asc.c_oAscObjectsAlignType.Selected; + if (item.value < 6) { + me.api.put_ShapesAlign(item.value, value); + Common.component.Analytics.trackEvent('DocumentHolder', 'Shape Align'); + } else if (item.value == 6) { + me.api.DistributeHorizontally(value); + Common.component.Analytics.trackEvent('DocumentHolder', 'Distribute Horizontally'); + } else if (item.value == 7){ + me.api.DistributeVertically(value); + Common.component.Analytics.trackEvent('DocumentHolder', 'Distribute Vertically'); + } } - me.fireEvent('editcomplete', me); - Common.component.Analytics.trackEvent('DocumentHolder', 'Image Shape Align'); } return new Common.UI.Menu({ @@ -2607,26 +2616,14 @@ define([ {caption : '--'}, new Common.UI.MenuItem({ caption : me.txtDistribHor, - iconCls : 'mnu-distrib-hor' - }).on('click', function(item) { - if (me.api) { - me.api.DistributeHorizontally(); - } - - me.fireEvent('editcomplete', me); - Common.component.Analytics.trackEvent('DocumentHolder', 'Distribute Horizontally'); - }), + iconCls : 'mnu-distrib-hor', + value : 6 + }).on('click', _.bind(onItemClick, me)), new Common.UI.MenuItem({ caption : me.txtDistribVert, - iconCls : 'mnu-distrib-vert' - }).on('click', function(item) { - if (me.api) { - me.api.DistributeVertically(); - } - - me.fireEvent('editcomplete', me); - Common.component.Analytics.trackEvent('DocumentHolder', 'Distribute Vertically'); - }) + iconCls : 'mnu-distrib-vert', + value : 7 + }).on('click', _.bind(onItemClick, me)) ] }) })() @@ -3200,6 +3197,12 @@ define([ menuAddCommentImg.setDisabled(disabled); /** coauthoring end **/ menuImgShapeAlign.setDisabled(disabled); + if (!disabled) { + var objcount = me.api.asc_getSelectedDrawingObjectsCount(), + slide_checked = Common.Utils.InternalSettings.get("pe-align-to-slide") || false; + menuImgShapeAlign.menu.items[7].setDisabled(objcount==2 && !slide_checked); + menuImgShapeAlign.menu.items[8].setDisabled(objcount==2 && !slide_checked); + } menuImageAdvanced.setDisabled(disabled); menuShapeAdvanced.setDisabled(disabled); if (menuChartEdit.isVisible()) diff --git a/apps/presentationeditor/main/app/view/Toolbar.js b/apps/presentationeditor/main/app/view/Toolbar.js index fc7c1fb3b..285bc9210 100644 --- a/apps/presentationeditor/main/app/view/Toolbar.js +++ b/apps/presentationeditor/main/app/view/Toolbar.js @@ -1762,8 +1762,8 @@ define([ textTabCollaboration: 'Collaboration', textTabProtect: 'Protection', mniImageFromStorage: 'Image from Storage', - txtSlideAlign: 'Align to slide', - txtObjectsAlign: 'Align selected objects' + txtSlideAlign: 'Align to Slide', + txtObjectsAlign: 'Align Selected Objects' } }()), PE.Views.Toolbar || {})); }); \ No newline at end of file diff --git a/apps/presentationeditor/main/locale/en.json b/apps/presentationeditor/main/locale/en.json index d37510ab8..6894b2214 100644 --- a/apps/presentationeditor/main/locale/en.json +++ b/apps/presentationeditor/main/locale/en.json @@ -1747,7 +1747,6 @@ "PE.Views.Toolbar.txtScheme8": "Flow", "PE.Views.Toolbar.txtScheme9": "Foundry", "PE.Views.Toolbar.txtUngroup": "Ungroup", - "PE.Views.Toolbar.txtSlideAlign": "Align to slide", - "PE.Views.Toolbar.txtObjectsAlign": "Align selected objects" - + "PE.Views.Toolbar.txtSlideAlign": "Align to Slide", + "PE.Views.Toolbar.txtObjectsAlign": "Align Selected Objects" } \ No newline at end of file