Skip to content

Conversation

@k0ta0uchi
Copy link
Contributor

@k0ta0uchi k0ta0uchi commented Oct 24, 2019

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

EB38HRgAcy

@ryu-s
Copy link
Member

ryu-s commented Nov 4, 2019

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

@k0ta0uchi
Copy link
Contributor Author

確かにDisplayNameだけでも良い気がしますが、ユーザーページに飛びたい場合、ユーザーIDがわかってるほうが良いので、ryu-sさんが仰る通り「DisplayName(Username)」にしてみました。
いかがでしょうか。
e64c761

@ryu-s
Copy link
Member

ryu-s commented Nov 11, 2019

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

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

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

@k0ta0uchi k0ta0uchi closed this Nov 12, 2019
@k0ta0uchi k0ta0uchi deleted the #124 branch November 12, 2019 05:06
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.

2 participants