Repository navigation
Conversation
4 tasks done
haranishi
marked this pull request as ready for review
October 11, 2026 19:34
Contributor
Author
|
@echelon レビューをお願いします。 Closes #1251。スポイトの読み取り元を編集可否から独立させ、ロック中や隔離の外側の図形からも色・外観を取得できるようにしました。クリック先へ書き込むAlt操作と通常の選択は、従来の編集制限を保ちます。 コミット 公開根拠の確認を待つDraft #1241 から、図形・画像の参照修正を独立させています。 |
This branch has not been deployed
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.
What and why
Closes #1251
スポイトが色を読むときに編集用の当たり判定を使っていたため、ロック中の図形・レイヤーや、隔離モードの外側の図形を参照できませんでした。読み取り用の当たり判定を追加し、通常クリック、Shiftの色指定、Shift+Altの外観追加、画像の色とグラデーションの停止点の色指定に使います。
クリック先を書き換えるAlt操作には編集用の判定を使います。参照元と選択は維持し、隠れた図形は参照しません。色と外観の変更はUndo一回で戻せます。公開 HelpXは図形・画像の色を別の図形へ適用する基本操作の参考です。ロック・隔離の条件は本イシューの期待と回帰テストに基づき、関連資料の点数・残り時間は維持しました。
Draft #1241 の読み取り用判定を独立させました。アートボードの背景色を取得する挙動は同PRで公開根拠を確認中で、このPRの対象は図形・画像の参照です。
How it was tested
cargo xtask ci: macOSの全7チェックが成功(5,386 passed / 0 failed / 4 ignored)。vectorcraft-cli perfを変更前後で実行し、両方とも全基準内。クリック判定は0.28 → 0.28 ms(基準2 ms)、400%ズームは10.08 → 9.94 ms(基準16 ms)。高負荷の警告があり、速度差は参考値です。c95999e2c0d6ee9e4997d00347803d55336e405cを指定した Linux + wasmの全チェックが成功しました(5,371 passed / 0 failed / 4 ignored)。上流の Affinity 3 corpus も成功しました。Checklist
cargo xtask cipasses (fmt, clippy, tests, assets, brands, layers, wasm).