From 30fd37cd24faec67d4f7f70632bdbcf37fa5bd24 Mon Sep 17 00:00:00 2001 From: ShimaginAndrey Date: Thu, 14 Oct 2021 12:53:14 +0300 Subject: [PATCH 1/7] [DE PE SSE] Fix Bug 53128 --- apps/common/mobile/resources/less/common-material.less | 2 ++ 1 file changed, 2 insertions(+) diff --git a/apps/common/mobile/resources/less/common-material.less b/apps/common/mobile/resources/less/common-material.less index 11df30aaa0..8631b30e97 100644 --- a/apps/common/mobile/resources/less/common-material.less +++ b/apps/common/mobile/resources/less/common-material.less @@ -427,9 +427,11 @@ .searchbar .input-clear-button { width: 18px; height: 18px; + margin-top: -8px; &:after { color: @white; font-size: 19px; + line-height: 19px; } } From a7a924ce28b882496ebc5fb62ea19a98222c9b24 Mon Sep 17 00:00:00 2001 From: ShimaginAndrey Date: Thu, 14 Oct 2021 16:44:53 +0300 Subject: [PATCH 2/7] [SSE] Fix Bug 53149 --- apps/spreadsheeteditor/mobile/locale/en.json | 1 + .../mobile/src/controller/ContextMenu.jsx | 17 +++++++++++++++++ 2 files changed, 18 insertions(+) diff --git a/apps/spreadsheeteditor/mobile/locale/en.json b/apps/spreadsheeteditor/mobile/locale/en.json index 14ca5cb962..0b89dd7a6a 100644 --- a/apps/spreadsheeteditor/mobile/locale/en.json +++ b/apps/spreadsheeteditor/mobile/locale/en.json @@ -41,6 +41,7 @@ }, "ContextMenu": { "errorCopyCutPaste": "Copy, cut, and paste actions using the context menu will be performed within the current file only.", + "errorInvalidLink": "The link reference does not exist. Please correct the link or delete it.", "menuAddComment": "Add Comment", "menuAddLink": "Add Link", "menuCancel": "Cancel", diff --git a/apps/spreadsheeteditor/mobile/src/controller/ContextMenu.jsx b/apps/spreadsheeteditor/mobile/src/controller/ContextMenu.jsx index 47212dde2e..f2a3377d6e 100644 --- a/apps/spreadsheeteditor/mobile/src/controller/ContextMenu.jsx +++ b/apps/spreadsheeteditor/mobile/src/controller/ContextMenu.jsx @@ -26,6 +26,7 @@ class ContextMenu extends ContextMenuController { this.onApiShowComment = this.onApiShowComment.bind(this); this.onApiHideComment = this.onApiHideComment.bind(this); this.getUserName = this.getUserName.bind(this); + this.onApiHyperlinkClick = this.onApiHyperlinkClick.bind(this); } static closeContextMenu() { @@ -43,6 +44,7 @@ class ContextMenu extends ContextMenuController { const api = Common.EditorApi.get(); api.asc_unregisterCallback('asc_onShowComment', this.onApiShowComment); api.asc_unregisterCallback('asc_onHideComment', this.onApiHideComment); + api.asc_unregisterCallback('asc_onHyperlinkClick', this.onApiHyperlinkClick); } @@ -58,6 +60,20 @@ class ContextMenu extends ContextMenuController { // super.onMenuClosed(); // } + onApiHyperlinkClick(url) { + const { t } = this.props; + + if(!url) { + f7.dialog.create({ + title: t('ContextMenu.notcriticalErrorTitle'), + text: t('ContextMenu.errorInvalidLink'), + buttons:[ + {text: 'OK'} + ] + }).open(); + } + } + onMenuItemClick(action) { const { t } = this.props; const _t = t("ContextMenu", { returnObjects: true }); @@ -175,6 +191,7 @@ class ContextMenu extends ContextMenuController { const api = Common.EditorApi.get(); api.asc_registerCallback('asc_onShowComment', this.onApiShowComment); api.asc_registerCallback('asc_onHideComment', this.onApiHideComment); + api.asc_registerCallback('asc_onHyperlinkClick', this.onApiHyperlinkClick); } initMenuItems() { From 535c20743dc9b56c262f0bf6accc5c7be58419ee Mon Sep 17 00:00:00 2001 From: ShimaginAndrey Date: Thu, 14 Oct 2021 17:51:22 +0300 Subject: [PATCH 3/7] [SSE] Fix Bug 53158 --- apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx b/apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx index b5923fa3c3..a0e1e35d0a 100644 --- a/apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx +++ b/apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx @@ -138,7 +138,7 @@ const EditLink = props => { placeholder={_t.textRequired} value={range} onChange={(event) => {setRange(event.target.value)}} - className={isIos ? 'list-input-right' : ''} + className={[isIos ? 'list-input-right' : '', linkSheet === '' && 'disabled'].join(' ')} /> } Date: Mon, 18 Oct 2021 18:27:44 +0300 Subject: [PATCH 4/7] [SSE] Correct RangeLink --- apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx b/apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx index a0e1e35d0a..517c3a8359 100644 --- a/apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx +++ b/apps/spreadsheeteditor/mobile/src/view/edit/EditLink.jsx @@ -107,7 +107,7 @@ const EditLink = props => { }; const valueRange = linkInfo.asc_getRange(); - const [range, setRange] = useState(valueRange || ''); + const [range, setRange] = useState(valueRange || 'A1'); return ( @@ -138,7 +138,7 @@ const EditLink = props => { placeholder={_t.textRequired} value={range} onChange={(event) => {setRange(event.target.value)}} - className={[isIos ? 'list-input-right' : '', linkSheet === '' && 'disabled'].join(' ')} + className={[isIos ? 'list-input-right' : '', curSheet === '' && 'disabled'].join(' ')} /> } Date: Tue, 19 Oct 2021 14:05:12 +0300 Subject: [PATCH 5/7] [SSE] Fix Bug 53074 --- apps/common/mobile/resources/less/common.less | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/apps/common/mobile/resources/less/common.less b/apps/common/mobile/resources/less/common.less index fa988e6722..67936d5db2 100644 --- a/apps/common/mobile/resources/less/common.less +++ b/apps/common/mobile/resources/less/common.less @@ -908,6 +908,14 @@ input[type="number"]::-webkit-inner-spin-button { } } +#idx-celleditor.expanded { + .functions-list { + &__mobile { + top: 70px; + } + } +} + .popover__functions { box-shadow: 0px 10px 100px rgba(0, 0, 0, 0.3); } From 3f99cb48e2459cd11cee8be8913b4bd58c0da903 Mon Sep 17 00:00:00 2001 From: ShimaginAndrey Date: Wed, 20 Oct 2021 16:21:52 +0300 Subject: [PATCH 6/7] [SSE] Fix Bug 53055 --- .../mobile/src/view/edit/EditSlide.jsx | 31 ++++++++----------- 1 file changed, 13 insertions(+), 18 deletions(-) diff --git a/apps/presentationeditor/mobile/src/view/edit/EditSlide.jsx b/apps/presentationeditor/mobile/src/view/edit/EditSlide.jsx index 549bf00325..6d5b2d8ba1 100644 --- a/apps/presentationeditor/mobile/src/view/edit/EditSlide.jsx +++ b/apps/presentationeditor/mobile/src/view/edit/EditSlide.jsx @@ -1,4 +1,4 @@ -import React, {Fragment, useState} from 'react'; +import React, {Fragment, useEffect, useState} from 'react'; import {observer, inject} from "mobx-react"; import {f7, Page, Navbar, List, ListItem, Row, BlockTitle, Link, Toggle, Icon, View, NavRight, ListItemCell, Range, Button, Segmented, ListButton} from 'framework7-react'; import { ThemeColorPalette, CustomColorPicker } from '../../../../../common/mobile/lib/component/ThemeColorPalette.jsx'; @@ -208,21 +208,25 @@ const PageTransition = props => { const storeFocusObjects = props.storeFocusObjects; const transitionObj = storeFocusObjects.slideObject.get_transition(); - const [_effect, setEffect] = useState(transitionObj.get_TransitionType()); - const valueEffectTypes = fillEffectTypes(_effect); - const [type, setType] = useState(valueEffectTypes); - let _effectDelay = transitionObj.get_SlideAdvanceDuration(); const [stateRange, changeRange] = useState((_effectDelay !== null && _effectDelay !== undefined) ? parseInt(_effectDelay / 1000.) : 0); const isDelay = transitionObj.get_SlideAdvanceAfter(); const isStartOnClick = transitionObj.get_SlideAdvanceOnMouseClick(); + + const _effect = transitionObj.get_TransitionType(); const nameEffect = getEffectName(_effect); + if(_effect != Asc.c_oAscSlideTransitionTypes.None) fillEffectTypes(_effect); const _effectType = transitionObj.get_TransitionOption(); const nameEffectType = getEffectTypeName(_effectType); + const _effectDuration = transitionObj.get_TransitionDuration(); + useEffect(() => { + changeRange((_effectDelay !== null && _effectDelay !== undefined) ? parseInt(_effectDelay / 1000.) : 0); + }, [_effectDelay]) + return ( @@ -240,8 +244,6 @@ const PageTransition = props => { onEffectClick: props.onEffectClick, fillEffectTypes, _effect, - setEffect, - setType }}> { _arrCurrentEffectTypes, onEffectTypeClick: props.onEffectTypeClick, _effect, - type, - setType + _effectType, }}> @@ -312,8 +313,7 @@ const PageTransition = props => { const PageEffect = props => { const { t } = useTranslation(); const _t = t("View.Edit", { returnObjects: true }); - const _effect = props._effect; - const [currentEffect, setEffect] = useState(_effect); + const [currentEffect, setEffect] = useState(props._effect); const _arrEffect = props._arrEffect; return ( @@ -334,9 +334,7 @@ const PageEffect = props => { { setEffect(elem.value); - props.setEffect(elem.value); let valueEffectTypes = props.fillEffectTypes(elem.value); - props.setType(valueEffectTypes); props.onEffectClick(elem.value, valueEffectTypes); }}> ) @@ -351,9 +349,7 @@ const PageType= props => { const { t } = useTranslation(); const _t = t("View.Edit", { returnObjects: true }); const _arrCurrentEffectTypes = props._arrCurrentEffectTypes; - const _effect = props._effect; - const type = props.type; - const [currentType, setType] = useState(type); + const [currentType, setType] = useState(props._effectType); return ( @@ -373,8 +369,7 @@ const PageType= props => { { setType(elem.value); - props.setType(elem.value); - props.onEffectTypeClick(elem.value, _effect); + props.onEffectTypeClick(elem.value, props._effect); }}> ) From 58a6d1da3a5de8be084313f684ce39581ac342a3 Mon Sep 17 00:00:00 2001 From: ShimaginAndrey Date: Thu, 21 Oct 2021 11:33:56 +0300 Subject: [PATCH 7/7] [DE PE SSE] Fix Bug 51892 --- apps/common/mobile/resources/less/common-material.less | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/common/mobile/resources/less/common-material.less b/apps/common/mobile/resources/less/common-material.less index 8631b30e97..5be411735e 100644 --- a/apps/common/mobile/resources/less/common-material.less +++ b/apps/common/mobile/resources/less/common-material.less @@ -74,7 +74,7 @@ margin-top: 8px; } .add-image, .inputs-list { - ul:after, :before{ + ul:after { display: none; } }