Merge pull request #595 from hogashi/revice-option

short option label
hogashi · Nov 28, 2023 · 5a8fe23 · 5a8fe23
2 parents dba0017 + 96a9ec9
commit 5a8fe23
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 17 deletions.
diff --git a/__tests__/Constants.test.js b/__tests__/Constants.test.js
@@ -72,12 +72,11 @@ describe('定数', () => {
     ]);
 
     expect(OPTIONS_TEXT).toStrictEqual({
-      // 公式Web
-      SHOW_ON_TIMELINE: 'タイムラインにボタンを表示',
-      SHOW_ON_TWEET_DETAIL: 'ツイート詳細にボタンを表示',
+      SHOW_ON_TIMELINE: 'タイムライン',
+      SHOW_ON_TWEET_DETAIL: '(旧表示で)ツイート詳細',
       // TweetDeck
-      SHOW_ON_TWEETDECK_TIMELINE: 'タイムラインにボタンを表示',
-      SHOW_ON_TWEETDECK_TWEET_DETAIL: 'ツイート詳細にボタンを表示',
+      SHOW_ON_TWEETDECK_TIMELINE: 'タイムライン',
+      SHOW_ON_TWEETDECK_TWEET_DETAIL: '(旧表示で)ツイート詳細',
       // 画像ページ
       STRIP_IMAGE_SUFFIX: '[Ctrl]+[s]で拡張子を校正',
     });

diff --git a/__tests__/__snapshots__/popup.test.js.snap b/__tests__/__snapshots__/popup.test.js.snap
@@ -5,7 +5,7 @@ exports[`Popup render 1`] = `
   className="p-3 flex flex-col justify-center items-center"
   style={
     {
-      "minWidth": "260px",
+      "minWidth": "230px",
     }
   }
 >
@@ -47,7 +47,7 @@ exports[`Popup render 1`] = `
             className="font-medium text-gray-900"
             htmlFor="SHOW_ON_TIMELINE"
           >
-            タイムラインにボタンを表示
+            タイムライン
           </label>
         </div>
       </div>
@@ -73,7 +73,7 @@ exports[`Popup render 1`] = `
             className="font-medium text-gray-900"
             htmlFor="SHOW_ON_TWEET_DETAIL"
           >
-            ツイート詳細にボタンを表示
+            (旧表示で)ツイート詳細
           </label>
         </div>
       </div>
@@ -108,7 +108,7 @@ exports[`Popup render 1`] = `
             className="font-medium text-gray-900"
             htmlFor="SHOW_ON_TWEETDECK_TIMELINE"
           >
-            タイムラインにボタンを表示
+            タイムライン
           </label>
         </div>
       </div>
@@ -134,7 +134,7 @@ exports[`Popup render 1`] = `
             className="font-medium text-gray-900"
             htmlFor="SHOW_ON_TWEETDECK_TWEET_DETAIL"
           >
-            ツイート詳細にボタンを表示
+            (旧表示で)ツイート詳細
           </label>
         </div>
       </div>

diff --git a/src/ButtonSetter.ts b/src/ButtonSetter.ts
@@ -31,8 +31,7 @@ export class ButtonSetter implements ButtonSetterType {
   // ツイート詳細にボタン表示
   public setButtonOnTweetDetail(currentOptions: OptionsBool): void {
     // 昔のビューの処理はしばらく残す
-    // TODO: Reactレイアウトでも実装する必要がある？
-    // ref: https://github.com/hogashi/twitterOpenOriginalImage/issues/32#issuecomment-578510155
+    // - 公式の新しいhtmlでは使えないが, 古いhtmlで閲覧している人がいるので残してある
     this._setButtonOnTweetDetail(currentOptions);
   }
 

diff --git a/src/ButtonSetterTweetDeck.ts b/src/ButtonSetterTweetDeck.ts
@@ -60,6 +60,7 @@ export class ButtonSetterTweetDeck implements ButtonSetterType {
   }
 
   // ツイート詳細にボタン表示
+  // - 公式の新しいhtmlでは使えないが, 古いhtmlで閲覧している人がいるので残してある
   public setButtonOnTweetDetail(currentOptions: OptionsBool): void {
     // ツイート詳細にボタン表示する設定がされているときだけ実行する
     if (!currentOptions[SHOW_ON_TWEETDECK_TWEET_DETAIL]) {

diff --git a/src/constants.ts b/src/constants.ts
@@ -100,11 +100,11 @@ export const OPTION_KEYS = [
 ] as const;
 export const OPTIONS_TEXT: { [key in keyof Options]: string } = {
   // 公式Web
-  SHOW_ON_TIMELINE: 'タイムラインにボタンを表示',
-  SHOW_ON_TWEET_DETAIL: 'ツイート詳細にボタンを表示',
+  SHOW_ON_TIMELINE: 'タイムライン',
+  SHOW_ON_TWEET_DETAIL: '(旧表示で)ツイート詳細',
   // TweetDeck
-  SHOW_ON_TWEETDECK_TIMELINE: 'タイムラインにボタンを表示',
-  SHOW_ON_TWEETDECK_TWEET_DETAIL: 'ツイート詳細にボタンを表示',
+  SHOW_ON_TWEETDECK_TIMELINE: 'タイムライン',
+  SHOW_ON_TWEETDECK_TWEET_DETAIL: '(旧表示で)ツイート詳細',
   // 画像ページ
   STRIP_IMAGE_SUFFIX: '[Ctrl]+[s]で拡張子を校正',
 };

diff --git a/src/extension-contexts/popup.tsx b/src/extension-contexts/popup.tsx
@@ -91,7 +91,7 @@ export const Popup = (props: Props): JSX.Element => {
     <div
       className="p-3 flex flex-col justify-center items-center"
       style={{
-        minWidth: '260px',
+        minWidth: '230px',
       }}
     >
       <h1 className="text-xl font-bold leading-7 text-gray-900 self-center sm:truncate sm:text-3xl sm:tracking-tight">