-
Notifications
You must be signed in to change notification settings - Fork 26
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
追加: インターレース解除機能 #54
追加: インターレース解除機能 #54
Conversation
"yadif2x": "Yadif 2x", | ||
"top_field_first": "Top field first", | ||
"bottom_field_first": "Bottom field first" | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
改行ないのは気になる
app/util/menus/DeinterlaceMenu.ts
Outdated
} | ||
|
||
appendMenuItems() { | ||
this.append({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここから手書きで全要素記述してるけど、なんか自動列挙できないのかな
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EDeinterlaceModeとEDeinterlaceFieldOrderがconst enumなので、単純にfor inで列挙はできなさそうですね。
const DEINTERLACE_MODES = [
{ name: 'Disable', value: EDeinterlaceMode.Disable, label: $t('deinterlace.disable') },
...
];
みたいな配列を前で配列を定義しておいて、実際にメニューを追加する部分を共通化することだったらできそうです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
うん、まだそのほうが保守性良さそう
❌ build failed |
1 similar comment
❌ build failed |
動画ソースの動作確認はしました。 キャプチャボードでの動作確認がしたい。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MonsterX + PS4(1080i設定)でしっかりデインターレースのon/offで効果検証したのを確認した。
このpull requestが解決する内容
インターレース解除機能を追加します。
動作確認手順