verygoodlee

弹幕应该去这个仓库提[MPV-Play-BiliBili-Comments](https://github.com/itKelis/MPV-Play-BiliBili-Comments) 已经修复但还没合并 itKelis/MPV-Play-BiliBili-Comments#31

目前自定义播放器没法传cid，代码里写死了，只有mpv会带上cid，自定义播放器不会传 https://github.com/LuckyPuppy514/Play-With-MPV/blob/04a6387b0a0875c36249d5f336a0f4db3a09cf8f/play-with-mpv.user.js#L2022-L2028 不是很理解你为啥mpv还要自定义，没啥必要吧

可能有部分超长文件名影响体验，虽然遇到到的概率不高， 宽度限制确实也很难做的准，特别是非等宽字体，CJK字符看做2宽度也只是个估计。 新版的 后缀/协议 + 时间 的提示信息挺不错的，建议把 后缀/协议 转成大写，看起来应该更协调

windows native context menu needs to escape the `&` character. https://github.com/tsl0922/mpv-menu-plugin/blob/ce1f5c812dfbd9fc4db2280ee97edd6814a6ae4c/src/lua/dyn_menu.lua#L136-L138 if not escaped, URLs like `https://example.com?foo=1&bar=2` in the playlist will not show the `&` character.

> That should be done in then.`video/out/win32/menu.c` Yes, different menu implementations should handle their own text escaping

Since `menu-data` is filled by select.lua, we should reconsider whether subprocesses should be executed asynchronously https://github.com/mpv-player/mpv/pull/15936#discussion_r2188166698 synchronized execution will block the script thread, which means that `menu-data` cannot be updated...

I would recommend changing `MAX_LOG_LINES` to a script option, just like in most terminals allow users to set the maximum number of lines

https://github.com/mpv-player/mpv/pull/15956#discussion_r1967951003 characters outside the window are also visible after scrolling to the right, so the text must be cut based on utf8 characters, and `...` needs to be added after...