From 144b94a30bf3241aed760e009dc24251f60fefb2 Mon Sep 17 00:00:00 2001 From: Alexey Koshelev Date: Fri, 6 Feb 2026 10:39:01 +0300 Subject: [PATCH] [PDFE] Hide review resolve button. For bug 79596) --- .../main/lib/controller/ReviewChanges.js | 4 +-- apps/common/main/lib/view/ReviewChanges.js | 29 +++++++++++-------- apps/pdfeditor/main/app/controller/Main.js | 6 +++- 3 files changed, 24 insertions(+), 15 deletions(-) diff --git a/apps/common/main/lib/controller/ReviewChanges.js b/apps/common/main/lib/controller/ReviewChanges.js index 6a4ccd09c1..69ce61f288 100644 --- a/apps/common/main/lib/controller/ReviewChanges.js +++ b/apps/common/main/lib/controller/ReviewChanges.js @@ -167,10 +167,10 @@ define([ } }, - setMode: function(mode) { + setMode: function(mode, tabOptions) { this.appConfig = mode; this.popoverChanges = new Common.Collections.ReviewChanges(); - this.view = this.createView('Common.Views.ReviewChanges', { mode: mode }); + this.view = this.createView('Common.Views.ReviewChanges', { mode: mode, tabOptions: (tabOptions || {}) }); if (!!this.appConfig.sharingSettingsUrl && this.appConfig.sharingSettingsUrl.length || this.appConfig.canRequestSharingSettings) { Common.Gateway.on('showsharingsettings', _.bind(this.changeAccessRights, this)); diff --git a/apps/common/main/lib/view/ReviewChanges.js b/apps/common/main/lib/view/ReviewChanges.js index cdf67c22fd..045e22baa5 100644 --- a/apps/common/main/lib/view/ReviewChanges.js +++ b/apps/common/main/lib/view/ReviewChanges.js @@ -271,6 +271,9 @@ define([ Common.UI.BaseView.prototype.initialize.call(this, options); this.appConfig = options.mode; + this.tabOptions = { + canCommentResolve: options?.tabOptions?.canCommentResolve ?? true + } this.lockedControls = []; var filter = Common.localStorage.getKeysFilter(); this.appPrefix = (filter && filter.length) ? filter.split(',')[0] : ''; @@ -499,18 +502,20 @@ define([ dataHintOffset: 'small' }); this.lockedControls.push(this.btnCommentRemove); - this.btnCommentResolve = new Common.UI.Button({ - cls: 'btn-toolbar x-huge icon-top', - caption: this.txtCommentResolve, - split: true, - iconCls: 'toolbar__icon btn-resolve-all', - lock: [_set.previewReviewMode, _set.viewFormMode, _set.hideComments, _set['Objects'], _set.lostConnect, _set.docLockView, _set.docLockForms, _set.viewMode, _set.slideMasterMode], - action: 'comment-resolve', - dataHint: '1', - dataHintDirection: 'bottom', - dataHintOffset: 'small' - }); - this.lockedControls.push(this.btnCommentResolve); + if(this.tabOptions.canCommentResolve) { + this.btnCommentResolve = new Common.UI.Button({ + cls: 'btn-toolbar x-huge icon-top', + caption: this.txtCommentResolve, + split: true, + iconCls: 'toolbar__icon btn-resolve-all', + lock: [_set.previewReviewMode, _set.viewFormMode, _set.hideComments, _set['Objects'], _set.lostConnect, _set.docLockView, _set.docLockForms, _set.viewMode, _set.slideMasterMode], + action: 'comment-resolve', + dataHint: '1', + dataHintDirection: 'bottom', + dataHintOffset: 'small' + }); + this.lockedControls.push(this.btnCommentResolve); + } } if (this.appConfig.isEdit && this.appConfig.canCoAuthoring && this.appConfig.canUseMailMerge) { diff --git a/apps/pdfeditor/main/app/controller/Main.js b/apps/pdfeditor/main/app/controller/Main.js index 7788f92660..a83dc74ffb 100644 --- a/apps/pdfeditor/main/app/controller/Main.js +++ b/apps/pdfeditor/main/app/controller/Main.js @@ -1664,7 +1664,11 @@ define([ var me = this, application = this.getApplication(), reviewController = application.getController('Common.Controllers.ReviewChanges'); - reviewController.setMode(me.appOptions).setConfig({config: me.editorConfig}, me.api).loadDocument({doc:me.document}); + + reviewController + .setMode(me.appOptions, { canCommentResolve: false }) + .setConfig({config: me.editorConfig}, me.api) + .loadDocument({doc:me.document}); if (this.appOptions.isEdit) { if (me.appOptions.isSignatureSupport || me.appOptions.isPasswordSupport)