⏴ go to previous day
07:11:36  *** Alberth has joined #openttd.dev

07:11:36  *** ChanServ sets mode: +v Alberth

07:22:38  *** Supercheese has quit IRC

07:57:32  *** Supercheese has joined #openttd.dev

07:57:32  *** ChanServ sets mode: +v Supercheese

11:51:42  <Alberth> https://devs.openttd.org/~alberth/diffs/del_object/

13:30:37  *** Supercheese has quit IRC

13:31:08  *** Supercheese has joined #openttd.dev

13:31:08  *** ChanServ sets mode: +v Supercheese

13:58:06  *** tycoondemon has joined #openttd.dev

15:36:56  *** tycoondemon2 has joined #openttd.dev

15:41:09  *** tycoondemon has quit IRC

18:48:29  *** frosch123 has joined #openttd.dev

18:48:29  *** ChanServ sets mode: +v frosch123

18:55:02  <frosch123> 30: this causes the object gui to no longer remember the previous selection, doesn't it?

18:59:14  <frosch123> 20: can the "ignore_abort_object" be replaced by moving the ResetObjectToPlace into an "else" branch of the "_selected_object_index != -1" ?

19:02:04  <frosch123> the other toolbars just call SetObjectToPlace when selecting a different tool

19:02:13  <frosch123> they do not call ResetObjectToPlace inbetween

19:05:10  <Alberth> 30 it does, but you loose the selection already when you eg dynamite something, In that case however, the window stays open now, at the same position in the sprites. Remembering the selection is thus only useful if you selected something, close the window yourself, and re-open it later

19:05:56  <frosch123> yes, i think that is what it was meant for

19:06:17  <frosch123> if people have 50 classes of objects, they do not want to scroll everytime and browse the list again

19:06:27  <Alberth> right now, the window gets closed a lot

19:06:56  <frosch123> we also remember station classes and stuff

19:07:00  <frosch123> and current railtype

19:07:05  <Alberth> ok, fair enough

19:07:22  <frosch123> so, yeah, maybe remembering the exact object isnot needed, but it should remember the class

19:09:13  <Alberth> oh, right 20  other tools (the treetool at least) do a this->RaiseButtons(), but that fails as the buttons are fake in the window

19:09:25  <Alberth> maybe I should fix that instead

19:09:39  <frosch123> :p

19:10:19  <Alberth> hmm, still not sure, I'd need to check what select object does exactly

21:44:40  *** Alberth has left #openttd.dev

22:35:28  *** frosch123 has quit IRC

continue to next day ⏵