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

ツイートの表示回数を非表示に #31

Closed
wants to merge 1 commit into from

Conversation

k-khr
Copy link

@k-khr k-khr commented Dec 23, 2022

日本時間 2022/12/23 の朝に追加された、「ツイートの表示回数」のラベルを非表示にするよう変更しました。

Before:
hide_impression_before

After:
hide_impression

@jumang4423
Copy link

jumang4423 commented Dec 23, 2022

これは本当にマージしてほしいです(マージで)

@yusukesaitoh
Copy link
Owner

@k-khr
ありがとうございます!早い!
自分もこの対応で良いと考えていたのですが、Twitterでアンケートをとったところ「インプレッションだけ隠すチェックを新設」してほしいニーズもありそうで、明日午後にその最終結果もふまえて対応決められればと思っています。
決まったら対応即ストアの審査に出したい所存です:bow:

@Toukotsu
Copy link

@k-khr さん
scss の変更について @yusukesaitoh さんの方針/目的と一致するかは判断つきかねますが、
設定ポップアップの説明テキストは変えておいた方がマージしやすいかもしれません

"message": "Hide numbers of replies, retweets, and likes"

    "message": "Hide numbers of views, replies, retweets, and likes"

"message": "ツイートへのリプライ数、リツイート数、いいね数を隠す"

    "message": "ツイートの表示数、リプライ数、リツイート数、いいね数を隠す"

など

// それはそれとしてこの機能はとても有用だと思うので実装してほしい

@Toukotsu
Copy link

@yusukesaitoh さん
おっと背景読めず&同時投稿失礼しました

ツイッターのアンケート拝見しましたが、インプレッションの非表示を単独で行いたい方が存外多かった…
リプRTいいねが纏まっているならインプレッション数もまとめるほうが自然かと個人的には感じますが
ご判断のほどお待ちしておりますー

@yusukesaitoh
Copy link
Owner

@k-khr
アンケートの結果( https://twitter.com/site_oh/status/1606125286868848642 )インプレッションのみ隠したいユーザが想像よりも多かったため、項目を分けてデフォルトONで新設する方向としました。
従って、こちらのプルリクはCloseとさせてください。

修正版はストアに提出済みです。

問題発生後すぐに投げていただいたことでアップデートへのやる気がかなり出ました。
ありがとうございました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants