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

ワールドのサムネイル画像を表示する機能を追加 #6

Closed
wants to merge 2 commits into from

Conversation

yokra9
Copy link
Contributor

@yokra9 yokra9 commented Feb 14, 2021

いつも便利に利用させていただいております。
名前があやふやなワールドを探すときに、サムネイル画像を表示できると便利かな?と思ったのでフォークして機能追加してみました。もしよければマージをご検討ください。


サムネイル画像が表示されるまでのフロー

  1. DataGridの[Show]ボタンをクリック
  2. イベントハンドラ内でワールド情報APIをコール
  3. 戻ってきたJSONをパースしてサムネイル画像URLを取得
  4. モデルのサムネイル画像を更新し、PropertyChangedを通知
  5. DataGridにサムネイル画像が表示される

@nukora
Copy link
Owner

nukora commented Feb 14, 2021

PRありがとうございます!
とても良い機能だと思います!
このままマージしたいのですが、現在2021.1.2で追加されたインバイトメッセージ表示機能などの検討をしており、UIが複雑になる事を懸念しております。
そこで、別画面で詳細画面を作り、使用頻度の低い機能をそちらに追加するという事を考えています。
おそらくそちらへの追加となってしまうかと思われます。
せっかく作って頂いて申し訳ございませんが、ひとまず保留とさせてください😔

@yokra9
Copy link
Contributor Author

yokra9 commented Feb 14, 2021

ご確認ありがとうございます。

このままマージしたいのですが、現在2021.1.2で追加されたインバイトメッセージ表示機能などの検討をしており、UIが複雑になる事を懸念しております。
そこで、別画面で詳細画面を作り、使用頻度の低い機能をそちらに追加するという事を考えています。

了解いたしました。確かにメイン画面が複雑化しすぎるのはよくないですよね。
詳細画面を追加したコミットがプッシュされましたら、本機能を詳細画面に付け替えてこちらのPRにコミットを追加しようと思います。

1.1.2の更新を取り込み
@nukora nukora mentioned this pull request Feb 19, 2021
@nukora
Copy link
Owner

nukora commented Mar 7, 2021

遅くなってしまい申し訳ございません。
こちらのPRをマージするのは難しかった為、私の方で同等の機能を実装という形になってしまいました。
機能の提案、PRをして頂き本当にありがとうございます……!

@nukora nukora closed this Mar 7, 2021
@yokra9 yokra9 deleted the feature/show_world_thumbs branch March 7, 2021 13:20
@yokra9
Copy link
Contributor Author

yokra9 commented Mar 7, 2021

v1.2リリース確認いたしました!わかりやすい表示形式・綺麗なコードで再実装されており嬉しかったです。ありがとうございました!

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