Repository navigation
Conversation
|
@adjetey0 is attempting to deploy a commit to the playcanvas Team on Vercel. A member of the Team first needs to authorize it. |
|
That is very drastic change to UX, and we definitely would not want this to be merged, as pre-selecting parent entity - is a conscious decision by user. This would be very annoying and inconvenient especially on larger scenes with many entities. |
|
Hi Maksims,
Thank you for taking the time to review my pull request. You make a
completely valid point about the UX—I hadn't fully considered how
disruptive this would be on larger scenes where users intentionally want to
drop items into specific selected folders. Taking away that ability would
definitely cause unnecessary reparenting work.
I apologize for the oversight on the workflow! I will go ahead and close
this PR so it doesn't clutter the queue.
Thanks again for the guidance and for all the hard work the team puts into
the editor.
… Message ID: ***@***.***>
|
|
There is another option: if particular behavior is very split, close to 50/50 or even 70/30, and is very common action, then it is worth considering a settings toggle, where default value should follow existing UX, but provide users ability to change such behavior. The challenging part - is to figure out if it is significant part of users that would switch to such behavior, as discovering some settings toggles - is almost never done by users. |
|
A settings toggle is a great compromise! I completely understand the concern about discoverability, but it gives power users the option without breaking the default UX. If you are open to it, I would love to take a stab at implementing that toggle. Could you point me to the file where editor settings/preferences are typically added? |
|
Also, if you select template before dragging it into the scene, it will be created under the root by default. Which is an easy to muscle memory. |
Fixes #257
What's Changed
Updated viewport drop logic for models, templates, and sprites to attach to the scene root instead of the currently selected entity.
Checks