Repository navigation
perf: Pico は使う部品だけ読み込む - #191
Merged
Merged
Conversation
|
Claude finished @5ym's task in 17s —— View job レビュー結果重大な問題は見つかりませんでした。
要確認
|
|
Claude finished @5ym's task in 11s —— View job レビュー結果重大な問題は見つかりませんでした。
要確認
|
@anyblades/blades 3.1.0-alpha.1 で部品ごとの入口ができたので、 全部入りの pico をやめて画面で使う部品だけを読み込む。 レイアウトの CSS が 90.1kB(gzip 12.7kB)から 60.3kB(gzip 8.9kB)になる。 外したのは画面に該当する要素もクラスも無いもの: - layout: container / grid / overflow-auto - forms: input-color / input-datetime / input-file / input-range / input-search - components: accordion / card / dropdown / group / loading / modal / progress / tooltip content の misc は画面に無くても残す。hidden 属性を隠す規則がここにあり、 外すと display を持つこちらの規則に付けた hidden が効かなくなる。 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01ShKjeB5A9VtgvRpkCCgym8
|
Claude finished @5ym's task in 10s —— View job レビュー結果重大な問題は見つかりませんでした。
要確認
|
プレリリースの間は規則がどの部品にあるかが変わり得て、そうなると build は通ったまま見た目だけ変わる。 まとめの PR に入れたままだと、ほかの依存の自動マージまで止まるので、Blades だけ別の PR にする。 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01ShKjeB5A9VtgvRpkCCgym8
|
Claude finished @5ym's task in 11s —— View job レビュー結果重大な問題は見つかりませんでした。
要確認
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@anyblades/blades 3.1.0-alpha.1(npm の
canary)で部品ごとの入口ができたので、全部入りの
@anyblades/blades/picoをやめて、画面で使う部品だけをlayer(pico)付きで読み込む。:rootでの--pico-*の上書きはそのまま。大きさ
レイアウトの CSS(
_app/immutable/assets/0.*.css、全ページで読む)残りで最大なのは
variables(明暗の色の定義。残した Pico の半分弱)で、これは外せない。残したもの
bodymain)/ section(<section>)/admin)/ embedded-content(<svg>)/ code(見本の<pre>)/ figure(見本の<figure>)/ misc(画面には無いが、[hidden]をdisplay: none !importantで隠す規則がここにある。外すとdisplayを持つ自前の規則に付けたhiddenが効かなくなるので残す)inputbuttonlabel)/ checkbox-radio-switch(role="switch")<nav>)[disabled]のボタン、reduced-motion)外したもの
画面に該当する要素もクラスも無いもの。
srcを grep して確認した。.container.grid.overflow-autoは使っていない。幅は自前の.page)typeの入力が無い)details)/ card(article)/ dropdown / group(role="group"role="search")/ loading(aria-busy。読み込み中は自前の.spin)/ modal(dialog)/ progress / tooltip(data-tooltip)使い始めるときに足せるよう、
app.cssの先頭に一覧をコメントで残した。3.1.0-alpha.1(先行版)について
部品ごとの入口は今のところ先行版(
canary)にしか無いので、版を固定している。先行版の間は規則がどの部品にあるかが変わり得て、build は通ったまま見た目だけ変わることがあるので、renovate.jsonで@anyblades/bladesだけ自動マージを止め、まとめの PR から外して別の PR にした(ほかの依存の自動マージは止めない)。3.1.0 が出ても版の固定のまま上がるので、^3.1.0へは手で戻し、この規則も消す。確認
bun i --frozen-lockfile→bun run build(Dockerfile と同じ手順)が通るsvelte-check0 errors / 0 warnings/adminの部品(スイッチ・無効のスイッチ・チェック・各種ボタン・無効のボタン・注意書き・札・表)を差し込んだもの 明・暗。違いは回転中の.spinの角度だけbiome checkはsrc/app.cssに整形の指摘が 1 件出るが、main の時点から出ているもので今回の行ではない🤖 Generated with Claude Code
https://claude.ai/code/session_01ShKjeB5A9VtgvRpkCCgym8