#284 Add a copy mind map button

b310-digital / teammapper

Mindmapping made simple: Host and create your own mindmaps. Share your mindmap sessions with your team and collaborate on mindmaps.

https://teammapper.org

MIT License

202 stars 16 forks source link

#284 Add a copy mind map button #285

Closed sorenjohanson closed 2 weeks ago

sorenjohanson commented 2 weeks ago

closes #284

JannikStreek commented 2 weeks ago

thanks, already looking good 👍

UX:

Maybe we can add a small info note popup that says "Successfully cloned" ? Otherwise the user might think nothing happened - as the map looks the same. Only thing I could found is the Dialog in material ui.

PR:

Please Add a "closes #ISSUE_NUMBER_HERE" in the description

Test:

Maybe add one controller test, only for this method? we can add other controller tests in a separate PR.

sorenjohanson commented 2 weeks ago

UX:

Maybe we can add a small info note popup that says "Successfully cloned" ? Otherwise the user might think nothing happened - as the map looks the same. Only thing I could found is the Dialog in material ui.

Do we already have something similar for other use cases? Could integrate ngx-toastr otherwise, but I don't know if that's maybe a bit overkill.

JannikStreek commented 2 weeks ago

UX:

Maybe we can add a small info note popup that says "Successfully cloned" ? Otherwise the user might think nothing happened - as the map looks the same. Only thing I could found is the Dialog in material ui.

Do we already have something similar for other use cases? Could integrate ngx-toastr otherwise, but I don't know if that's maybe a bit overkill.

only material dialogs which are also quite heavy for just displaying a simply message. otherwise we could also just code/style or own, not sure if a library is suited just for small alerts.