1c7 / Youtube-Auto-Subtitle-Download

(no longer maintained)(停止维护)功能:下载 Youtube 字幕,Download Youtube Subtitles(备注:因为不能带来收入,而且我自己也没有下载字幕的需求了,所以不再维护,除非情况有所改变)
https://greasyfork.org/zh-CN/scripts/5368-youtube-subtitle-downloader-v20
820 stars 141 forks source link

Fix YouTube URLEncoding the subtitle #4

Closed cnbeining closed 8 years ago

cnbeining commented 8 years ago

刚才PR开反了。

总之加一个试试。

按道理,如果源文件没有被URLEncode,也不会造成新的混乱。

1c7 commented 8 years ago

好,我在测,测好了 Chrome 53 下面没问题了我就 Merge 。 谢谢哈

1c7 commented 8 years ago

能把有问题的那个 Youtube 视频的 URL 提供一下吗? 毕竟你肯定是使用时发现下载下来的字幕不行才去看源码的。 我想测测看 :D

cnbeining commented 8 years ago

https://www.youtube.com/watch?v=rN13Kjnm_FY https://www.youtube.com/watch?v=rN13Kjnm_FY

在 2016年9月5日,上午12:47,Cheng Zheng notifications@github.com 写道:

能把有问题的那个 Youtube 视频的 URL 提供一下吗? 毕竟你肯定是使用时发现下载下来的字幕不行才去看源码的。 我想测测看 :D

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/1c7/Youtube-Auto-Subtitle-Download/pull/4#issuecomment-244658158, or mute the thread https://github.com/notifications/unsubscribe-auth/ABfRgirfgdIsnFgFdAeYRPNv7aBntQZFks5qm574gaJpZM4J0vFz.

1c7 commented 8 years ago

image image

1c7 commented 8 years ago

image image

目前发现的一点点小问题就是换行不对了。这一层楼是对的。上一层楼不对。

cnbeining commented 8 years ago

有个.

image

1c7 commented 8 years ago

好,收到,我去看看。

1c7 commented 8 years ago

我想我明白了,我用的是 \r\n
在windows 下面当然不会有什么问题, 看截图你像是用的 Linux,

标签应该是编辑器判断到是 windows 换行自动加的。 解决方法:我想现在的编辑器应该都有提供这种换行符的转换的。 Windows -> Linux 的
cnbeining commented 8 years ago

按理说Linux也可以处理的吧。MplayerX没问题。

1c7 commented 8 years ago

嗯,那既然从视频播放的角度来说,完全没问题,不影响正常观看,那就暂时不改了吧? \r\n 换成 Linux 的 \n (Linux 好像就只是一个 \n 哈,不知道记错没)
目前看来唯一的好处就只是不显示 CR 而已。(直接查看字幕文件的情况下)

你怎么想?

感谢PR,我会在 README.MD 放个 contributor

cnbeining commented 8 years ago

我当时遇到的问题是整个subtitle好像被URLEncode过了。

保险起见 可以先不merge 看看是不是偶发事件 或和某个浏览器版本或插件冲突有关。

谢谢。

1c7 commented 8 years ago

好,那这个 PR 我暂时先关掉,避免对别人造成困惑。 一旦有问题了你马上告诉我,我们一起看看是啥问题,解决完了你再 PR 然后 merge。 没问题吧?

cnbeining commented 8 years ago

没问题 感谢您的时间。