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

chore(frontend): turn off enableCondensedLine by default #14644

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

KisaragiEffective
Copy link
Collaborator

addresses #14642

What

#14642

Why

画面の横幅に押しつぶされるのは多分不本意だと思う

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Sep 29, 2024
Copy link

codecov bot commented Sep 29, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 19.62%. Comparing base (a3d236c) to head (ad1e2d8).
Report is 203 commits behind head on develop.

Files with missing lines Patch % Lines
packages/frontend/src/store.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #14644       +/-   ##
============================================
- Coverage    41.74%   19.62%   -22.13%     
============================================
  Files         1549      715      -834     
  Lines       196555   101377    -95178     
  Branches      2767      985     -1782     
============================================
- Hits         82055    19893    -62162     
+ Misses      113939    80940    -32999     
+ Partials       561      544       -17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@syuilo
Copy link
Member

syuilo commented Sep 29, 2024

途切れるよりこっちの方が体験良くないかしら

@KisaragiEffective
Copy link
Collaborator Author

先頭の文字が読めたほうが前方一致できていいと思う
第三者の意見も聞きたいところ

@Sayamame-beans
Copy link
Collaborator

Sayamame-beans commented Sep 29, 2024

私は潰れて欲しくないですね(読めなくなるケースの方が多そう)
どうせ読めないなら初めだけでも見える方が良い気がします

@syuilo
Copy link
Member

syuilo commented Sep 29, 2024

意見を集めるためにもデフォルトでオンにする必要がある

@KisaragiEffective
Copy link
Collaborator Author

誰もがGitHubに意見を書くわけじゃないからあまり説得力がないと思う

@KisaragiEffective
Copy link
Collaborator Author

(GitHubに限らず任意の場所で)

@fruitriin
Copy link
Contributor

fruitriin commented Sep 29, 2024

げむすきとみすてむずに導入されてるからアンケートとって

@syuilo
Copy link
Member

syuilo commented Sep 29, 2024

慣れの問題もあるから1ヶ月くらい経過してから取りたいわね

@fruitriin
Copy link
Contributor

体感では、PCだと気にならないけどスマホだとそもそも文字が圧縮されすぎて読めなくなる
読めないので名前を短くする人が多数現れたくらい

@Sayamame-beans
Copy link
Collaborator

見えないものに慣れも何も無いと思ったりします

@syuilo
Copy link
Member

syuilo commented Sep 29, 2024

見えなくなるケースが多くなさそう

@tai-cha
Copy link
Contributor

tai-cha commented Oct 1, 2024

読めなくなるラインの文字数が分かればあるサーバーにおける登録されてるユーザーの文字数とかで機械的になんとなくどれぐらい読めなくなる人がいそうかは分かる気がする

※個人的にはおおよそデスクトップ版やタブレットでのデフォルト表示では名前の表示に確保される幅が長いけれどデッキ表示やスマートフォンからの利用だとそうでもないといった印象

@anatawa12
Copy link
Member

あまりおえててなかったけど、ユーザ名欄のenableCondensedLine、acct含めた長さで幅が小さくなるのはacctが長くなりがちなのを考えると読めなく/読みづらくなるパターンが結構あると思う。(14642 のスクショからの推測)

少なくとも私はdeck UIをPCで、スマホではdefault UIを主につかってますが、リモートユーザの場合ほぼ普通の長さの範囲でacctの部分的省略が行われています。

(またオフトピ(14642の一部)になりますが 幅が半分だとモバイルだときつい気がします。0.7あたりくらいであってほしい気がする)

@tai-cha
Copy link
Contributor

tai-cha commented Oct 8, 2024

やはり肌感覚ではあるけど圧縮されているという声を聞くしそれの言われ方があまり好ましいと思われている気がしない

@tai-cha
Copy link
Contributor

tai-cha commented Oct 8, 2024

もしかしたらオフにするためのオプションが分かりずらい(翻訳されていない、その他の実験的機能にいる)のも一因ではあるのかもしれない

@syuilo
Copy link
Member

syuilo commented Oct 8, 2024

0.7状態がまだ提供されてないからそこから1ヶ月程度見る必要がありそう

@kakkokari-gtyih
Copy link
Contributor

やはり肌感覚ではあるけど圧縮されているという声を聞くしそれの言われ方があまり好ましいと思われている気がしない

0.7状態がまだ提供されてないからそこから1ヶ月程度見る必要がありそう

v2024.9.0リリース時点では圧縮の最小値の指定がなかったので名前/acctがどれだけ長くてもギュになっていた
v2024.10.0をリリースしてしばらくしたらまた評価が変わってくるかも

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants