GraphiteEditor / GraphiteEditor/Graphite
Make MenuLists populate their contents when opened, not beforehand
- 主要言語
- Rust
- スター
- 27.2k
- フォーク
- 1.3k
- 平均マージ
- 20時間 5分
- マージ済み PR(30日)
- 57
説明
Sending the full MenuList contents each time a layout is sent seems to be causing massive performance slowdowns and memory allocations that have to later be GC'd. We want a MenuList to request its contents from the backend when it's opened, rather than being sent ahead of time in the layout that contains the MenuList. Profiling showed that MenuList creation was one cause of the editor being very slow (back when we were using Vue and its virtual DOM, at least— we'd need to re-profile it under Svelte to get a proper point of comparison).
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
まず、MenuList の内容がレイアウトにどのように含まれるか、またメニューを開いたときにバックエンドがそれらをどのように提供できるかを追跡します。現在の作成およびレイアウトのフローをプロファイリングし、その後、開かれていない MenuLists が完全な内容を送信しなくなっていることと、1 つを開いたときにその内容が引き続き正しく読み込まれることを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- rust
- 領域
- frontend, performance
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100