search

CommentViewerCollection / MultiCommentViewer

いろんな配信サイトのコメントを表示できるコメビュです
https://ryu-s.github.io/app/multicommentviewer
GNU General Public License v3.0
180 stars 44 forks source link

#124 TwitchのDisplayNameに対応 #125

Closed k0ta0uchi closed 4 years ago

k0ta0uchi commented 4 years ago

暫定的に対応してみました。 DisplayNameがUserIdと同じ場合は表示しません。 本当は別カラムに表示したいですが取り急ぎ…。

EB38HRgAcy

ryu-s commented 4 years ago

Twitchの機能を実装する際にDisplayNameの扱いをどうするか迷ったのですが、放置したまま忘れていました。 改めて考えてみると、表示するのはDisplayNameだけでも良いような気がします。 Usernameが必要な場面ってあるのでしょうか？もし無ければDisplayNameだけにしたいです。 Usernameがあった方が良いという場合でもDisplayName優先で「DisplayName（Username）」の方が見やすいと思います。

k0ta0uchi commented 4 years ago

確かにDisplayNameだけでも良い気がしますが、ユーザーページに飛びたい場合、ユーザーIDがわかってるほうが良いので、ryu-sさんが仰る通り「DisplayName（Username）」にしてみました。 いかがでしょうか。 https://github.com/CommentViewerCollection/MultiCommentViewer/pull/125/commits/e64c7610390d7928509d2400f554707fbda84dd2

ryu-s commented 4 years ago

ユーザーページに飛びたい場合

コンテキストメニューに「ユーザページに飛ぶ」もしくは「ユーザIDをコピーする」項目を追加するのがベターだと思います。

あとプルリクはコミットを1つにまとめた上で、最新のdevelopブランチを指定していただけるとうれしいです。