From 631cea5e0683c472ec5070f9f73f628def1fa259 Mon Sep 17 00:00:00 2001 From: Julia Radzhabova Date: Mon, 10 Feb 2020 20:19:09 +0300 Subject: [PATCH] [SSE mobile] Disable cell editing in opensource version --- .../mobile/app/controller/edit/EditCell.js | 104 ++++++------------ 1 file changed, 32 insertions(+), 72 deletions(-) diff --git a/apps/spreadsheeteditor/mobile/app/controller/edit/EditCell.js b/apps/spreadsheeteditor/mobile/app/controller/edit/EditCell.js index b34602d99..c603a4903 100644 --- a/apps/spreadsheeteditor/mobile/app/controller/edit/EditCell.js +++ b/apps/spreadsheeteditor/mobile/app/controller/edit/EditCell.js @@ -51,29 +51,8 @@ define([ 'use strict'; SSE.Controllers.EditCell = Backbone.Controller.extend(_.extend((function() { - var _fontsArray = [], - _stack = [], - _cellInfo = undefined, - _cellStyles = [], - _fontInfo = {}, - _borderInfo = {color: '000000', width: Asc.c_oAscBorderStyles.Medium}, - _styleSize = {width: 100, height: 50}, - _isEdit = false; - - function onApiLoadFonts(fonts, select) { - _.each(fonts, function(font){ - var fontId = font.asc_getFontId(); - _fontsArray.push({ - id : fontId, - name : font.asc_getFontName(), -// displayValue: font.asc_getFontName(), - imgidx : font.asc_getFontThumbnail(), - type : font.asc_getFontType() - }); - }); - - Common.NotificationCenter.trigger('fonts:load', _fontsArray, select); - } + var _stack = [], + _borderInfo = {color: '000000', width: Asc.c_oAscBorderStyles.Medium}; return { models: [], @@ -92,22 +71,26 @@ define([ 'style:click' : this.onStyleClick } }); + this._fontsArray = []; + this._styleSize = {width: 100, height: 50}; + this._cellStyles = []; + this._cellInfo = undefined; + this._fontInfo = {}; + this._isEdit = false; }, setApi: function (api) { var me = this; me.api = api; - me.api.asc_setThumbnailStylesSizes(_styleSize.width, _styleSize.height); + me.api.asc_setThumbnailStylesSizes(me._styleSize.width, me._styleSize.height); - me.api.asc_registerCallback('asc_onInitEditorFonts', _.bind(onApiLoadFonts, me)); me.api.asc_registerCallback('asc_onSelectionChanged', _.bind(me.onApiSelectionChanged, me)); me.api.asc_registerCallback('asc_onEditorSelectionChanged', _.bind(me.onApiEditorSelectionChanged, me)); - me.api.asc_registerCallback('asc_onInitEditorStyles', _.bind(me.onApiInitEditorStyles, me)); }, setMode: function (mode) { - _isEdit = mode.isEdit; + this._isEdit = mode.isEdit; }, onLaunch: function () { @@ -125,13 +108,11 @@ define([ $('#font-italic').single('click', _.bind(me.onItalic, me)); $('#font-underline').single('click', _.bind(me.onUnderline, me)); - me.getView('EditCell').renderStyles(_cellStyles); + me.getView('EditCell').renderStyles(me._cellStyles); me.initSettings(); }, - - onPageShow: function (view, pageId) { var me = this; @@ -156,14 +137,14 @@ define([ } else if (!_.isUndefined(pageId) && pageId.indexOf('#edit-cell-format') > -1) { me.initCellFormat(); } else { - me.initCellSettings(_cellInfo); + me.initCellSettings(me._cellInfo); } }, // Public getFonts: function() { - return _fontsArray; + return this._fontsArray; }, getStack: function() { @@ -171,20 +152,20 @@ define([ }, getFontInfo: function () { - return _fontInfo; + return this._fontInfo; }, getCell: function () { - return _cellInfo; + return this._cellInfo; }, getStyleSize: function () { - return _styleSize; + return this._styleSize; }, initFontsPage: function () { var me = this, - displaySize = _fontInfo.size; + displaySize = this._fontInfo.size; _.isUndefined(displaySize) ? displaySize = this.textAuto : displaySize = displaySize + ' ' + this.textPt; @@ -195,7 +176,7 @@ define([ initTextColorPage: function () { var me = this, palette = me.getView('EditCell').paletteTextColor, - color = me._sdkToThemeColor(_fontInfo.color); + color = me._sdkToThemeColor(this._fontInfo.color); if (palette) { palette.select(color); @@ -204,9 +185,11 @@ define([ }, initFillColorPage: function () { + if (_.isUndefined(this._cellInfo)) return; + var me = this, palette = me.getView('EditCell').paletteFillColor, - color = me._sdkToThemeColor(_cellInfo.asc_getFill().asc_getColor()); + color = me._sdkToThemeColor(me._cellInfo.asc_getFill().asc_getColor()); if (palette) { palette.select(color); @@ -228,13 +211,15 @@ define([ }, initTextFormat: function () { + if (_.isUndefined(this._cellInfo)) return; + var me = this, $pageTextFormat = $('.page[data-page=edit-text-format]'), - hAlign = _cellInfo.asc_getHorAlign(), - vAlign = _cellInfo.asc_getVertAlign(), + hAlign = me._cellInfo.asc_getHorAlign(), + vAlign = me._cellInfo.asc_getVertAlign(), hAlignStr = 'left', vAlignStr = 'bottom', - isWrapText = _cellInfo.asc_getFlags().asc_getWrapText(); + isWrapText = me._cellInfo.asc_getFlags().asc_getWrapText(); if (vAlign == Asc.c_oAscVAlign.Top) vAlignStr = 'top'; @@ -294,7 +279,7 @@ define([ // Init font name var fontName = fontObj.asc_getName() || this.textFonts; - _fontInfo.name = fontName; + this._fontInfo.name = fontName; $('#font-fonts .item-title').html(fontName); @@ -306,8 +291,8 @@ define([ // Init font size - _fontInfo.size = fontObj.asc_getSize(); - var displaySize = _fontInfo.size; + this._fontInfo.size = fontObj.asc_getSize(); + var displaySize = this._fontInfo.size; _.isUndefined(displaySize) ? displaySize = this.textAuto : displaySize = displaySize + ' ' + this.textPt; @@ -316,9 +301,9 @@ define([ // Init font color - _fontInfo.color = fontObj.asc_getColor(); + this._fontInfo.color = fontObj.asc_getColor(); - var color = _fontInfo.color, + var color = this._fontInfo.color, clr = me._sdkToThemeColor(color); $('#text-color .color-preview').css('background-color', '#' + (_.isObject(clr) ? clr.color : clr)); @@ -356,21 +341,12 @@ define([ me.initTextFormat(); }, - onApiInitEditorStyles: function(styles){ - window.styles_loaded = false; - _cellStyles = styles; - - this.getView('EditCell').renderStyles(styles); - - window.styles_loaded = true; - }, - // Handlers onFontSize: function (e) { var me = this, $button = $(e.currentTarget), - fontSize = _fontInfo.size; + fontSize = this._fontInfo.size; if ($button.hasClass('decrement')) { _.isUndefined(fontSize) ? me.api.asc_decreaseFontSize() : fontSize = Math.max(1, --fontSize); @@ -464,10 +440,6 @@ define([ }, onCellFormat: function (e) { - var $target = $(e.currentTarget), - type = decodeURIComponent(atob($target.data('type'))); - - this.api.asc_setCellFormat(type); }, onBorderStyle: function (e) { @@ -504,21 +476,9 @@ define([ // API handlers onApiEditorSelectionChanged: function(fontObj) { - if (!_isEdit) { - return; - } - - _fontInfo = fontObj; - this.initFontSettings(fontObj); }, onApiSelectionChanged: function(cellInfo) { - if (!_isEdit) { - return; - } - - _cellInfo = cellInfo; - this.initCellSettings(cellInfo); }, // Helpers