Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

路線図エディタの実装 #23

Merged
merged 2 commits into from
Jul 2, 2023
Merged

路線図エディタの実装 #23

merged 2 commits into from
Jul 2, 2023

Conversation

hutinoatari
Copy link
Member

close #14
まず駅コンポーネントの追加だけ実装した。
次は追加済コンポーネントに対する編集機能を付ける予定。

編集結果はJSONファイルにPropデータだけ吐き出して地図側から参照してもらうか、もしくはRailRoadMap.tsxそのものを書き換えるのとどっちがいいだろうか?

@hutinoatari hutinoatari requested a review from csenet June 30, 2023 11:31
@hutinoatari
Copy link
Member Author

そもそも本番環境にビルドしたらtsxファイルが存在しないだろ!前者の方法で進めます。

Copy link
Member

@csenet csenet left a comment

Choose a reason for hiding this comment

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

LGTM with nits
エディタの原型を実装していただきありがとうございます!
ユーザーの操作に関してや今回実装したい管理画面のUIに関しては別のIssueで追加していきたいと思います。
少しめんどくさいかもしれませんが、駅をドラッグできると嬉しいなと感じました。
Rootにnode_modulesが残っていたのでそのファイルだけ削除お願いします。
出力形式に関しては、JSONまたはXMLなど任意のフォーマットで出力できるようにして、表示がわでいい感じにParseできれば嬉しい気がします。

@hutinoatari
Copy link
Member Author

ドラッグ操作はRailコンポーネントが厳しそう(節点を移動させたくなるので)。
https://konvajs.org/docs/react/Drag_And_Drop.html
例えばKonva.jsはdraggableが実装してあってReactにも対応してるみたいです。

@hutinoatari
Copy link
Member Author

konvaでドラッグできるようにした。

  • Textの中央寄せが効いていない。
  • ドラッグによる位置の変更がPropsに反映しない(追加済みコンポーネントの編集機能をまだ追加していないので)。

@hutinoatari
Copy link
Member Author

てなわけで最低限の実装ヨシ!

@hutinoatari hutinoatari merged commit 2c41e21 into main Jul 2, 2023
@csenet
Copy link
Member

csenet commented Jul 3, 2023

ありがとうございます!
RailComponentはドラッグ可能にするのはつらそうなので、マウスで引けるようにしてもらえると良いかもしれません

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.

管理画面のSVGに対応したエディタを作る
2 participants