Impl/permalink

[westpaddy]

提出されたファイル見て回ってるときにブラウザで「戻る」したときの違和感がすごかったので、各ファイルにURLをマップしました。

[krtx]

Mac chrome と safari で確認しました。良いと思います。

[westpaddy]

URLが#/id123/example_report/file/./hoge/fuga.txtになってたのを#/id123/example_report/file/hoge/fuga.txtになるように直しました。

[westpaddy]

browse.rbでアプレットのレスポンス返す時のcontent-typeが抜けてる…

[westpaddy]

browse.rbのレスポンスにファイルタイプを含めるようにして、レスポンスのタイプ判定方法を変更しました。

[skymountain]

ちょっと教えてほしいんですけど (1) user.js で params.splat = '' としているのに (edited) どーして file_view.js ではちゃんとした splat がきてるんでしょう つまり，何故 file_view.js で splat は空文字列ではないんでしょう (2) user.js で /* としてますが，こうすると URL から path や splat なんかのパラメータはどうやって取ってきてるんでしょう． (3) splat という単語はどういう意味で使ってる？

[westpaddy]

splatはreact-routerの機能です。react-routerがURLからパラメータをマッピングしてくれます。 https://github.com/rackt/react-router/blob/master/docs/guides/path-matching.md 上の例はpropsでアクセスしてますが、こっちはRouter.StateをmixinしてgetParams()でアクセスしてます。

[skymountain]

ありがとうございます．上のページをちゃんと見てからまたコメントします．

[skymountain]

Breadcrum と FileBrowser(FileEntry) でディレクトリにアクセスする時で splat の扱いが異っており，このため Breadcrum からディレクトリをたどる(具体的には Breadcrum で上のディレクトリに移動してから FIleBrowser で再度ディレクトリを潜る)と途中で存在しないディレクトリに行けてしまいます．Breacrum はディレクトリを '/' なしでsplat を設定していますが，FileEntry では (suffix として) '/' ありで設定しています．

[westpaddy]

直しました。 「params使うかどうかの判定が～」というのがよくわからなかったのですがこれでいいんでしょうか。

[skymountain]

いいと思います． （ちなみに当初の想定していたのは各タブ情報の中に path や name と同じように params ハッシュを用意して，file にだけ { splat: '' } というのを用意しておき，StatusHeader.render で ローカル変数の params と tab.params || {} をマージさせることを考えてました．）

[skymountain]

マージします :+1: