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

CWがサーバートップのTLに隠されず表示されてしまう #11221

Closed
nakajima0528 opened this issue Jul 9, 2023 · 7 comments · Fixed by #14156
Closed

CWがサーバートップのTLに隠されず表示されてしまう #11221

nakajima0528 opened this issue Jul 9, 2023 · 7 comments · Fixed by #14156
Labels
⚠️bug? This might be a bug

Comments

@nakajima0528
Copy link

💡 Summary

ログインしていないサーバーのトップに表示されるTLピックアップにおいて、本来CWで隠されている内容が表示されてしまう

🥰 Expected Behavior

CWは本来のTLと同じく隠された状態で表示される

🤬 Actual Behavior

無題
↑TLでの表示(隠すボタンを押すことで展開されている)

無題
↑サーバートップにピックアップされた際の表示(CWで伏せた部分が隠されず表示され、注釈もなくなっている)

📝 Steps to Reproduce

  1. ノート投稿画面で「隠す」を選ぶ
  2. 注釈と伏せたい部分を入力する
  3. 投稿する

📌 Environment

Misskey version:13.13.2
Your OS:Windows10
Your browser:chrome

以前投稿したissueと関連するかもしれません(CWがプレビューで伏せテキストと注釈が逆になる #10622)。

@nakajima0528 nakajima0528 added the ⚠️bug? This might be a bug label Jul 9, 2023
@tai-cha
Copy link
Member

tai-cha commented Jul 21, 2023

この現象でR-18なノートが普通に伏せられずにトップに出てたのでびっくりした

@EbiseLutica
Copy link
Contributor

改善案いくつかあると思ってて

  • CWのUIを表示するよう改善する
  • そもそもCWのついたノートは除外する

かなあって感じ CWでがついている理由を考えると、個人的には後者を推します

@acid-chicken
Copy link
Member

後者は CW 使用によるエンゲージメントのディスアドバンテージを促進させるので反対

@u1-liquid
Copy link
Sponsor Contributor

非ログイン時に流れるパブリックな画面なのでCWの投稿は流れない方がいいと思います。
なんならNSFW画像が付いてるノートも流れない方がよさそう

@syuilo
Copy link
Member

syuilo commented Jul 21, 2023

NSFW除外してた気がしたけど気のせいか

@u1-liquid
Copy link
Sponsor Contributor

notes/featuredを描画してるだけじゃなかったっけ特に除外処理はしてないかも
ちなみに非ログイン時の「タイムラインを見てみる」にもNSFWは流れてるかも

@EbiseLutica
Copy link
Contributor

後者は CW 使用によるエンゲージメントのディスアドバンテージを促進させるので反対

非ログインページの右がエンゲージメントにそこまで影響を与えているかというと個人的には何とも言えない感触ある。

ただ、よくよく考えたら隠す必要はないね。ハイライトに載っている以上パブリック公開範囲だし。
むしろサーバーを探している人に対し、恣意的に編集されたハイライトを見せる方が悪手かもと思ったので、やはりCWボタンの表示がいいかも

(ユーザー名やアイコンも出したほうが良くない?とは思うけど)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants