IRC logs for #openttd on OFTC at 2025-07-20
            
00:03:36 <DorpsGek> [OpenTTD/OpenTTD] James103 commented on issue #14467: [Crash]: Fatal Application Failure https://github.com/OpenTTD/OpenTTD/issues/14467
00:41:26 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #14467: [Crash]: Fatal Application Failure https://github.com/OpenTTD/OpenTTD/issues/14467
01:26:13 *** lobster has quit IRC (Remote host closed the connection)
01:53:18 *** dh1 has joined #openttd
02:18:20 *** Wormnest has quit IRC (Quit: Leaving)
02:54:03 *** gnu_jj_ has joined #openttd
02:57:21 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)
03:49:14 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzzโ€ฆ)
04:13:33 *** keikoz has joined #openttd
04:46:55 *** WormnestAndroid has quit IRC (Remote host closed the connection)
04:46:58 *** WormnestAndroid has joined #openttd
05:19:35 *** dh1 has joined #openttd
05:44:02 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/16396365758
06:59:10 *** Wolf01 has joined #openttd
07:59:42 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain opened pull request #159: Change: migrate OpenTTD user MinchinWeb to GitHub user minchinweb https://github.com/OpenTTD/BaNaNaS/pull/159
08:01:13 <peter1138[d]> Well.
08:01:28 <truebrain> How deep is the well?
08:01:59 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain merged pull request #159: Change: migrate OpenTTD user MinchinWeb to GitHub user minchinweb https://github.com/OpenTTD/BaNaNaS/pull/159
08:02:24 <truebrain> I still can't believe we just run a database on GitHub, and that has been working better than the MySQL versions we had before that ๐Ÿ˜›
08:02:41 <andythenorth> I can ๐Ÿ™‚
08:05:24 <truebrain> These days I would do it slightly different, and build either an SQLite or Flatbuffer via a GitHub Actions, and load that server-side. As, you won't believe it, it takes a few minutes to load in all this YAML ๐Ÿ˜›
08:05:28 <truebrain> and it is not getting faster ๐Ÿ˜„
08:26:59 *** mindlesstux has quit IRC (Quit: The Lounge - https://thelounge.chat)
08:27:48 *** mindlesstux has joined #openttd
09:12:16 <peter1138[d]> Feel free to revamp ๐Ÿ˜‰
09:12:17 *** reldred has quit IRC (Quit: User went offline on Discord a while ago)
09:13:01 <peter1138[d]> Hmm, automatic colour-push/pop is doable, except that most strings still contain a colour code, so it's active all the time.
09:49:45 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14468: Fix 03f5f7145f: Wrong colour used when string POP_COLOURs back to initial colour. https://github.com/OpenTTD/OpenTTD/pull/14468
09:51:59 <peter1138[d]> Well, I guess.
10:33:43 <pickpacket> orudge: I worry about that mail from atari. I don't like how they implied that they would have any rights over OpenTTD. A lawyer friend of mine thought it sounded like a fishing expedition. May I ask what you replied?
10:35:01 <peter1138[d]> It did not imply that, you are reading something else into it.
10:36:42 <_jgr_> pickpacket: Perhaps you'd be better off emailing him privately if you really want a response, I can't imagine that he'd want to broadcast whatever he might have replied here
10:36:42 <pickpacket> peter1138[d]: "while protecting our rights to the game"?
10:36:53 <pickpacket> _jgr_: true
10:37:04 * pickpacket sighs
10:37:22 <michi_cc> the game = transport tycoon
10:37:59 <pickpacket> I've seen too many IP fights to feel comfortable with any corporation sending something like this
10:38:49 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzzโ€ฆ)
10:39:13 *** dh1 has joined #openttd
10:39:36 <pickpacket> michi_cc: that's my first thought too, but in that case there's a misplaced modifier
10:45:14 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14384: Change: Extend industry cargo history to cover quarters and years. https://github.com/OpenTTD/OpenTTD/pull/14384
10:46:22 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14384: Change: Extend industry cargo history to cover quarters and years. https://github.com/OpenTTD/OpenTTD/pull/14384#issuecomment-3094435611
10:49:38 *** dh1 has quit IRC (Ping timeout: 480 seconds)
11:03:38 *** dres_1025 has joined #openttd
11:03:38 <dres_1025> Theoretically could a setting be made that would allow for locomotives to be placed anywhere in a train?
11:04:03 <peter1138[d]> They can be placed (almost) anywhere in a train. You just need one on the front as well.
11:04:23 <peter1138[d]> And no, it's way more involved than "a setting".
11:06:32 <andythenorth> armchair lawyering in a public channel can create confusion, best avoided
11:08:00 <andythenorth> peter1138[d]: instead of doing it properly, what if we just force a 1hp invisible engine to the front of every train, then net 1hp of the power value when reading it
11:08:06 <andythenorth> oh speed though ๐Ÿ˜
11:08:10 <andythenorth> and railtype
11:08:34 <andythenorth> why is doing it properly always easier and better? ๐Ÿ˜›
11:17:10 *** Flygon has quit IRC (Remote host closed the connection)
11:17:50 <peter1138[d]> Right, we have the cargo types key within the graph window.
11:17:57 <peter1138[d]> But the companys key is separate.
11:18:11 <peter1138[d]> There was a PR a while back that tried some complex things to split it all out and what not.
11:18:12 <peter1138[d]> But.
11:18:22 *** dh1 has joined #openttd
11:18:49 <peter1138[d]> I think I want to merge them together instead.
11:19:14 <peter1138[d]> And then have the highlight-on-hover functionality work for companies as well.
11:20:08 <peter1138[d]> "But the font is so small!" ... interface scaling, right?
11:26:26 *** dh1 has quit IRC (Ping timeout: 480 seconds)
11:28:29 *** gelignite has joined #openttd
11:36:02 *** dh1 has joined #openttd
11:52:16 *** dh1 has quit IRC (Ping timeout: 480 seconds)
12:18:01 *** dh1 has joined #openttd
12:19:28 <digitalfox> Hello, good day for releasing the mighty beta 3? ๐Ÿ™‚
12:28:16 *** dh1 has quit IRC (Ping timeout: 480 seconds)
12:29:15 <DorpsGek> [OpenTTD/OpenTTD] michalc commented on discussion #14453: Transparent animated gif of vehicles https://github.com/OpenTTD/OpenTTD/discussions/14453
12:40:44 *** dh1 has joined #openttd
12:44:40 <emperorjake> pickpacket: Where can I find more information about this?
12:48:46 *** dh1 has quit IRC (Ping timeout: 480 seconds)
12:49:09 <_glx_> https://discordapp.com/channels/142724111502802944/1008473233844097104/1394693142933934292
12:52:26 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14321: Add: Include history of cargo delivered and waiting to industry graphs. https://github.com/OpenTTD/OpenTTD/pull/14321#pullrequestreview-3036007901
12:55:53 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14465: Fix #14464: Invalid string parameter in scenario editor when unable to build industry. https://github.com/OpenTTD/OpenTTD/pull/14465#pullrequestreview-3036008871
13:03:33 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14465: Fix #14464: Invalid string parameter in scenario editor when unable to build industry. https://github.com/OpenTTD/OpenTTD/pull/14465
13:03:36 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #14464: [Bug]: Invalid Parameter when (failing to) place industries in scenario editor https://github.com/OpenTTD/OpenTTD/issues/14464
13:03:58 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14321: Add: Include history of cargo delivered and waiting to industry graphs. https://github.com/OpenTTD/OpenTTD/pull/14321
13:10:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14384: Change: Extend industry cargo history to cover quarters and years. https://github.com/OpenTTD/OpenTTD/pull/14384
13:15:09 *** dh1 has joined #openttd
13:21:24 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14468: Fix 03f5f7145f: Wrong colour used when string POP_COLOURs back to initial colour. https://github.com/OpenTTD/OpenTTD/pull/14468#pullrequestreview-3036016057
13:23:11 *** dh1 has quit IRC (Ping timeout: 480 seconds)
13:30:23 <DorpsGek> [OpenTTD/OpenTTD] LordAro merged pull request #14468: Fix 03f5f7145f: Wrong colour used when string POP_COLOURs back to initial colour. https://github.com/OpenTTD/OpenTTD/pull/14468
13:47:28 *** dh1 has joined #openttd
13:48:21 <peter1138[d]> Eh...
13:48:24 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1396488920077963346/image.png?ex=687e4528&is=687cf3a8&hm=a4a170da47ff02af100f9102513993773c36ae67176e64150df80866f7e5cd08&
13:48:24 <peter1138[d]> Kinda wide ๐Ÿ˜ฎ
13:49:38 <peter1138[d]> The "(Company X)" adds a bit.
13:53:10 <talltyler> Does the player ever need to know the company number?
14:02:07 <peter1138[d]> Probably not here, it's just what was shown in the existing legend window.
14:03:26 <peter1138[d]> > 3 files changed, 158 insertions(+), 122 deletions(-)
14:03:32 <peter1138[d]> Hmm, annoying amount ๐Ÿ˜ฎ
14:05:05 <peter1138[d]> The mini-"blot" is a bit different too, I guess.
14:05:14 <peter1138[d]> The usual sprite won't fit, of course.
14:06:07 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14460: Codechange: Use StationID as StationIDStack Titem type https://github.com/OpenTTD/OpenTTD/pull/14460
14:07:15 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14459: Fix: [Linkgraph] Incorrect NodeID to StationID conversion for EraseFlows https://github.com/OpenTTD/OpenTTD/pull/14459
14:10:58 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1396494598087512245/image.png?ex=687e4a71&is=687cf8f1&hm=a05622ab302774458f6ad7e45f05fa66ce85546b676686222b787609f448d138&
14:10:58 <peter1138[d]> Does this mean we have more companies now? :p
14:15:23 *** Tirili has quit IRC (Read error: Connection reset by peer)
14:15:26 <mnhebi> 100 company OpenTTD server when :d
14:27:29 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
14:27:42 *** Smedles has joined #openttd
14:29:52 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14006: Change: Automatically push/pop colours when formatting a sub-string. https://github.com/OpenTTD/OpenTTD/pull/14006
14:40:03 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14006: Change: Automatically push/pop colours when formatting a sub-string. https://github.com/OpenTTD/OpenTTD/pull/14006#pullrequestreview-3036062825
14:45:31 <_glx_> so regarding nuget cache for vcpkg, it seems the PRs are not able to store packages, but they should be able to retrieve them
14:46:56 <_glx_> after a lot of search there could be a way for PRs to store packages, but that requires a classic PAT (plain annoying to manage)
14:49:42 <_glx_> I think the simple way is to let post merge CI run store the cache (it has the correct permissions) and PRs will use it if available
14:51:37 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #14426: Codechange: [Actions] Use NuGet for vcpkg binary caching https://github.com/OpenTTD/OpenTTD/pull/14426
14:53:05 *** gelignite has quit IRC ()
14:55:51 *** Tirili has joined #openttd
15:01:52 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #14426: Codechange: [Actions] Use NuGet for vcpkg binary caching https://github.com/OpenTTD/OpenTTD/pull/14426#issuecomment-3094587649
15:12:54 <asasnat> I'm curious, when was the snow melt/grass growth pattern changed to not be so liney (like in the original game)?
15:35:44 *** Wormnest has joined #openttd
16:35:37 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
16:35:50 *** Smedles has joined #openttd
17:04:00 <peter1138[d]> I'm curious, what do you think has changed?
17:07:05 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #14426: Codechange: [Actions] Use NuGet for vcpkg binary caching https://github.com/OpenTTD/OpenTTD/pull/14426#pullrequestreview-3036098894
17:12:33 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14457: Codechange: Add distinct type to hold pixel drawing colour. https://github.com/OpenTTD/OpenTTD/pull/14457#pullrequestreview-3036100168
17:18:02 *** xava_ has joined #openttd
17:18:12 <xava_> hi
17:18:50 *** xava_ has quit IRC ()
17:19:06 *** xava has joined #openttd
17:19:54 <peter1138[d]> Well
17:20:24 <xava> it seems that the main thread handles a mouse event immediately, and when refresh, processes mouse events the second time.
17:20:35 <xava> what is the second time for?
17:23:50 <xava> I mean, I can see that WndProcGdi(...) function handle mouse and keyboard inputs.
17:24:19 <xava> but, in VideoDriver::Tick(), InputLoop() is called again.
17:24:25 <xava> I just don't understand
17:25:35 <xava> BTW, I'm new to OpenTTD source code.
17:26:55 <michi_cc> VideoDriver::InputLoop is for stuff not coming by OS events (e.g. on Windows stuff like keyboard modifier key state), ::InputLoop() is the generic, non-OS specific part of the handling.
17:27:59 <peter1138[d]> Overloaded naming then.
17:28:37 <LordAro> sounds like something that could be improved pretty easily :D
17:28:43 <LordAro> (the naming, that is)
17:30:36 <xava> oh. what kind of mouse inputs is handled by ::InputLoop()?
17:31:11 <xava> Is OS mouse events handling not enough?
17:33:06 <xava> wait
17:34:28 <xava> so there are three of them? WndProcGdi, VideoDriver::InputLoop, and ::InputLoop()?
17:37:06 *** wm_ has joined #openttd
17:37:18 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
17:37:30 *** Smedles has joined #openttd
17:41:31 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #14426: Codechange: [Actions] Use NuGet for vcpkg binary caching https://github.com/OpenTTD/OpenTTD/pull/14426#pullrequestreview-3036106470
17:45:57 *** Tirili has quit IRC (Quit: Leaving)
17:54:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14457: Codechange: Add distinct type to hold pixel drawing colour. https://github.com/OpenTTD/OpenTTD/pull/14457
17:54:24 <DorpsGek> [OpenTTD/OpenTTD] PeterN dismissed a review for pull request #14457: Codechange: Add distinct type to hold pixel drawing colour. https://github.com/OpenTTD/OpenTTD/pull/14457#pullrequestreview-3036100168
17:55:55 <pickpacket> peter1138[d]: are you working on extending the colour palette for company liveries?
17:57:02 *** xava has quit IRC (Quit: Leaving)
17:57:47 *** Tirili has joined #openttd
18:01:59 <_glx_> no, the palette is full ๐Ÿ™‚
18:02:56 *** xava has joined #openttd
18:07:05 <peter1138[d]> Eh, MSVC, what?
18:07:50 *** xava has quit IRC ()
18:08:09 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14457: Codechange: Add distinct type to hold pixel drawing colour. https://github.com/OpenTTD/OpenTTD/pull/14457
18:23:45 <Rubidium> looks like PC_NONE already got defined in some Windows SDK header
18:24:40 <Rubidium> maybe use PixelColour{} instead of PC_NONE?
18:24:53 <peter1138[d]> Ohhhh
18:25:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14457: Codechange: Add distinct type to hold pixel drawing colour. https://github.com/OpenTTD/OpenTTD/pull/14457
18:48:32 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14457: Codechange: Add distinct type to hold pixel drawing colour. https://github.com/OpenTTD/OpenTTD/pull/14457#pullrequestreview-3036123255
19:06:12 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #14384: Change: Extend industry cargo history to cover quarters and years. https://github.com/OpenTTD/OpenTTD/pull/14384#pullrequestreview-3036127841
20:00:39 *** tokai|noir has joined #openttd
20:00:39 *** ChanServ sets mode: +v tokai|noir
20:05:56 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14449: Codechange: Decouple glyph map from SpriteFontCache instances. https://github.com/OpenTTD/OpenTTD/pull/14449#pullrequestreview-3036152359
20:07:35 *** tokai has quit IRC (Ping timeout: 480 seconds)
20:41:56 *** Flygon has joined #openttd
20:45:01 <andythenorth> 384x384 maps?
20:45:11 <andythenorth> might be a nice size
20:54:44 *** aaaaaaaar42vg2f has joined #openttd
21:01:30 *** aaaaaaaar42vg2f has quit IRC (Quit: Page closed)
21:15:04 <ahyangyi> 384x384 maps, with the middle 128x128 inaccessible ๐Ÿ˜›
21:15:59 <andythenorth> donuts
21:18:31 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:21:49 <peter1138[d]> o
21:22:10 <peter1138[d]> Regions?
21:22:39 <peter1138[d]> We did have that 'discussion' a week or so ago.
21:23:48 <talltyler> Regions in what sense?
21:24:07 <peter1138[d]> Fog-of-war type.
21:24:29 <peter1138[d]> And 'week' was a joke ๐Ÿ™‚
21:24:42 <talltyler> I find it funny that Transport Fever 3 is pushing hard for "so much map variety" but still has TTD climates. New with this release is sub-artic.
21:24:54 <talltyler> Maybe in Transport Fever 4 we'll finally get Toyland.
21:40:32 <peter1138[d]> Hmm, can I tell clangd to pretend that stdafx.h is included...
21:41:23 *** greeter has quit IRC (Remote host closed the connection)
21:45:09 *** greeter has joined #openttd
21:46:51 <peter1138[d]> Hmm, maybe relevant but I dunno ๐Ÿ˜ฎ <https://github.com/clangd/clangd/issues/856>
21:48:32 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14384: Change: Extend industry cargo history to cover quarters and years. https://github.com/OpenTTD/OpenTTD/pull/14384
21:50:17 *** xava has joined #openttd
21:51:08 *** xava has quit IRC ()
21:53:14 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14384: Change: Extend industry cargo history to cover quarters and years. https://github.com/OpenTTD/OpenTTD/pull/14384#issuecomment-3094815078
21:56:32 <peter1138[d]> Wel.
21:56:36 <peter1138[d]> Also, well.
21:57:59 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14457: Codechange: Add distinct type to hold pixel drawing colour. https://github.com/OpenTTD/OpenTTD/pull/14457
21:58:47 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14449: Codechange: Decouple glyph map from SpriteFontCache instances. https://github.com/OpenTTD/OpenTTD/pull/14449
22:01:42 <peter1138[d]> Okay, can someone explain how a 32bpp LUT system should look, replacing the current palette recolour maps, and what it would achieve?
22:03:24 <peter1138[d]> IIRC the main issue is that, for artists, the current system 32bpp + 8bpp mask is difficult to work with, and doesn't support blending at the edges.
22:03:41 <peter1138[d]> But also: most artists are just drawing 8bpp anyway.
22:04:18 <talltyler> Ugh, managed to get house overbuilding working for >1-tile buildings, now it crashes for 1-tile buildings ๐Ÿ˜›
22:04:49 <talltyler> Probably need to clear the northern tile of multi-tile buildings...
22:04:52 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14450: Codechange: Add font cache factory providers. https://github.com/OpenTTD/OpenTTD/pull/14450
22:05:07 *** Tirili has quit IRC (Remote host closed the connection)
22:05:42 <peter1138[d]> If you clear it through the command system (but with Do, not Post) it should do the right thing?
22:07:38 *** keikoz has quit IRC (Ping timeout: 480 seconds)
22:09:23 *** xava has joined #openttd
22:09:26 <xava> !
22:09:35 <talltyler> Maybe. I might just be overcomplicating things.
22:10:10 <talltyler> I'm skipping a lot of "is this a safe place to build a house" checks that maybe I should leave alone.
22:11:57 <talltyler> Maybe I can just skip `DoCommand::Auto` if we're overbuilding...
22:12:10 <xava> Guys, I commented out the "::InputLoop()" line in VideoDriver::Tick(), and nothing happened.
22:14:03 <xava> So. What does that line do?
22:16:51 <_jgr_> There is more than one of those in VideoDriver::Tick(), which one is executed depends on whether a separate thread os used to run the game state
22:17:50 <_jgr_> I'm presuming that you commented out the first one, in which case it is not unexpected that you don't see anything different
22:18:03 <_jgr_> You can ready the definition of that function to see what it does
22:18:09 <_jgr_> read*
22:20:31 <xava> I comment out the line in VideoDriver::Tick() in video_driver.cpp.
22:20:38 <peter1138[d]> I think HandleMouseEvents() is actually the problem xava is talking about.
22:20:42 <peter1138[d]> 'problem' ๐Ÿ™‚
22:20:59 <peter1138[d]> ::InputLoop() itself actually does other stuff that won't be immediately visible.
22:21:04 <_glx_> in most case I think the call in ::Tick() will just take care of hovering
22:21:10 <peter1138[d]> But that other stuff isn't really anything to do with an InputLoop.
22:21:37 <_glx_> all other events are handled internally in driver
22:21:48 *** wm_ has quit IRC (Ping timeout: 480 seconds)
22:21:50 <peter1138[d]> ::InputLoop calls HandleMouseEvents, but the drivers also call HandleMouseEvents anyway.
22:21:54 <xava> and "::InputLoop();" line which commented out is not the dedicated server one, but the latter one in Tick();
22:22:12 <peter1138[d]> Maybe there's a case where some are not handled immediately. I haven't looked.
22:22:46 <_glx_> for me it's just hovering, when there's no click nor mouse moves
22:23:02 <peter1138[d]> Ah yes ๐Ÿ™‚
22:23:08 <_glx_> so no calls from OS driver
22:23:57 <xava> Ok I'll try hovering
22:24:24 <peter1138[d]> The other bits in InputLoop() that are actually related to window management could just be called from UpdateWindows(), I think?
22:26:52 <xava> Ohhhh, right, hovering. hovering wrongly worked after I commented that line.
22:26:58 <xava> thx guys
22:28:11 *** xava has quit IRC (Quit: Leaving)
22:33:54 *** xava has joined #openttd
22:36:01 <talltyler> Oh, fun fact, `TileArea ta = tile;` doesn't add `tile` to the TileArea. That's stupid.
22:38:21 <_glx_> actually it does, but since it has no width/height it's empty ๐Ÿ™‚
22:38:42 <talltyler> So is there a better way than to immediately do `ta.Add(tile);` ?
22:39:33 <_glx_> `ta(tile, 1, 1)`
22:40:28 <talltyler> Thanks ๐Ÿ™‚
22:42:01 *** tony_pixel has quit IRC (Quit: User went offline on Discord a while ago)
22:53:16 *** xava has quit IRC (Quit: Leaving)
23:05:17 <talltyler> https://cdn.discordapp.com/attachments/1008473233844097104/1396629066076913765/buttons.png?ex=687ec7ad&is=687d762d&hm=33d29123da88b833853f4a57b4cce2aa2d79e0e7eb50779b26cb0839a3172124&
23:05:17 <talltyler> On/off buttons with labels (four buttons total), or big toggle buttons containing the label? (two buttons total)
23:05:47 <talltyler> https://cdn.discordapp.com/attachments/1008473233844097104/1396629192971518102/buttons.png?ex=687ec7cb&is=687d764b&hm=17626a07fd96ca718a8101dcf80ff5b55c80f0c814112569e98b7b7ca9442f85&
23:05:47 <talltyler> On/Off buttons or big toggle buttons containing the title strings instead?
23:08:56 <talltyler> https://cdn.discordapp.com/attachments/1008473233844097104/1396629983396626574/toggles.png?ex=687ec888&is=687d7708&hm=7d3d0bbd13af86c0fbc1f2ff1980b159ed657681fa720ef4f98eb4d68fb8e6e0&
23:08:56 <talltyler> (toggle button example, `Medium` and `Better roads` selected)
23:09:34 <peter1138[d]> The "Off" and "On" buttons came via the "Coverage area highlight" way, I suppose.
23:09:59 <talltyler> Do you have a preference?
23:11:26 <talltyler> Looking at these two windows beside each other makes me itch, with inconsistent label colours and colon use ๐Ÿ˜ฌ
23:11:48 <peter1138[d]> Yes, the town gen window is a bit odd ๐Ÿ™‚
23:11:51 <peter1138[d]> But
23:12:12 <talltyler> For coverage area highlight, I have considered deleting the buttons and always showing the highlight ๐Ÿ™‚
23:12:21 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1396630844663271454/image.png?ex=687ec955&is=687d77d5&hm=af0713048709b9893212157ea1eb9184320ae6745ece228a518651dc4be8184c&
23:12:36 <talltyler> OK, so I'm consistant with that ๐Ÿ˜„
23:12:46 <peter1138[d]> So with the options here, this has both labels AND toggles
23:13:28 <peter1138[d]> Is "Overbuild existing" self-explanitory?
23:13:57 <talltyler> I think I'll do a toggle, and then open a PR to always show the coverage area highlight, to attract new pitchforks ๐Ÿ™‚
23:14:10 <talltyler> There is a tooltip for it if you hover the buttons
23:14:18 <talltyler> (it = overbuild existing)
23:14:18 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1396631334482612224/image.png?ex=687ec9ca&is=687d784a&hm=358ba5637b5f03593da60ee23d40057987ba8a939ef14f2362131c5c894f84b0&
23:14:18 <peter1138[d]> These are another option that might not fit.
23:14:48 <peter1138[d]> (Stylistically fit, that is)
23:15:03 <talltyler> I do like those toggles, but they are only in menus, not toolbars
23:15:12 <talltyler> (arbitrary distinction, of course)
23:15:28 <peter1138[d]> I... don't think "menus" and "toolbars" the correct words here.
23:15:50 <talltyler> I wonder if toggle buttons that show state only through their state of depression (heh) is good design or something we should avoid ๐Ÿค”
23:16:26 <talltyler> Wasn't that one of the reasons for the new sliding toggles?
23:18:04 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1396632281141088298/image.png?ex=687ecaab&is=687d792b&hm=3df0f72501e0547092db5040519e0fe07193ae5da850bc72fafebf6650972d8f&
23:18:04 <peter1138[d]> These toggles seem fine and unambiguous?
23:18:22 <talltyler> OK, let's do that, then.
23:18:30 <peter1138[d]> The thing with the toggles was that we had some that are red/green, and some that just stayed grey.
23:18:35 <peter1138[d]> So it was inconsistent.
23:18:40 <talltyler> Ah, right.
23:18:53 <peter1138[d]> We made it consistent, but as we were already touching it, investigated changing the look too ๐Ÿ™‚
23:19:13 <talltyler> Red/green is a colourblindness issue. If grey is hard to see, it's equally hard for everyone. ๐Ÿ˜›
23:19:23 <talltyler> (It's easy for me to see though)
23:19:25 <peter1138[d]> I think sticking with Off / On would be fine in this case, it matches the existing toggle.
23:19:40 <talltyler> ...but I want to change those other toggles too ๐Ÿ˜›
23:19:58 <talltyler> I just love scope creep
23:20:07 <peter1138[d]> Well.
23:23:12 <peter1138[d]> Here's a bug that someone could fix.
23:23:50 <peter1138[d]> Clicking any of the buttons in the Signal Selection window does not produce the "bip" sound.
23:43:04 <talltyler> โ€œSomeoneโ€ huh?
23:43:29 <talltyler> A bug I have not found, because I turn button sounds off ๐Ÿ™‚
23:43:50 *** xava has joined #openttd
23:50:19 *** xava has quit IRC (Quit: Leaving)