GraphiteEditor / GraphiteEditor/Graphite

Make MenuLists populate their contents when opened, not beforehand

オープン
#909 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
Performance Web
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。