Merge pull request #1378 from Moyf/master

Add a 'Suppress message if no change' feature with settings.

Author: pjkaufman

src/lang/locale/de.ts

@@ -128,6 +128,10 @@ export default {
         'name': 'Meldung beim Linten anzeigen',
         'description': 'Zeigen Sie die Anzahl der Zeichen an, die sich nach dem Linten geändert haben',
       },
+      'suppress-message-when-no-change': {
+        'name': 'Meldung bei keiner Änderung unterdrücken',
+        'description': 'Wenn aktiviert, wird keine Meldung angezeigt, wenn keine tatsächlichen Änderungen auftreten.',
+      },
       'lint-on-file-change': {
         'name': 'Linten bei Dateiänderungen',
         'description': 'Wenn eine Datei geschlossen oder zu einer neuen Datei gewechselt wird, wird die vorherige Datei gelinted.',

src/lang/locale/en.ts

@@ -162,6 +162,10 @@ export default {
         'name': 'Display message on lint',
         'description': 'Display the number of characters changed after linting',
       },
+      'suppress-message-when-no-change': {
+        'name': 'Suppress Message When No Change',
+        'description': 'If enabled, no message will be shown when no actual changes occur.',
+      },
       'lint-on-file-change': {
         'name': 'Lint on Focused File Change',
         'description': 'When a file is closed or a new file is swapped to, the previous file is linted.',

src/lang/locale/es.ts

@@ -101,6 +101,10 @@ export default {
         'name': 'Mostrar mensaje en analizar',
         'description': 'Mostrar el número de caracteres modificados después de analizar',
       },
+      'suppress-message-when-no-change': {
+        'name': 'Suprimir mensaje cuando no hay cambios',
+        'description': 'Si está habilitado, no se mostrará ningún mensaje cuando no ocurran cambios reales.',
+      },
       'folders-to-ignore': {
         'name': 'Carpetas para omitir',
         'description': 'Carpetas que se deben omitir al analizar todos los archivos o al guardar en línea.',

src/lang/locale/ru.ts

@@ -150,6 +150,10 @@ export default {
         'name': 'Показывать уведомление после линта',
         'description': 'Показывать количество символов, измененных после линта',
       },
+      'suppress-message-when-no-change': {
+        'name': 'Подавить уведомление при отсутствии изменений',
+        'description': 'Если включено, уведомление не будет показано, когда не происходит фактических изменений.',
+      },
       'lint-on-file-change': {
         'name': 'Линт при изменении файла в фокусе',
         'description': 'Когда файл закрывается или заменяется на новый, предыдущий файл линтится.',

src/lang/locale/tr.ts

@@ -127,6 +127,10 @@ export default {
         'name': 'Düzeltme sonrası mesajı göster',
         'description': 'Düzeltme sonrası değişen karakter sayısını gösterir',
       },
+      'suppress-message-when-no-change': {
+        'name': 'Değişiklik Olmadığında Mesajı Bastır',
+        'description': 'Etkinleştirildiğinde, gerçek değişiklikler olmadığında mesaj gösterilmez.',
+      },
       'lint-on-file-change': {
         'name': 'Dosya Değişikliğinde Düzeltme',
         'description': 'Bir dosya kapatıldığında veya yeni bir dosya açıldığında, önceki dosya düzeltilir.',

src/lang/locale/zh-cn.ts

@@ -150,6 +150,10 @@ export default {
         'name': '格式化后显示消息',
         'description': '格式化后显示修改了多少字符',
       },
+      'suppress-message-when-no-change': {
+        'name': '无修改时不显示消息',
+        'description': '开启后，若无实际改动发生则不显示消息。',
+      },
       'lint-on-file-change': {
         'name': '文件修改时格式化',
         'description': '当文件关闭或是切换到新文件时，格式化之前的文件',

src/main.ts

@@ -123,6 +123,9 @@ export default class LinterPlugin extends Plugin {
   async loadSettings() {
     const data = await this.loadData();
     this.settings = Object.assign({}, DEFAULT_SETTINGS, data);
+    if (typeof this.settings.suppressMessageWhenNoChange !== 'boolean') {
+      this.settings.suppressMessageWhenNoChange = false;
+    }
     if (typeof this.settings.logLevel === 'number') {
       this.settings.logLevel = convertNumberToLogLevel(this.settings.logLevel);
     }
@@ -892,7 +895,11 @@ export default class LinterPlugin extends Plugin {
   }
 
   private displayChangedMessage(charsAdded: number, charsRemoved: number) {
+    const suppressMessageWhenNoChange = this.settings.suppressMessageWhenNoChange ?? false;
     if (this.settings.displayChanged) {
+      if (suppressMessageWhenNoChange && charsAdded + charsRemoved === 0) {
+        return;
+      }
       const message = dedent`
         ${charsAdded} ${getTextInLanguage('notice-text.characters-added')}
         ${charsRemoved} ${getTextInLanguage('notice-text.characters-removed')}

src/settings-data.ts

@@ -29,6 +29,7 @@ export interface LinterSettings {
   };
   lintOnSave: boolean;
   displayChanged: boolean;
+  suppressMessageWhenNoChange?: boolean;
   settingsConvertedToConfigKeyValues: boolean;
   recordLintOnSaveLogs: boolean;
   lintOnFileChange: boolean;
@@ -49,6 +50,7 @@ export const DEFAULT_SETTINGS: Partial<LinterSettings> = {
   lintOnSave: false,
   recordLintOnSaveLogs: false,
   displayChanged: true,
+  suppressMessageWhenNoChange: false,
   lintOnFileChange: false,
   displayLintOnFileChangeNotice: false,
   settingsConvertedToConfigKeyValues: false,

src/ui/linter-components/tab-components/general-tab.ts

@@ -54,6 +54,10 @@ export class GeneralTab extends Tab {
       displayLintOnActiveFileChangeSetting.hide();
     }
 
+    tempDiv = this.contentEl.createDiv();
+    const suppressMessageWhenNoChangeSetting = new ToggleSetting( tempDiv, 'tabs.general.suppress-message-when-no-change.name', 'tabs.general.suppress-message-when-no-change.description', 'suppressMessageWhenNoChange', this.plugin);
+    this.addSettingSearchInfoForGeneralSettings(suppressMessageWhenNoChangeSetting);
+
     const sysLocale = navigator.language?.toLowerCase();
     const localeValues = ['system-default'];
     const localeDescriptions = [getTextInLanguage('tabs.general.same-as-system-locale').replace('{SYS_LOCALE}', sysLocale)];