Merge pull request #2950 from ONLYOFFICE/fix/fix-bugs

[DE PE SSE PDF] Fix bug 67213
ONLYOFFICE · Apr 19, 2024 · a10e75b · a10e75b
2 parents fb76d37 + 4cd87ff
commit a10e75b
Show file tree

Hide file tree

Showing 9 changed files with 13 additions and 9 deletions.
diff --git a/apps/documenteditor/main/app/controller/Statusbar.js b/apps/documenteditor/main/app/controller/Statusbar.js
@@ -304,7 +304,7 @@ define([
             var tip = new Common.UI.SynchronizeTip({
                 target  : me.btnTurnReview.$el,
                 text    : text,
-                placement: 'top-left',
+                placement: Common.UI.isRTL() ? 'top-right' : 'top-left',
                 showLink: !!storage
             });
             tip.on({

diff --git a/apps/documenteditor/main/app/view/SignatureSettings.js b/apps/documenteditor/main/app/view/SignatureSettings.js
@@ -365,7 +365,7 @@ define([
                     text    : tipText,
                     showLink: showLink,
                     textLink: this.txtContinueEditing,
-                    placement: 'left-bottom'
+                    placement: Common.UI.isRTL() ? 'right-bottom' : 'left-bottom'
                 });
                 tip.on({
                     'dontshowclick': function() {

diff --git a/apps/documenteditor/main/app/view/Toolbar.js b/apps/documenteditor/main/app/view/Toolbar.js
@@ -3092,9 +3092,10 @@ define([
             },
 
             createSynchTip: function () {
+                var direction = Common.UI.isRTL() ? 'left' : 'right';
                 this.synchTooltip = new Common.UI.SynchronizeTip({
                     extCls: (this.mode.compactHeader) ? undefined : 'inc-index',
-                    placement: this.mode.isDesktopApp ? 'bottom-right' : 'right-bottom',
+                    placement: this.mode.isDesktopApp ? 'bottom-' + direction : direction + '-bottom',
                     target: this.btnCollabChanges.$el
                 });
                 this.synchTooltip.on('dontshowclick', function () {

diff --git a/apps/pdfeditor/main/app/controller/Toolbar.js b/apps/pdfeditor/main/app/controller/Toolbar.js
@@ -1222,7 +1222,7 @@ define([
                     if (!Common.localStorage.getItem("pdfe-embed-hide-submittip")) {
                         me.requiredTooltip = new Common.UI.SynchronizeTip({
                             extCls: 'colored',
-                            placement: 'bottom-right',
+                            placement: Common.UI.isRTL() ? 'bottom-left' : 'bottom-right',
                             target: me.toolbar.btnSubmit.$el,
                             text: me.textRequired,
                             showLink: false,

diff --git a/apps/pdfeditor/main/app/view/Toolbar.js b/apps/pdfeditor/main/app/view/Toolbar.js
@@ -1660,9 +1660,10 @@ define([
             },
 
             createSynchTip: function () {
+                var direction = Common.UI.isRTL() ? 'left' : 'right';
                 this.synchTooltip = new Common.UI.SynchronizeTip({
                     extCls: (this.mode.compactHeader) ? undefined : 'inc-index',
-                    placement: this.mode.isDesktopApp ? 'bottom-right' : 'right-bottom',
+                    placement: this.mode.isDesktopApp ? 'bottom-' + direction : direction + '-bottom',
                     target: this.btnCollabChanges.$el
                 });
                 this.synchTooltip.on('dontshowclick', function () {

diff --git a/apps/presentationeditor/main/app/view/SignatureSettings.js b/apps/presentationeditor/main/app/view/SignatureSettings.js
@@ -314,7 +314,7 @@ define([
                     text    : tipText,
                     showLink: showLink,
                     textLink: this.txtContinueEditing,
-                    placement: 'left-bottom'
+                    placement: Common.UI.isRTL() ? 'right-bottom' : 'left-bottom'
                 });
                 tip.on({
                     'dontshowclick': function() {

diff --git a/apps/presentationeditor/main/app/view/Toolbar.js b/apps/presentationeditor/main/app/view/Toolbar.js
@@ -1948,9 +1948,10 @@ define([
             },
 
             createSynchTip: function () {
+                var direction = Common.UI.isRTL() ? 'left' : 'right';
                 this.synchTooltip = new Common.UI.SynchronizeTip({
                     extCls: (this.mode.compactHeader) ? undefined : 'inc-index',
-                    placement: this.mode.isDesktopApp ? 'bottom-right' : 'right-bottom',
+                    placement: this.mode.isDesktopApp ? 'bottom-' + direction : direction + '-bottom',
                     target: this.btnCollabChanges.$el
                 });
                 this.synchTooltip.on('dontshowclick', function () {

diff --git a/apps/spreadsheeteditor/main/app/view/SignatureSettings.js b/apps/spreadsheeteditor/main/app/view/SignatureSettings.js
@@ -360,7 +360,7 @@ define([
                     text    : tipText,
                     showLink: showLink,
                     textLink: this.txtContinueEditing,
-                    placement: 'left-bottom'
+                    placement: Common.UI.isRTL() ? 'right-bottom' : 'left-bottom'
                 });
                 tip.on({
                     'dontshowclick': function() {

diff --git a/apps/spreadsheeteditor/main/app/view/Toolbar.js b/apps/spreadsheeteditor/main/app/view/Toolbar.js
@@ -3213,9 +3213,10 @@ define([
         },
 
         createSynchTip: function () {
+            var direction = Common.UI.isRTL() ? 'left' : 'right';
             this.synchTooltip = new Common.UI.SynchronizeTip({
                 extCls: (this.mode.compactHeader) ? undefined : 'inc-index',
-                placement: this.mode.isDesktopApp ? 'bottom-right' : 'right-bottom',
+                placement: this.mode.isDesktopApp ? 'bottom-' + direction : direction + '-bottom',
                 target: this.btnCollabChanges.$el
             });
             this.synchTooltip.on('dontshowclick', function() {