[LE] Closing bgObjectPanel easier

MichaelSinsbeck / platformer

A ninja platformer written in LÖVE

7 stars 0 forks source link

[LE] Closing bgObjectPanel easier #150

Closed michalove closed 10 years ago

michalove commented 10 years ago

I totally like the way, I can select tiles in the bgObjectPanel. But before I read the commit message, I didn't intuitively understand how to exit the panel. Right now there are two possible ways:

right click
click outside the panel I think we should add two more ways:
ok button inside the panel
pressing f again And additionally, a tool tip should be shown, explaining this.

Germanunkol commented 10 years ago

Fully agree. It's just plain lacyness that this isn't working yet :)

michalove commented 10 years ago

Okay, great.

While we are at the panel: Is it possible to get a "preview" of which tiles will be selected, when the user drags a box? Maybe it is sufficient to move the "selecting event" from the mouse-release to the update? In that case, we should disable the "shift" key for adding a selection.

Germanunkol commented 10 years ago

I've made the selection box better: It should now be pretty obvious which tiles are selected. It wasn't very predictable because if the tile size fit less than an integer number of times into the box, that was't considered, so sometimes the border would be above tiles that would then not be selection. Now, everything the box ever touches will be selected. If this is okay for you, I'll close this issue as done. I've added the "ok" button and pressing f will close the box. The tool tips will all need to be re-worked anyway... I'll make a seperate issue for that.