Skip to content
This repository has been archived by the owner on Jul 15, 2024. It is now read-only.

Emakiが将来どうなるかのスケッチを追加 #31

Merged
merged 9 commits into from
Feb 11, 2024
Merged

Conversation

pxfnc
Copy link
Collaborator

@pxfnc pxfnc commented Feb 11, 2024

なぜやるのか

emakiのAPIが将来的にどうなるかの方針が今のリポジトリ上で参照できないため、ざっくりとした方針があったほうが実装の方針がブレづらくなって良い

やったこと

  • README.md にまだ存在しないEmakiのexampleを追加

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
pxfnc and others added 2 commits February 11, 2024 19:27
Co-authored-by: Yoshitaka Totsuka <y047aka@users.noreply.github.com>
Co-authored-by: Yoshitaka Totsuka <y047aka@users.noreply.github.com>
@pxfnc
Copy link
Collaborator Author

pxfnc commented Feb 11, 2024

propsではなく、controlsにしたほうが良さそう

README.md Outdated Show resolved Hide resolved
Copy link
Collaborator Author

@pxfnc pxfnc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Emakiから直接exportするのではなく、右側に出すControlはEmaki.Controlからexportするほうがよさそう

README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@pxfnc
Copy link
Collaborator Author

pxfnc commented Feb 11, 2024

📝 onChangeの部分は https://github.com/ymtszw/setem があればもっと楽になるが、exampleには載せず、将来いずれ書くドキュメントの付録にくっつける程度

@pxfnc pxfnc merged commit 1302030 into main Feb 11, 2024
@pxfnc pxfnc deleted the emaki-feature branch February 11, 2024 10:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants