-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore: hide thumbnail if website is sensitive #12581
chore: hide thumbnail if website is sensitive #12581
Conversation
adminが選択したサイトを表示させない需要があるかもしれませんが私はすべてのsensitiveなサイトのthumbnailを非表示する必要はないと思っています。 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #12581 +/- ##
===========================================
+ Coverage 78.76% 78.90% +0.14%
===========================================
Files 955 957 +2
Lines 103873 104065 +192
Branches 8337 8358 +21
===========================================
+ Hits 81814 82114 +300
+ Misses 22059 21951 -108 ☔ View full report in Codecov by Sentry. |
(filter 処理をかけてでも)ブラーして表示する意味があるのかが知りたいという話 |
ブラーかかっててもある程度サムネイルが見れるの有用だと思うのですが |
サムネイルレベルの領域にブラーかけたらもう平均色プラスアルファ程度の情報量しかなさそう |
情報量多そう少なそうくらいはあるとは思うんですがうーん...表示させるのに固執する理由もないので非表示にしますか...... |
b4c14d3
to
c7cf813
Compare
せっかくリンク先がセンシティブであるという情報が取れているなら、サムネイル非表示にするだけではもったいないかも?(サムネイルが非表示になることは非センシティブな場合でも普通にあるから、それだけでリンク先がセンシティブであると判断はできない) まあそれは後々対応すれば良いかしら |
うちのモデレータからの要望もありとりあえずblurも何もなくサムネイルが表示されるのが困るって話だったので、とりあえず先に実装した感じです。 何かしら追加の表記を行う(思いつくのはチャンネルと同様のセンシティブオーバーレイを追加する?)のもいいと思います。 |
なんかセンシティブを表示するのは後で対応にしたいです |
🙏🙏 |
What
#12419 の対応準備。あとはsummarly側でsensitiveの判断を正しく行うようにするとtwitterについても動くようになると思います。
twitterについては詳しくないので私はしませんが、dlsiteなどは
mixi:content-rating
というmetaタグがあったのでこれの対応をsummarly側に投げようと考えてますWhy
個別対応はモデレーション的に負荷がかかる。
Additional info (optional)
Checklist