Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Block articles in the News tab of search results individually / 検索結果の「ニュース」タブの記事を個別にブロックする #14

Closed
25 tasks done
ikemo3 opened this issue Jun 23, 2019 · 1 comment · Fixed by #155
Assignees
Labels
bug enhancement
Milestone

Comments

@ikemo3
Copy link
Owner

@ikemo3 ikemo3 commented Jun 23, 2019

現在Google検索の「ニュース」タブをクリックしたときに同じ内容に対して複数の記事がある場合、以下の問題がある。

  • 先頭の記事がブロック対象になっているため、他の記事は開かないと読めない
  • 他の記事はブロック対象になっていない

特に後者が致命的なので修正する必要がある。

調査結果

  • 先頭の記事がブロック対象になっているのは class="g" のため
    • class="top" で画像、それ以外は class="gG0TJc" で取れるようだ。
    • 画像をブロックするかどうかの考慮が必要
    • 個別に表示を制御するか、2つを <div> で囲むかのどちらか。後者でも崩れは発生しなさそう。
  • それ以外は class="card-section" で区別するので良さそう。
    • こちらの方だけ優先するのもアリか。
  • レイアウト調整
    • 画像には3つのclassがついている。
      • class="dfhHve" に画像のサイズ width: 100px
      • class="NQHJEb"margin-right: 8px

ToDo

  • 既存のコードの整理
    • ブロックのパターン間で重複しているコードを何とかしたい
  • パターン対応
    • card-section
    • top
  • レイアウト調整
    • 一番最後の記事のブロックリンクの位置がずれている
    • コンパクトモード時のメニューの位置の調整
      • class="top": 下に移動
      • class="card-section": 3px右にずらす
    • 「すべて表示」がブロックされないようにする
    • ハードブロックの場合、画像からリンクを外す
  • オプション機能の追加
  • その他
    • GoogleTopNewsとGoogleNewsTopという似た名前のクラスがあるので整理する。
      • 前者が検索結果の「トップニュース」、後者がGoogleニュースでグループ化された場合の最初の記事

テスト

  • ブロックのテスト(デフォルト)
    • Google Newsで画像があるもの(class="top"がある)
    • Google Newsでグループ化されているもの(class="card-section"がある)
    • 「すべて表示」が出る場合
  • ブロックのテスト(コンパクト)
    • Google Newsで画像があるもの(class="top"がある)
    • Google Newsでグループ化されているもの(class="card-section"がある)
    • 「すべて表示」が出る場合
  • メニューの位置のテスト
    • デフォルト
    • コンパクトモード
@ikemo3 ikemo3 added enhancement bug urgent labels Jun 23, 2019
@ikemo3 ikemo3 self-assigned this Feb 29, 2020
@ikemo3 ikemo3 removed the urgent label Feb 29, 2020
@ikemo3
Copy link
Owner Author

@ikemo3 ikemo3 commented Mar 30, 2020

Reopen this because there are tasks.

@ikemo3 ikemo3 reopened this Mar 30, 2020
@ikemo3 ikemo3 added this to the v1.9.1 milestone Mar 30, 2020
@ikemo3 ikemo3 closed this as completed Apr 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant