MonashAlexandria / snapapps

a visual programming language inspired by Scratch
http://snap.berkeley.edu
2 stars 5 forks source link

Default costume might be problematic #41

Closed eduvik closed 10 years ago

eduvik commented 10 years ago

The default 'turtle' costume is too big to effectively fit in some most cells. It should either be replaced with a smaller default costume, or be able to change the default costume. If this is not feasible, documentation should be advising users to create and set a costume early in their program

Craxic commented 10 years ago

Sounds like a fair point! I'll make some changes as soon as I can.

Craxic commented 10 years ago

OK. I've altered it so that the initial costume is much smaller and updates its size depending on the selected grid size. I decided if I went any smaller, it would be a little bit too small.

Let me know if this works for you.

eduvik commented 10 years ago

The size is better (though I think a couple more pixels could be shaved off), but the sprites don't 'centre' themselves in cells. When the 'move to neighbouring cell' block is executed, I'd expect that the sprite actually be in a cell (or at least most of the sprite in a cell), but currently it's not obvious which cell the sprite is currently in. This may be to do with what co-ordinate is the centre/origin/(0,0) on the sprite perhaps?

image

Also, I think a default costume that is fatter might be useful, so it fills the cell better. Perhaps a fatter arrow, or a circle with an arrow inside it?

Craxic commented 10 years ago

OK, I'll see what I can do!

aidanlane commented 10 years ago

Thanks Matt.

Btw, you can set the 'centre', can't you? A universal way of scaling the sprite could be good too, just with a better UI than the old Scribble.

Craxic commented 10 years ago

OK, after quite some time I finally got around to this... Sorry.

I made the costume slightly smaller and moved its centre from the tip to the actual centre. See 3ba35e602539414da0e040f3a66b318070292d3c.

If you have time, let me know what you think. For now, I'll close the issue again.