yurafuca / nicosapo Goto Github PK

タイトル通りです。どういう不具合か少し調べてみたんですけど、
コミュニティ画面の「(このユーザに) 自動入場: オン」で自動入場リストに登録すると、
ＵＲＬがコミュニティの番号ではなくそのコミュ主のユーザーIDの番号が登録されてるみたいです。
オプションの自動入場リスト（ユーザ）で確認しました。
例　コミュ番号 :co111111
　　ユーザーID :user/222222
　　登録されるURL :co222222

https://twitter.com/chinopon/status/985936614307196928

とても便利なものを見つけたと思い、愛用させていただいております。「未来の公式」リストなんですが、予約タブ同様に放送日時順になるととても嬉しいです。ご検討いただければ幸いです。

https://twitter.com/lemon0541/status/962528608374767616

http://live2.nicovideo.jp/watch/lv309793175

1.非入会、未フォロー、ランキング外→表示なし
2.非入会、未フォロー、ランキング入→公式放送タブ
3.非入会、既フォロー、ランキング外、無料放送→ユーザー放送タブ
4.非入会、既フォロー、ランキング入、無料放送→ユーザー、公式タブ両方
5.非入会、既フォロー、ランキング入、有料限定→公式放送タブのみ
6.入会済、既フォロー、ランキング入、有料限定→公式放送タブのみ
7.入会済、既フォロー、ランキング外、有料限定→条件にあう放送がなくて確認できず

https://twitter.com/ramia_2525/status/961959039524511748

https://twitter.com/sakurapudding23/status/978119843919482882

こちら配信主をしていまして、ゲームをしながらコメビュとブラウザを画面上に常時展開しているのですが、その際のブラウザサイズが横幅480pix程なのです。
ブラウザの横幅が小さい場合、にこさぽを開くとにこさぽウィンドウの左端が見切れてしまうのです。
もし対応可能であれば何卒、お願いします。

(このコミュニティに) 自動入場: オンについて
Chrome 65.0.3325.181(64-bit)
にこさぽ 3.1.1

現在フォロー中のコミュ(http://ch.nicovideo.jp/MillionRADIO)において
おまけ放送はフォロー中タブに表示され自動入場もしてくれるのですが
本放送のほうがフォロー中タブに表示されず自動入場も機能しません

本放送は全編(TS込み)無料
おまけ放送は最初の5分ほど無料であとは有料
となっています

ご確認のほどよろしくお願いします

v6.0.0 でコミュニティ・チャンネルベースの仕組みからユーザベースの仕組みに移行したが，自分のアイコンを設定していないユーザはポップアップに表示されるアイコンが真っ白になってしまう．配信者を判別できないし見栄えもよくないので直したい．

アイコンの url を取得する箇所:

アイコンを表示する箇所:

background-image は複数の url を指定できるので placeholder を 2 つ目の url として指定すれば解決できそう．

See: CSS での複数の背景の利用方法
https://developer.mozilla.org/ja/docs/Web/CSS/CSS_Background_and_Borders/Using_CSS_multiple_backgrounds

ニコサポにおいて、ポップアップウィンドウをピン止めしたい。

以下のウィンドウ全体を捜査しても、一時的にピン止めして閉じ名ようにしたいです。

一時的に複数の放送を開きたいときがあるが、現時点では開いては閉じ開いては閉じをしなければいけない状態です。
もし可能であれば、搭載していただけると嬉しいです。

https://twitter.com/sakurapudding23/status/962528984293556226

いくつか PR を出せたらと思っているので最新のソースを push していただけると嬉しいです

nicosapo は initial commit から 4 年経っていて，学生時代に書いたコードがそのまま動いている．通知と自動入場系のロジックは一部を TypeScript で書き直してテストも用意したがそれ以外は危うい箇所が多い．

たとえば現在の番組 id とユーザ id を取得する 'IdHolder' というクラスがあって，けっこう渋い気持ちになる．
https://github.com/yurafuca/nicosapo/blob/develop/src/javascripts/modules/IdHolder.js

インスタンス化しないと使えないので頻繁に RegExp.prototype.exec() が走っているし，変数名もよくない．v6.0.0 でユーザベースの仕組みに移行したので getCommunityid() は getUserId() とかにしたい．

安心して機能を修正できるように nicosapo を全体的にリファクタリングしたい．
とくにこだわりはないので，気になったところを自由に直すくらいの感覚で pullreq を投げてもらえればたぶんマージします．

v6.0.0 でコミュニティ・チャンネルベースの仕組みからユーザベースの仕組みに移行したため動かなくなってしまった．

v6.0.0 まで

1. getplayerstatus でいまの配信が終了しているか確認する
2. 終了していたら getplayerstatus で co111111 が配信しているか確認する
3. 配信していたらそれに移動する

v6.0.0

1. getplayerstatus でいまの配信が終了しているか確認する
2. 終了していたら getplayerstatus で user/111111 が配信しているか確認する
3. 配信していたらそれに移動する

getplayerstatus はユーザ id に対応していないため 2 で失敗していると思われる．

See:

ユーザの配信状況を取得できる api が存在すれば直せるはず．