Page MenuHomeWildfire Games

drummyfish (Miloslav Číž)
User

Projects

User Details

User Since
Jun 25 2018, 1:23 PM (24 w, 2 d)

Recent Activity

Jun 29 2018

drummyfish updated the diff for D1588: [Atlas] Allow returning to current selection to place objects .

Added a new tool icon.

Jun 29 2018, 9:27 PM
drummyfish updated the summary of D1588: [Atlas] Allow returning to current selection to place objects .
Jun 29 2018, 9:24 PM
drummyfish added a comment to D1588: [Atlas] Allow returning to current selection to place objects .

It'd be good to get first item from the list.

Jun 29 2018, 8:41 PM
drummyfish added a comment to D1588: [Atlas] Allow returning to current selection to place objects .

Could I ask for feedback again?

Jun 29 2018, 7:09 PM

Jun 27 2018

drummyfish updated the diff for D1588: [Atlas] Allow returning to current selection to place objects .

Added default object for the Placement tool.

Jun 27 2018, 2:11 PM

Jun 26 2018

drummyfish added a comment to D1588: [Atlas] Allow returning to current selection to place objects .

I think default is good, because as I mentioned above, we have the default logic for the Paint Terrain tool.

Jun 26 2018, 9:24 PM
drummyfish added a comment to D1588: [Atlas] Allow returning to current selection to place objects .

Try it out please. A new icon for the tool would have to be made, for now I gave it the same one as the Move tool. ATM if you start the editor and select the Place tool without selecting any object first, a warning is written out in the window - should I leave it like this, or try to grey the button out? Or maybe automatically select the first item in the list? Not sure.

Jun 26 2018, 7:24 PM
drummyfish updated the diff for D1588: [Atlas] Allow returning to current selection to place objects .

Added "Place" tool to the toolbar.

Jun 26 2018, 7:19 PM
drummyfish added a comment to D1588: [Atlas] Allow returning to current selection to place objects .

Yeah, if I changed the current double click solution to a single click, that would be an acceptable minimal solution to the ticket. But adding the new Placement tool/mode is better IMHO as it solves a deeper underlying problem. Adding unnecessary UI is definitely bad, but here it makes sense, is consistent with the rest of the GUI and makes it more predictable, because the rule seems to be:

Jun 26 2018, 6:28 PM
drummyfish added a comment to D1588: [Atlas] Allow returning to current selection to place objects .

The problem, that we don't have a visible tool for the object placement. So I can suggest to add this tool. It can be together with the doubleclick.

Jun 26 2018, 10:14 AM
drummyfish added a comment to D1588: [Atlas] Allow returning to current selection to place objects .
In D1588#63744, @elexis wrote:

I'm not good with wxWidgets either, doesn't EVT_COMMAND_LEFT_CLICK do what we want? http://docs.wxwidgets.org/3.1/classwx_command_event.html (I don't recall if we still support wx 3.0, not unlikely do)
(Notice that we'd still need the EVT_LISTBOX event for keyboard induced selectionchanges as far as I see)

Jun 26 2018, 10:04 AM

Jun 25 2018

drummyfish added a comment to D1588: [Atlas] Allow returning to current selection to place objects .

Isn't atlas supposed to activate the placer tool again once the objects tab is activated again (i.e. independent of the means to activate the panel)?

Jun 25 2018, 7:41 PM
drummyfish created D1588: [Atlas] Allow returning to current selection to place objects .
Jun 25 2018, 1:51 PM