Browse Source

fix: clarify that `synchronous` is for page mode

tophf 4 years ago
parent
commit
6815399972
2 changed files with 7 additions and 4 deletions
  1. 4 3
      src/_locales/en/messages.yml
  2. 3 1
      src/options/views/tab-settings/index.vue

+ 4 - 3
src/_locales/en/messages.yml

@@ -502,10 +502,11 @@ labelViewTable:
   description: Label for option in dashboard script list to show the scripts as a table.
   description: Label for option in dashboard script list to show the scripts as a table.
   message: Table view
   message: Table view
 labelXhrInject:
 labelXhrInject:
-  description: NEW_LOCALE_ITEM
-  message: Synchronous
+  description: >-
+    (`$1` will be shown as `page`) Option in Advanced settings to enable synchronous injection.
+  message: Synchronous $1 mode
 labelXhrInjectHint:
 labelXhrInjectHint:
-  description: NEW_LOCALE_ITEM
+  description: Tooltip of the `Synchronous page mode` option in Advanced settings.
   message: >-
   message: >-
     Enable only if you have a script that needs to run before the page starts
     Enable only if you have a script that needs to run before the page starts
     loading and currently it's running too late. Just like Instant injection
     loading and currently it's running too late. Just like Instant injection

+ 3 - 1
src/options/views/tab-settings/index.vue

@@ -104,7 +104,9 @@
           <label>
           <label>
             <setting-check name="xhrInject"/>
             <setting-check name="xhrInject"/>
             <tooltip :content="i18n('labelXhrInjectHint')">
             <tooltip :content="i18n('labelXhrInjectHint')">
-              <span v-text="i18n('labelXhrInject')"/>
+              <locale-group i18n-key="labelXhrInject">
+                <code>page</code>
+              </locale-group>
             </tooltip>
             </tooltip>
           </label>
           </label>
         </div>
         </div>