From afdbcfb2d7936fd98a998f1f7a2c96905da24ba7 Mon Sep 17 00:00:00 2001
From: RyuJongJin <266630912+RyuJongJin@users.noreply.github.com>
Date: Sat, 10 Oct 2026 03:44:16 +0900
Subject: [PATCH] =?UTF-8?q?=EC=9B=B9=20=EB=B8=8C=EB=9D=BC=EC=9A=B0?=
 =?UTF-8?q?=EC=A0=80:=20=EB=8F=99=EC=98=81=EC=83=81=EC=9D=84=20=EB=AA=BB?=
 =?UTF-8?q?=20=EC=B0=BE=EC=9D=80=20=ED=8E=98=EC=9D=B4=EC=A7=80=EC=9D=98=20?=
 =?UTF-8?q?=EB=8B=A4=EC=9A=B4=EB=A1=9C=EB=93=9C=20=EB=B2=84=ED=8A=BC?=
 =?UTF-8?q?=EC=9D=80=20=ED=85=8C=EB=91=90=EB=A6=AC=EB=A7=8C=20(137)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- 덜 눈에 띄게 하려던 tonal 버튼이 이 테마에서 채움 버튼과 거의 같아 보였다 → OutlinedButton · IconButton.outlined.
- YouTube 첫 화면 · 구독 목록은 미리보기 <video> 가 있어 동영상 있음으로 보였다 → 동영상 주소가 정해진 사이트
  (YouTube · Vimeo · TikTok · 치지직 등) 는 주소로만, 그 밖은 페이지 안의 <video> 로 (downloadLooksUseful).
- browser_test.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 lib/core/web_address.dart | 12 ++++++++++++
 lib/ui/browser_page.dart  |  8 ++++----
 test/browser_test.dart    | 15 +++++++++++++--
 3 files changed, 29 insertions(+), 6 deletions(-)

diff --git a/lib/core/web_address.dart b/lib/core/web_address.dart
index b9370c5..851b310 100644
--- a/lib/core/web_address.dart
+++ b/lib/core/web_address.dart
@@ -24,3 +24,15 @@ bool looksLikeVideoPage(String url) {
     caseSensitive: false,
   ).hasMatch(url);
 }
+
+/// 137: 동영상 페이지 주소가 정해져 있는 사이트 (YouTube 등). 이런 사이트는 주소만으로 판단한다 -
+/// 첫 화면 · 구독 목록에도 미리보기 &lt;video&gt; 가 있어 "동영상 있음" 으로 보였다.
+bool isKnownVideoSite(String url) => RegExp(
+      r'^https?://([\w-]+\.)*(youtube\.com|youtu\.be|vimeo\.com|dailymotion\.com|twitch\.tv|tiktok\.com|instagram\.com|'
+      r'tv\.naver\.com|chzzk\.naver\.com|bilibili\.com|nicovideo\.jp)(/|$)',
+      caseSensitive: false,
+    ).hasMatch(url);
+
+/// 137: 다운로드 버튼을 눈에 띄게 (채움) 할지 - 주소가 동영상 페이지이거나, 주소로 판단하지 않는 사이트에서 &lt;video&gt; 를 찾았을 때
+bool downloadLooksUseful(String url, {required bool pageHasVideo}) =>
+    looksLikeVideoPage(url) || (pageHasVideo && !isKnownVideoSite(url));
diff --git a/lib/ui/browser_page.dart b/lib/ui/browser_page.dart
index d9cdc2f..40c6cbc 100644
--- a/lib/ui/browser_page.dart
+++ b/lib/ui/browser_page.dart
@@ -399,8 +399,8 @@ class _BrowserPageState extends State<BrowserPage> with RouteAware {
   /// 27: 어느 사이트에서나 받아 볼 수 있다 (yt-dlp 가 지원하면 받고, 아니면 다운로드 목록에 이유가 보인다)
   bool get _canDownload => widget.downloads != null && _url.startsWith('http');
 
-  /// 동영상이 있어 보이는 페이지 (버튼을 눈에 띄게)
-  bool get _likelyVideo => looksLikeVideoPage(_url) || _hasVideo;
+  /// 동영상이 있어 보이는 페이지 (버튼을 눈에 띄게). 137: 못 찾은 페이지는 테두리만 있는 버튼
+  bool get _likelyVideo => downloadLooksUseful(_url, pageHasVideo: _hasVideo);
 
   String get _downloadTip =>
       _likelyVideo ? tr('다운로드') : tr('다운로드 (이 페이지에서 동영상을 찾지 못했지만 받아 볼 수 있습니다)');
@@ -650,7 +650,7 @@ class _BrowserPageState extends State<BrowserPage> with RouteAware {
                   onPressed: _canDownload ? _download : null,
                   icon: const Icon(Icons.download, size: 18),
                 )
-              : IconButton.filledTonal(
+              : IconButton.outlined(
                   tooltip: _downloadTip,
                   visualDensity: VisualDensity.compact,
                   onPressed: _canDownload ? _download : null,
@@ -665,7 +665,7 @@ class _BrowserPageState extends State<BrowserPage> with RouteAware {
                     icon: const Icon(Icons.download, size: 18),
                     label: Text(tr('다운로드')),
                   )
-                : FilledButton.tonalIcon(
+                : OutlinedButton.icon(
                     onPressed: _canDownload ? _download : null,
                     icon: const Icon(Icons.download, size: 18),
                     label: Text(tr('다운로드')),
diff --git a/test/browser_test.dart b/test/browser_test.dart
index 24a2419..db95a00 100644
--- a/test/browser_test.dart
+++ b/test/browser_test.dart
@@ -146,6 +146,14 @@ void main() {
       expect(looksLikeVideoPage('https://tv.naver.com/v/123'), isTrue);
       expect(looksLikeVideoPage('https://www.google.com/'), isFalse);
     });
+    test('137: 다운로드 버튼을 눈에 띄게 할지 - YouTube 첫 화면 · 구독 목록은 미리보기 <video> 가 있어도 아님', () {
+      expect(downloadLooksUseful('https://www.youtube.com/', pageHasVideo: true), isFalse);
+      expect(downloadLooksUseful('https://m.youtube.com/feed/subscriptions', pageHasVideo: true), isFalse);
+      expect(downloadLooksUseful('https://www.youtube.com/watch?v=abc', pageHasVideo: false), isTrue);
+      // 주소로 판단하지 않는 사이트는 페이지 안의 <video> 로
+      expect(downloadLooksUseful('https://example.com/post/1', pageHasVideo: true), isTrue);
+      expect(downloadLooksUseful('https://example.com/post/1', pageHasVideo: false), isFalse);
+    });
     test('앱 안 브라우저 쿠키 → yt-dlp', () {
       // 내보낸 cookies.txt 가 없으면 쿠키 없이 (프로필 폴더를 직접 읽지 않는다)
       expect(ytDlpCookieArgs(browser: internalBrowserCookies, internalProfile: r'C:\w\EBWebView\Default'), isEmpty);
@@ -382,9 +390,11 @@ void main() {
     expect(find.text('수정 · 이동'), findsOneWidget);
     await tester.tapAt(const Offset(5, 880));
     await tester.pumpAndSettle();
-    FilledButton dl() => tester.widget<FilledButton>(find.ancestor(of: find.text('다운로드'), matching: find.byWidgetPredicate((w) => w is FilledButton)));
-    // 27: 동영상 페이지가 아니어도 받아 볼 수 있다 (덜 눈에 띄는 버튼 · 안내)
+    ButtonStyleButton dl() =>
+        tester.widget<ButtonStyleButton>(find.ancestor(of: find.text('다운로드'), matching: find.byWidgetPredicate((w) => w is ButtonStyleButton)));
+    // 27: 동영상 페이지가 아니어도 받아 볼 수 있다 (덜 눈에 띄는 버튼 · 안내). 137: 테두리만 있는 버튼
     expect(dl().onPressed, isNotNull);
+    expect(dl(), isA<OutlinedButton>());
     expect(find.byTooltip('다운로드 (이 페이지에서 동영상을 찾지 못했지만 받아 볼 수 있습니다)'), findsOneWidget);
 
     // 주소창 입력 → 이동
@@ -395,6 +405,7 @@ void main() {
     host.onTitle('재미있는 영상');
     await tester.pump();
     expect(dl().onPressed, isNotNull);
+    expect(dl(), isA<FilledButton>(), reason: '137: 동영상 페이지는 진한 버튼');
     await tester.tap(find.text('다운로드'));
     await tester.pump();
     expect(video.started, ['https://www.youtube.com/watch?v=abc']);