Skip to content

feat: integrate planet arrangement#9

Merged
tknkaa merged 5 commits intomainfrom
feature/integrate-planet-arrangement
Mar 4, 2026
Merged

feat: integrate planet arrangement#9
tknkaa merged 5 commits intomainfrom
feature/integrate-planet-arrangement

Conversation

@faithia-anastasia
Copy link
Contributor

・PlanetArrangement/index.tsxの内容をSimulation/indextsxに統合した
・Simulation直下にcomponentsディレクトリを作成し、PlanetMeshやPreviewPlanetなどの関数を移動した
・index.tsxのuseEffectをuseControls内のonChangeに変更した
・useControlsのHelperにresetCameraPosition機能を追加した

@tknkaa
Copy link
Contributor

tknkaa commented Mar 4, 2026

細かい話だけど、PR の名前は Feature/ だとブランチ名っぽく見えるので、feat: のほうがいいかも。

@tknkaa tknkaa changed the title Feature/integrate planet arrangement feat: integrate planet arrangement Mar 4, 2026
@tknkaa
Copy link
Contributor

tknkaa commented Mar 4, 2026

ただ変更自体はよさそうなので merge しますね。

@tknkaa tknkaa merged commit 68bd74f into main Mar 4, 2026
1 check passed
@tknkaa tknkaa deleted the feature/integrate-planet-arrangement branch March 4, 2026 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants