xtne6f / EDCB

BonDriver based multifunctional EPG software
Other
165 stars 37 forks source link

ジャンル絞り込みのクリアボタン追加など #3

Closed tkntrec closed 10 years ago

tkntrec commented 10 years ago

2ch_EDCB/38/256でジャンル絞り込みの機能要望が出ているようなのですが、いかがでしょうか? 二つ目のコミット(大ジャンル名の追加)は不要なら外します。

xtne6f commented 10 years ago

ありがとうございます。一つ目頂きます。 二つ目も良いんですが(ニュース配下に特撮!?みたいになるので)、影響がリストボックスにとどまらないので ContentKindInfo.csにpublic string ListBoxView(←名前は自由に)プロパティを追加して SearchKeyDescView.xamlのリストボックスにDisplayMemberPath="ListBoxView"でいい感じになると思います。 // こんな事できるのねC#…初めて知りました

tkntrec commented 10 years ago

二つ目、直してみました。 勉強になります。(お手数おかけしました。)

番組情報と同様の表示に変更しました。こんな感じでしょうか?

大ジャンルの存在感が若干薄いような気もしますが‥。

ときに、ContentKindInfo.cs、1行にまとめた方が良いですか? return ContentName + (SubName=="" ? "" : " - " + SubName);

searchwindow

xtne6f commented 10 years ago

いい感じです。ジャンル絞込みの一般的な流れだと、確かにハイフンつなぎのほうが解りよいですね。

スタイルは適当に空気読みつつ自己流でいいと思います(なんだそりゃ 自分は三項演算子厨なので断然1行ですが 原作者は使用しない方向に徹している感じがします。 c9073daf8c5846737f5e93fc687a4f3f595ba498 で問題ないです。

tkntrec commented 10 years ago

断然1行ですか‥。 自分もどちらかというと1行なのですが、先のコミットでは一応行儀良くif使ってみました。 では、1行に変えて、修正をとりまとめたものを別ブランチでプルリクエストします。 (ちょっと修正が細かい修正が積んだときの作法がよく分からないので、一応こちらにもコミットしておきます。)