diff --git a/apps/common/main/resources/less/header.less b/apps/common/main/resources/less/header.less index a01e118a0..5bd584d65 100644 --- a/apps/common/main/resources/less/header.less +++ b/apps/common/main/resources/less/header.less @@ -362,8 +362,8 @@ border-radius: 20px; background-color: @icon-toolbar-header-ie; background-color: @icon-toolbar-header; - color: @toolbar-header-text-on-background-document-ie; - color: @toolbar-header-text-on-background-document; + color: @toolbar-header-text-on-background-ie; + color: @toolbar-header-text-on-background; font-size: 10px; line-height: 20px; overflow: hidden; diff --git a/apps/documenteditor/main/resources/less/variables.less b/apps/documenteditor/main/resources/less/variables.less index e31d64a73..a0658b824 100644 --- a/apps/documenteditor/main/resources/less/variables.less +++ b/apps/documenteditor/main/resources/less/variables.less @@ -3,8 +3,8 @@ // -------------------------------------------------- @header-background-color-ie: @toolbar-header-document-ie; @header-background-color: var(--toolbar-header-document); -@toolbar-header-text-on-background-document-ie: @text-toolbar-header-on-background-document-ie; -@toolbar-header-text-on-background-document: var(--text-toolbar-header-on-background-document); +@toolbar-header-text-on-background-ie: @text-toolbar-header-on-background-document-ie; +@toolbar-header-text-on-background: var(--text-toolbar-header-on-background-document); // Active color // ------------------------- diff --git a/apps/presentationeditor/main/app/controller/LeftMenu.js b/apps/presentationeditor/main/app/controller/LeftMenu.js index 3be667173..5c847f570 100644 --- a/apps/presentationeditor/main/app/controller/LeftMenu.js +++ b/apps/presentationeditor/main/app/controller/LeftMenu.js @@ -62,7 +62,6 @@ define([ 'hide': _.bind(this.onHideChat, this) }, 'Common.Views.Header': { - 'file:settings': _.bind(this.clickToolbarSettings,this), 'history:show': function () { if ( !this.leftMenu.panelHistory.isVisible() ) this.clickMenuFileItem('header', 'history'); diff --git a/apps/presentationeditor/main/app/controller/Main.js b/apps/presentationeditor/main/app/controller/Main.js index 72428694d..b71acf9fe 100644 --- a/apps/presentationeditor/main/app/controller/Main.js +++ b/apps/presentationeditor/main/app/controller/Main.js @@ -2338,6 +2338,7 @@ define([ this.api.asc_coAuthoringDisconnect(); appHeader.setCanRename(false); appHeader.getButton('users') && appHeader.getButton('users').hide(); + appHeader.getButton('share') && appHeader.getButton('share').setVisible(false); this.getApplication().getController('LeftMenu').getView('LeftMenu').showHistory(); this.disableEditing(true); this._renameDialog && this._renameDialog.close(); diff --git a/apps/presentationeditor/main/app/controller/Statusbar.js b/apps/presentationeditor/main/app/controller/Statusbar.js index 12281714c..9ccc8ae48 100644 --- a/apps/presentationeditor/main/app/controller/Statusbar.js +++ b/apps/presentationeditor/main/app/controller/Statusbar.js @@ -61,9 +61,6 @@ define([ 'Statusbar': { 'langchanged': this.onLangMenu }, - 'Common.Views.Header': { - 'statusbar:hide': _.bind(me.onChangeCompactView, me) - }, 'ViewTab': { 'statusbar:hide': _.bind(me.onChangeCompactView, me) } diff --git a/apps/presentationeditor/main/app/controller/Toolbar.js b/apps/presentationeditor/main/app/controller/Toolbar.js index e07d4939a..01b1da671 100644 --- a/apps/presentationeditor/main/app/controller/Toolbar.js +++ b/apps/presentationeditor/main/app/controller/Toolbar.js @@ -141,7 +141,6 @@ define([ 'menu:show': this.onFileMenu.bind(this, 'show') }, 'Common.Views.Header': { - 'toolbar:setcompact': this.onChangeCompactView.bind(this), 'print': function (opts) { var _main = this.getApplication().getController('Main'); _main.onPrint(); diff --git a/apps/presentationeditor/main/app/controller/ViewTab.js b/apps/presentationeditor/main/app/controller/ViewTab.js index 9fedc8a61..1fb0b495b 100644 --- a/apps/presentationeditor/main/app/controller/ViewTab.js +++ b/apps/presentationeditor/main/app/controller/ViewTab.js @@ -69,6 +69,7 @@ define([ if (api) { this.api = api; this.api.asc_registerCallback('asc_onZoomChange', _.bind(this.onZoomChange, this)); + this.api.asc_registerCallback('asc_onNotesShow', _.bind(this.onNotesShow, this)); } return this; }, @@ -84,12 +85,25 @@ define([ if (mode.canBrandingExt && mode.customization && mode.customization.statusBar === false || !Common.UI.LayoutManager.isElementVisible('statusBar')) { this.view.chStatusbar.$el.remove(); } + + if (!mode.isEdit) { // if view tab will be visible in view/restricted-editing mode + this.view.chToolbar.hide(); + var me = this; + Common.NotificationCenter.on('tab:visible', _.bind(function(action, visible){ + if ((action=='plugins' || action=='review') && visible) { + me.view.chToolbar.show(); + } + }, this)); + + this.view.chRulers.hide(); + } + this.addListeners({ 'ViewTab': { 'zoom:toslide': _.bind(this.onBtnZoomTo, this, 'toslide'), 'zoom:towidth': _.bind(this.onBtnZoomTo, this, 'towidth'), 'rulers:change': _.bind(this.onChangeRulers, this), - 'notes:change': _.bind(this.onChangeNotes, this), + 'notes:change': _.bind(this.onChangeNotes, this) }, 'Toolbar': { 'view:compact': _.bind(function (toolbar, state) { @@ -100,14 +114,6 @@ define([ 'view:hide': _.bind(function (statusbar, state) { this.view.chStatusbar.setValue(!state, true); }, this) - }, - 'Common.Views.Header': { - 'rulers:hide': _.bind(function (isChecked) { - this.view.chRulers.setValue(!isChecked, true); - }, this), - 'notes:hide': _.bind(function (isChecked) { - this.view.chNotes.setValue(!isChecked, true); - }, this), } }); }, @@ -137,6 +143,11 @@ define([ } }, + onNotesShow: function(bIsShow) { + this.view.chNotes.setValue(bIsShow, true); + Common.localStorage.setBool('pe-hidden-notes', !bIsShow); + }, + onAppReady: function (config) { var me = this; if (me.view) { @@ -186,7 +197,6 @@ define([ this.api.asc_SetViewRulers(checked); Common.localStorage.setBool('pe-hidden-rulers', !checked); Common.Utils.InternalSettings.set("pe-hidden-rulers", !checked); - this.view.fireEvent('rulers:hide', [!checked]); Common.NotificationCenter.trigger('layout:changed', 'rulers'); Common.NotificationCenter.trigger('edit:complete', this.view); }, @@ -194,7 +204,6 @@ define([ onChangeNotes: function (btn, checked) { this.api.asc_ShowNotes(checked); Common.localStorage.setBool('pe-hidden-notes', !checked); - this.view.fireEvent('notes:hide', [!checked]); Common.NotificationCenter.trigger('edit:complete', this.view); }, diff --git a/apps/presentationeditor/main/app/controller/Viewport.js b/apps/presentationeditor/main/app/controller/Viewport.js index b79d21b1d..357d30f0f 100644 --- a/apps/presentationeditor/main/app/controller/Viewport.js +++ b/apps/presentationeditor/main/app/controller/Viewport.js @@ -81,7 +81,6 @@ define([ toolbar.setExtra('left', me.header.getPanel('left', config)); }, 'view:compact' : function (toolbar, state) { - me.header.mnuitemCompactToolbar.setChecked(state, true); me.viewport.vlayout.getItem('toolbar').height = state ? Common.Utils.InternalSettings.get('toolbar-height-compact') : Common.Utils.InternalSettings.get('toolbar-height-normal'); }, @@ -107,21 +106,6 @@ define([ if ( me.header.btnSave ) me.header.btnSave.setDisabled(state); } - }, - // Events generated by main view - 'Viewport': { - - }, - 'ViewTab': { - 'rulers:hide': function (state) { - me.header.mnuitemHideRulers.setChecked(state, true); - }, - 'notes:hide': function (state) { - me.header.mnuitemHideNotes.setChecked(state, true); - }, - 'statusbar:hide': function (view, state) { - me.header.mnuitemHideStatusBar.setChecked(state, true); - } } }); Common.NotificationCenter.on('preview:start', this.onPreviewStart.bind(this)); @@ -129,9 +113,7 @@ define([ setApi: function(api) { this.api = api; - this.api.asc_registerCallback('asc_onZoomChange', this.onApiZoomChange.bind(this)); this.api.asc_registerCallback('asc_onCoAuthoringDisconnect',this.onApiCoAuthoringDisconnect.bind(this)); - this.api.asc_registerCallback('asc_onNotesShow', this.onNotesShow.bind(this)); Common.NotificationCenter.on('api:disconnect', this.onApiCoAuthoringDisconnect.bind(this)); }, @@ -168,9 +150,6 @@ define([ this.boxSdk = $('#editor_sdk'); this.boxSdk.css('border-left', 'none'); - this.header.mnuitemFitPage = this.header.fakeMenuItem(); - this.header.mnuitemFitWidth = this.header.fakeMenuItem(); - Common.NotificationCenter.on('app:face', this.onAppShowed.bind(this)); Common.NotificationCenter.on('app:ready', this.onAppReady.bind(this)); }, @@ -222,126 +201,6 @@ define([ }, onAppReady: function (config) { - var me = this; - if ( me.header.btnOptions ) { - var compactview = !config.isEdit; - if ( config.isEdit ) { - if ( Common.localStorage.itemExists("pe-compact-toolbar") ) { - compactview = Common.localStorage.getBool("pe-compact-toolbar"); - } else - if ( config.customization && config.customization.compactToolbar ) - compactview = true; - } - - me.header.mnuitemCompactToolbar = new Common.UI.MenuItem({ - caption: me.header.textCompactView, - checked: compactview, - checkable: true, - value: 'toolbar' - }); - if (!config.isEdit) { - me.header.mnuitemCompactToolbar.hide(); - Common.NotificationCenter.on('tab:visible', _.bind(function(action, visible){ - if ((action=='plugins' || action=='review') && visible) { - me.header.mnuitemCompactToolbar.show(); - } - }, this)); - } - - me.header.mnuitemHideStatusBar = new Common.UI.MenuItem({ - caption: me.header.textHideStatusBar, - checked: Common.localStorage.getBool("pe-hidden-status"), - checkable: true, - value: 'statusbar' - }); - - if ( config.canBrandingExt && config.customization && config.customization.statusBar === false || !Common.UI.LayoutManager.isElementVisible('statusBar')) - me.header.mnuitemHideStatusBar.hide(); - - me.header.mnuitemHideRulers = new Common.UI.MenuItem({ - caption: me.header.textHideLines, - checked: Common.Utils.InternalSettings.get("pe-hidden-rulers"), - checkable: true, - value: 'rulers' - }); - if (!config.isEdit) - me.header.mnuitemHideRulers.hide(); - - me.header.mnuitemHideNotes = new Common.UI.MenuItem({ - caption: me.header.textHideNotes, - checked: Common.localStorage.getBool('pe-hidden-notes', config.customization && config.customization.hideNotes===true), - checkable: true, - value: 'notes' - }); - - me.header.mnuitemFitPage = new Common.UI.MenuItem({ - caption: me.textFitPage, - checkable: true, - checked: me.header.mnuitemFitPage.isChecked(), - value: 'zoom:page' - }); - - me.header.mnuitemFitWidth = new Common.UI.MenuItem({ - caption: me.textFitWidth, - checkable: true, - checked: me.header.mnuitemFitWidth.isChecked(), - value: 'zoom:width' - }); - - me.header.mnuZoom = new Common.UI.MenuItem({ - template: _.template([ - '' - ].join('')), - stopPropagation: true, - value: me.header.mnuZoom.options.value - }); - - me.header.btnOptions.setMenu(new Common.UI.Menu({ - cls: 'pull-right', - style: 'min-width: 180px;', - items: [ - me.header.mnuitemCompactToolbar, - me.header.mnuitemHideStatusBar, - me.header.mnuitemHideRulers, - me.header.mnuitemHideNotes, - {caption:'--'}, - me.header.mnuitemFitPage, - me.header.mnuitemFitWidth, - me.header.mnuZoom, - {caption:'--'}, - new Common.UI.MenuItem({ - caption: me.header.textAdvSettings, - value: 'advanced' - }) - ] - }) - ); - - var _on_btn_zoom = function (btn) { - btn == 'up' ? me.api.zoomIn() : me.api.zoomOut(); - Common.NotificationCenter.trigger('edit:complete', me.header); - }; - - (new Common.UI.Button({ - el : $('#hdr-menu-zoom-out', me.header.mnuZoom.$el), - cls : 'btn-toolbar' - })).on('click', _on_btn_zoom.bind(me, 'down')); - - (new Common.UI.Button({ - el : $('#hdr-menu-zoom-in', me.header.mnuZoom.$el), - cls : 'btn-toolbar' - })).on('click', _on_btn_zoom.bind(me, 'up')); - - me.header.btnOptions.menu.on('item:click', me.onOptionsItemClick.bind(this)); - } }, onLayoutChanged: function(area) { @@ -446,51 +305,9 @@ define([ me.header.lockHeaderBtns( 'undo', _need_disable ); me.header.lockHeaderBtns( 'redo', _need_disable ); - me.header.lockHeaderBtns( 'opts', _need_disable ); me.header.lockHeaderBtns( 'users', _need_disable ); }, - onApiZoomChange: function(percent, type) { - this.header.mnuitemFitPage.setChecked(type == 2, true); - this.header.mnuitemFitWidth.setChecked(type == 1, true); - this.header.mnuZoom.options.value = percent; - - if ( this.header.mnuZoom.$el ) - $('.menu-zoom label.zoom', this.header.mnuZoom.$el).html(percent + '%'); - }, - - onOptionsItemClick: function (menu, item, e) { - var me = this; - - switch ( item.value ) { - case 'toolbar': me.header.fireEvent('toolbar:setcompact', [menu, item.isChecked()]); break; - case 'statusbar': me.header.fireEvent('statusbar:hide', [item, item.isChecked()]); break; - case 'rulers': - me.api.asc_SetViewRulers(!item.isChecked()); - Common.localStorage.setBool('pe-hidden-rulers', item.isChecked()); - Common.Utils.InternalSettings.set("pe-hidden-rulers", item.isChecked()); - Common.NotificationCenter.trigger('layout:changed', 'rulers'); - Common.NotificationCenter.trigger('edit:complete', me.header); - me.header.fireEvent('rulers:hide', [item.isChecked()]); - break; - case 'notes': - me.api.asc_ShowNotes(!item.isChecked()); - Common.localStorage.setBool('pe-hidden-notes', item.isChecked()); - Common.NotificationCenter.trigger('edit:complete', me.header); - me.header.fireEvent('notes:hide', [item.isChecked()]); - break; - case 'zoom:page': - item.isChecked() ? me.api.zoomFitToPage() : me.api.zoomCustomMode(); - Common.NotificationCenter.trigger('edit:complete', me.header); - break; - case 'zoom:width': - item.isChecked() ? me.api.zoomFitToWidth() : me.api.zoomCustomMode(); - Common.NotificationCenter.trigger('edit:complete', me.header); - break; - case 'advanced': me.header.fireEvent('file:settings', me.header); break; - } - }, - onApiCoAuthoringDisconnect: function(enableDownload) { if (this.header) { if (this.header.btnDownload && !enableDownload) @@ -507,11 +324,6 @@ define([ this.header && this.header.lockHeaderBtns( 'rename-user', disable); }, - onNotesShow: function(bIsShow) { - this.header && this.header.mnuitemHideNotes.setChecked(!bIsShow, true); - Common.localStorage.setBool('pe-hidden-notes', !bIsShow); - }, - textFitPage: 'Fit to Page', textFitWidth: 'Fit to Width' }, PE.Controllers.Viewport)); diff --git a/apps/presentationeditor/main/index.html b/apps/presentationeditor/main/index.html index c8b8c07bd..d97ecfad9 100644 --- a/apps/presentationeditor/main/index.html +++ b/apps/presentationeditor/main/index.html @@ -75,6 +75,15 @@ background: rgba(255, 255, 255, 0.2); } + .loadmask > .brendpanel .circle { + vertical-align: middle; + width: 20px; + height: 20px; + border-radius: 20px; + margin: 0 10px; + background: rgba(255,255,255,.2); + } + .loadmask > .sktoolbar { background: #f7f7f7; background: var(--background-toolbar, #f7f7f7); @@ -273,7 +282,7 @@