Firefox optimization

wtetsu · Oct 16, 2021 · 6b2cf22 · 6b2cf22
1 parent 2dbe3d1
commit 6b2cf22
Show file tree

Hide file tree

Showing 4 changed files with 44 additions and 27 deletions.
diff --git a/src/options/component/organism/Tips.tsx b/src/options/component/organism/Tips.tsx
@@ -7,6 +7,7 @@
 import React from "react";
 import { res } from "../../logic";
 import { ExternalLink } from "../atom/ExternalLink";
+import { Links } from "../../resource/links";
 
 type TipsProps = {
   style: React.CSSProperties;
@@ -22,35 +23,24 @@ const LINK_STYLE: React.CSSProperties = {
 };
 
 export const Tips: React.VFC<TipsProps> = (props) => {
+  const tips = [
+    { text: res.get("windowManipulation"), href: Links.windowManipulation },
+    { text: res.get("downloadDictData"), href: Links.downloadDictData },
+    { text: res.get("setKeyboardShortcuts"), href: Links.setKeyboardShortcuts },
+  ];
+
   return (
     <div style={{ ...props.style }}>
       <div style={STYLE_INNER}>
-        <ExternalLink
-          href="https://github.com/wtetsu/mouse-dictionary/wiki/Window-manipulation"
-          icon={true}
-          style={LINK_STYLE}
-        >
-          <img src="img/hint.png" width="16" height="16" style={{ position: "relative", top: 3, marginRight: 3 }} />
-          {res.get("windowManipulation")}
-        </ExternalLink>
-        <br />
-        <ExternalLink
-          href="https://github.com/wtetsu/mouse-dictionary/wiki/Download-dictionary-data"
-          icon={true}
-          style={LINK_STYLE}
-        >
-          <img src="img/hint.png" width="16" height="16" style={{ position: "relative", top: 3, marginRight: 3 }} />
-          {res.get("downloadDictData")}
-        </ExternalLink>
-        <br />
-        <ExternalLink
-          href="https://github.com/wtetsu/mouse-dictionary/wiki/Keyboard-shortcuts"
-          icon={true}
-          style={LINK_STYLE}
-        >
-          <img src="img/hint.png" width="16" height="16" style={{ position: "relative", top: 3, marginRight: 3 }} />
-          {res.get("setKeyboardShortcuts")}
-        </ExternalLink>
+        {tips.map((e, i) => (
+          <>
+            <ExternalLink key={i} href={e.href} icon={true} style={LINK_STYLE}>
+              <img src="img/hint.png" width="16" height="16" style={{ position: "relative", top: 3, marginRight: 3 }} />
+              {e.text}
+            </ExternalLink>
+            <br />
+          </>
+        ))}
       </div>
     </div>
   );

diff --git a/src/options/extern/env.ts b/src/options/extern/env.ts
@@ -12,7 +12,7 @@ const optionsEnv = { ...env } as Env;
 if (BROWSER === "FIREFOX") {
   // Larger registerRecordsAtOnce causes memory hog
   // when loading large dictionary data in Firefox.
-  optionsEnv.registerRecordsAtOnce = 1000;
+  optionsEnv.registerRecordsAtOnce = 10000;
   optionsEnv.support = { localGetBytesInUse: false };
 } else {
   optionsEnv.registerRecordsAtOnce = 100000;

diff --git a/src/options/resource/links.ts b/src/options/resource/links.ts
@@ -0,0 +1,21 @@
+/**
+ * Mouse Dictionary (https://github.com/wtetsu/mouse-dictionary/)
+ * Copyright 2018-present wtetsu
+ * Licensed under MIT
+ */
+
+import { BROWSER, ExternalLinks } from "../types";
+
+const Links: ExternalLinks = {
+  windowManipulation: "https://github.com/wtetsu/mouse-dictionary/wiki/Window-manipulation",
+  downloadDictData: "https://github.com/wtetsu/mouse-dictionary/wiki/Download-dictionary-data",
+  setKeyboardShortcuts: "",
+};
+
+if (BROWSER === "FIREFOX") {
+  Links.setKeyboardShortcuts = "https://github.com/wtetsu/mouse-dictionary/wiki/Keyboard-shortcuts-(Firefox)";
+} else {
+  Links.setKeyboardShortcuts = "https://github.com/wtetsu/mouse-dictionary/wiki/Keyboard-shortcuts";
+}
+
+export { Links };
diff --git a/src/options/types.ts b/src/options/types.ts
@@ -71,4 +71,10 @@ export type DictionaryFile = {
   format: DictionaryFileFormat;
 };
 
+export type ExternalLinks = {
+  windowManipulation: string;
+  downloadDictData: string;
+  setKeyboardShortcuts: string;
+};
+
 export declare const BROWSER: "CHROME" | "FIREFOX" | "SAFARI";