Describe the bug
コンテキストメニューの「新しいタブを画像で開く」などが音声や動画を右クリックした際にも表示される。
音声や動画は画像ではないので、それに合わせた文言であるべき。
To Reproduce
https://developer.mozilla.org/ja/docs/Web/HTML/Reference/Elements/audio
を開き、html組み込みの音声を表示する。
音声部分を右クリックしコンテキストメニューを表示する。
この時、以下のように画像向けのコンテキストメニューが表示される。
Expected behavior
「新しいタブを音声で開く」「名前を付けて音声を保存」といった、適切な名前になるべき。
Your Environment
- Chronos version: 15.1.135.0 (予定)
- ThinApps version: Any
Your Configuration
Your TRACE log (Chronos_trace.log)
Additional context
https://github.com/ThinBridge/Chronos/blob/472195d2a9e29bb834bc683af0219a57f3f1fac1/client_handler.cpp?plain=1#L396C1-L396C61
ここで if (Flg & (CM_TYPEFLAG_MEDIA | CM_MEDIATYPE_IMAGE)) != 0 をしているのだが、CM_MEDIATYPE_IMAGEは cef_context_menu_media_type_t型で、Flgの型であるcef_context_menu_type_flags_t型ではないので、この比較が間違えている。
cef_context_menu_media_type_t mediaType = params->GetMediaType();
if (mediaType == CM_MEDIATYPE_IMAGE)
{
...
}
のように、別途GetMediaTypeを実行して確認する必要がある。
ただし、単に上記のように修正すると音声や動画を新しいタブで開くといったことができなくなってしまうので、何らかの対応が必要。
Describe the bug
コンテキストメニューの「新しいタブを画像で開く」などが音声や動画を右クリックした際にも表示される。
音声や動画は画像ではないので、それに合わせた文言であるべき。
To Reproduce
https://developer.mozilla.org/ja/docs/Web/HTML/Reference/Elements/audio
を開き、html組み込みの音声を表示する。
音声部分を右クリックしコンテキストメニューを表示する。
この時、以下のように画像向けのコンテキストメニューが表示される。
Expected behavior
「新しいタブを音声で開く」「名前を付けて音声を保存」といった、適切な名前になるべき。
Your Environment
Your Configuration
Your TRACE log (Chronos_trace.log)
Additional context
https://github.com/ThinBridge/Chronos/blob/472195d2a9e29bb834bc683af0219a57f3f1fac1/client_handler.cpp?plain=1#L396C1-L396C61
ここで
if (Flg & (CM_TYPEFLAG_MEDIA | CM_MEDIATYPE_IMAGE)) != 0をしているのだが、CM_MEDIATYPE_IMAGEはcef_context_menu_media_type_t型で、Flgの型であるcef_context_menu_type_flags_t型ではないので、この比較が間違えている。のように、別途
GetMediaTypeを実行して確認する必要がある。ただし、単に上記のように修正すると音声や動画を新しいタブで開くといったことができなくなってしまうので、何らかの対応が必要。