Skip to content
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

Nicovideo対応 #66

Merged
merged 2 commits into from
Mar 31, 2015
Merged

Nicovideo対応 #66

merged 2 commits into from
Mar 31, 2015

Conversation

tkrkt
Copy link
Collaborator

@tkrkt tkrkt commented Mar 31, 2015

routes.rbとかいじってるけど、マージするだけでいけるのかな。

@pandeiro245
Copy link
Owner

image
ニコ動ってFLASHなんですね。
HTML5になったと思ってましたw
僕のSafariだとこうなりました…。

@pandeiro245 pandeiro245 merged commit 448f7e2 into pandeiro245:master Mar 31, 2015
@pandeiro245
Copy link
Owner

@tkrkt マージさせて頂きました!
ニコニコ動画にしかない作業音楽いっぱいあるので
嬉しいですっ!!

ただ、YouTube等と違ってauto_playがないので
下記のようにして取り急ぎの注意書きを追加させて頂きました。
521ee33

引き続きよろしくお願い致します!

@tkrkt
Copy link
Collaborator Author

tkrkt commented Apr 1, 2015

外部プレーヤー読込みAPIを解析してやれば、自動再生はできる、かもしれません。
その場合コメントは表示されませんが。
コメント取得APIあるので自前で再現できるけど、キリないなあ。

http://blog.mikumiku.moe/blog/2014/08/nicothumbwatch4html5-howtowork.html

@pandeiro245
Copy link
Owner

集中して仕事しないといけないわけだからコメントは不要かとw
オートプレイに関してはこちらで管理しますね!
#67

@pandeiro245
Copy link
Owner

あと、権利問題的にはどうなんだろう…汗

class @Nicovideo
@fetch: (sm_id, callback) ->
console.log 'nicovideo', sm_id # sm_id: 'smXXXXXXX'
url = "./nicovideo/#{sm_id}"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tkrkt いまさら気づいたのですがこの行は不要?w

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

なんだこれw いらないですね。
他にもcoffeeに慣れていない部分がちらほら。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants