k0sukey / har

Simple HTTP Archive viewer
1 stars 0 forks source link

Preview を軽くしたい #4

Open k0sukey opened 5 years ago

k0sukey commented 5 years ago

内容によっては激重になる。hljs 入れるとまじで重い。

og24715 commented 5 years ago

雑に書いたから React のお作法に則っていない可能性ある

何もない状態からすべてを描画しようとすると重くなるのはしょうがないかなという気がしますが、フィルターの変更で再描画の必要ない行も再描画されてそうなのでここは改善出来そう。

k0sukey commented 5 years ago

フィルターの変更で再描画の必要ない行も再描画されてそう

やはりね。ちょっとその方向でお願いしたい 🙏

og24715 commented 5 years ago

https://github.com/k0sukey/har/blob/9b8a111c592c3fbc5b7ada51f01294f6813719ef/src/components/Content/Request.tsx#L268-L270

Request コンポーネントが描画されるたびに毎回絞り込みされるのもったいないなあと思って useMemo で絞り込みの結果メモ化してみたら余計遅くなってしまった。

k0sukey commented 5 years ago

悲しい🤤

k0sukey commented 4 years ago

react-json-view の enableClipboard オプションが重い一因になっている模様。 enableClipboard={false} にしてあげるだけでだいぶ改善された。ただこれ便利なので、うーむ。