thomasahle / TrinityProject

A mindblowing trainbased game for entire family
http://thomasahle.github.com/TrinityProject/
GNU General Public License v3.0
8 stars 3 forks source link

Add a way to DELETE a component already inserted #31

Closed MattPellegrini closed 12 years ago

MattPellegrini commented 12 years ago

I think this works now, I'll work on better controls rather than just clicking tomorrow. I can't seem to brake it at the moment, anyone want to try?

thomasahle commented 12 years ago

This works really well Matt! I may be slightly brain damaged, but ctrl+click was the first thing I instinctively tried. (After looking for a 'delete' button). Now we just need to change it all to drag'n'drop ;-) It also works very nicely, that you can only delete a split once it's empty.

On Sun, Jun 24, 2012 at 1:26 AM, Matt reply@reply.github.com wrote:

I think this works now, I'll work on better controls rather than just clicking tomorrow. I can't seem to brake it at the moment, anyone want to try?


Reply to this email directly or view it on GitHub: https://github.com/thomasahle/TrinityProject/issues/31#issuecomment-6529465

Mvh. Thomas

MattPellegrini commented 12 years ago

Thanks :)

yh ctrl-click would work, It will all eventually need to be drag and drop so the keyboard controls are only important for the HTML version. I was thinking that for drag and drop we could have it so that when a component is dragged from the track it deletes the component and then the user is dragging the component as if they had not inserted it yet.

I think this is better than having to drag it to a trash icon as swapping the order of components becomes easy (at least non composite components)

Maybe also have the option of selecting a delete tool from the component bar so that just clicking on a component deletes it. This will make things easy if the user can't find space off the track to drop the component (i.e to not insert it)

Also we need the track to help the user out with placement. So when they are dragging a component, all ID components should glow or something to let the user know that they can place that here. - But this is a separate issue on the issue tracker, and one I'll get working on today, I digress.

Hope the Internship is good

MattPellegrini commented 12 years ago

Think this is done.