⏴ go to previous day
00:11:01 <DorpsGek> [OpenTTD/nml] glx22 opened pull request #285: Fix: [Actions] nmlc version could not be determined during regression https://github.com/OpenTTD/nml/pull/285
00:17:52 <DorpsGek> [OpenTTD/OpenTTD] aeonofdiscord opened pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673
00:37:50 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
00:58:01 *** WormnestAndroid has joined #openttd
02:44:33 *** D-HUND has joined #openttd
02:45:18 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
02:45:46 *** WormnestAndroid has joined #openttd
02:47:57 *** debdog has quit IRC (Ping timeout: 480 seconds)
02:48:00 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
02:49:10 *** WormnestAndroid has joined #openttd
03:01:10 *** D-HUND is now known as debdog
03:02:21 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
03:32:26 *** keikoz has joined #openttd
04:23:53 *** felix has quit IRC ()
04:24:14 *** felix has joined #openttd
04:28:12 *** NGC3982 has quit IRC (Ping timeout: 480 seconds)
04:41:22 *** NGC3982 has joined #openttd
04:49:54 <DorpsGek> [OpenTTD/OpenTTD] EmperorJake commented on pull request #10666: Extend callback 161 (engine name) with value 0x22 for autoreplace context - see #10399 https://github.com/OpenTTD/OpenTTD/pull/10666#issuecomment-1512429015
06:14:28 *** sla_ro|master has joined #openttd
06:41:22 <petern> While I am yoloing allowing more objects per NewGRF, I've also extended stations and roadstops per NewGRF.
06:41:38 <petern> Nobody is asking for this...
06:51:41 <DorpsGek> [OpenTTD/nml] LordAro commented on pull request #285: Fix: [Actions] nmlc version could not be determined during regression https://github.com/OpenTTD/nml/pull/285#pullrequestreview-1389405549
06:58:03 <petern> Hmm, not able to reach the class limits yet though.
07:16:31 <petern> Hmm, I might do some name tidying...
07:21:23 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #10672: Change: Remove limit of objects per NewGRF. https://github.com/OpenTTD/OpenTTD/pull/10672
07:24:00 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10672: Change: Remove limit of objects per NewGRF. https://github.com/OpenTTD/OpenTTD/pull/10672#issuecomment-1512581911
07:30:20 *** Extrems` has joined #openttd
07:30:20 *** Extrems has quit IRC (Read error: Connection reset by peer)
07:30:24 *** Extrems` is now known as Extrems
07:51:02 *** FIREDR4GON has quit IRC (Quit: User went offline on Discord a while ago)
07:55:24 <DorpsGek> [OpenTTD/OpenTTD] JGRennison commented on pull request #10672: Change: Remove limit of objects per NewGRF. https://github.com/OpenTTD/OpenTTD/pull/10672#issuecomment-1512624475
08:15:46 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10672: Change: Remove limit of objects per NewGRF. https://github.com/OpenTTD/OpenTTD/pull/10672#issuecomment-1512656496
10:17:32 <DorpsGek> [OpenTTD/grfcodec] SoapGentoo commented on pull request #16: Gentoo patches https://github.com/OpenTTD/grfcodec/pull/16#issuecomment-1512819776
10:23:26 <DorpsGek> [OpenTTD/nml] glx22 commented on pull request #285: Fix: [Actions] nmlc version could not be determined during regression https://github.com/OpenTTD/nml/pull/285#pullrequestreview-1389791464
10:34:33 <LordAro> glx[d]: you intentionally ignoring the grfcodec PR (i wouldn't blame you) ?
10:36:10 <Heiki> 3
10:36:42 <Heiki> oops
10:36:47 <LordAro> 4
10:44:15 <petern> Silly me for using silly Bootstrap: container is too narrow, and container-fluid is too wide. Woe.
10:48:19 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
10:49:00 *** WormnestAndroid has joined #openttd
10:50:09 <pickpacket> 5?
10:55:56 *** WormnestAndroid has quit IRC (Remote host closed the connection)
10:56:00 *** WormnestAndroid has joined #openttd
11:06:52 <Rubidium_> 7 :D
11:07:08 <petern> 11
11:12:18 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:13:10 *** WormnestAndroid has joined #openttd
11:14:19 <glx[d]> LordAro: It's not really intentional, I don't like looking at grfcodec source, and some of the changes seem to be dirty quick fixes
11:15:21 <glx[d]> (and of course the recurrent free pings 😉 )
12:20:11 *** Smedles_ has joined #openttd
12:22:33 *** Smedles has quit IRC (Ping timeout: 480 seconds)
12:29:22 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097861428339298316/image.png
12:29:22 <petern> My VS Code is going mad...
12:32:45 <pickpacket> petern: nice colour highlighting, but hard to figure out what is what
12:34:54 <andythenorth> petern: Change the grid var and recompile 😛
12:39:19 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10671: Change title screen GUI: Move buttons to bottom edge of screen https://github.com/OpenTTD/OpenTTD/pull/10671#pullrequestreview-1389956715
12:46:41 <TrueBrain> is TallTyler now saying he never spotted that the main screen shows the OpenTTD version? 😄
12:46:53 <TrueBrain> main screen .. intro window .. give it a name 😛
12:51:15 <TallTyler> Uh, apparently I am. One of those things you see all the time but never really notice, I guess!
12:51:22 <TrueBrain> 😄 😄
12:51:38 <TrueBrain> invaluable during development; avoids thinking: what version am I running here? 😛
12:51:59 <glx[d]> you can even hold mouse button on it and move the window 😉
12:52:23 <TallTyler> I still occasionally find new features (usually hidden Ctrl ones) by proofreading new/changed tooltips 🙂
12:52:43 <TallTyler> Well, new to me 😛
12:54:01 <TrueBrain> so now you know! 😛
12:54:16 <pickpacket> gf's OpenTTD crashes every now and then. Version 13.0 on Windows, through Steam. Crashes with a message about assertion error
12:57:38 <glx[d]> scroll bar ?
12:58:28 <glx[d]> there are some known crashes in 13.0, fixed in 13.1
13:02:23 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673#pullrequestreview-1390064458
13:02:39 <pickpacket> glx[d]: is 13.1 out?
13:02:44 <TrueBrain> Discord desktop client made a change, causing the markdown link text to no longer work .. so annoying
13:02:56 <TrueBrain> I wish they would stop making bugs; would make my life better 😄 😄
13:03:20 <TallTyler> pickpacket: Yes, 13.1 was released last week. 🙂
13:04:10 <TrueBrain> if you use Steam's "default" branch, it will auto-update!
13:09:06 <Rubidium_> 13.1 isn't out when you're still on IRC ;D
13:09:47 <TrueBrain> and who is to blame for that? 😛
13:11:18 <Rubidium_> good question; I'd say the person that forgot "update IRC channel topic" in the release workflow
13:11:35 <TrueBrain> nah; it is LordAro
13:11:37 <TrueBrain> it always is
13:11:39 <TrueBrain> can't be different
13:14:05 <LordAro> not me, i don't have permission :p
13:14:17 <LordAro> @whoami
13:14:17 <DorpsGek> LordAro: I don't recognize you.
13:14:20 <TrueBrain> yes, because you STILL haven't fixed that 😛
13:14:25 <LordAro> i know!
13:14:28 <TrueBrain> so many years we have been trying ... SO MANY
13:14:29 <LordAro> isn't that fun?
13:14:33 <TrueBrain> kinda 🙂
13:15:15 *** glx has joined #openttd
13:15:15 *** ChanServ sets mode: +v glx
13:15:37 <glx> @whoami
13:15:37 <DorpsGek> glx: glx
13:15:48 <glx> @topic set 1 13.1
13:15:48 *** DorpsGek changes topic to "13.1 | Website: *.openttd.org (source: github, translator: translator, server list: servers, wiki: wiki) | Don't ask to ask, just ask | 'Latest' is not a valid version, 'Most recent' neither | English only"
13:16:01 *** glx has quit IRC ()
13:16:06 <glx[d]> done
13:17:48 <glx[d]> stupid vcpkg, I want 2 boost packages for grfcodec, it installs everything
13:18:53 <Rubidium_> how much effort would it be to nml openttd's GRFs?
13:19:04 <glx[d]> some
13:21:13 <Rubidium_> might it be that if openttd doesn't require grfcodec, all those grfcodec issues are "magically" not important anymore, because the Linux distributions do not need to include it anymore?
13:24:44 <glx[d]> openttd doesn't require grfcodec
13:25:07 <glx[d]> it's only needed if you want to touch the included grfs
13:27:16 <Rubidium_> yet at least Debian and Gentoo force recompile the grfs upon building
13:27:16 <glx[d]> oh but gentoo has it as build dependency
13:29:13 <glx[d]> I can try to look at converting to nml
13:29:16 <Rubidium_> https://salsa.debian.org/openttd-team/openttd/-/blob/master/debian/rules#L68 (line 32 makes CMake require it too)
13:29:46 <Rubidium_> although that would probably also require cmake detecting NML
13:31:19 <glx[d]> well that's doable
13:47:33 *** sla_ro|master has quit IRC ()
13:52:34 <dP> Rubidium_: would probably be quite simple to grf-py decompile them ;)
13:52:44 <dP> as they're just resources iirc
13:56:06 <petern> That would clear up all the "let's split images into separate files so we get less merge conflicts" ... "nah can't be bothered" messiness...
14:07:06 <TallTyler> There’s also the question of what zephyris is doing with OpenGFX2,might want to coordinate that with any effort to redo the main OpenGFX 🙂
14:13:51 <Eddi|zuHause> nobody talked about OpenGFX
14:23:49 *** gelignite has joined #openttd
14:28:58 *** Flygon has joined #openttd
14:36:41 <glx[d]> ok detecting nmlc is as simple as detecting grfcodec
14:39:25 <Rubidium_> good point... opengfx depends on grfid which is part of the grfcodec repositor, so it's not only OpenTTD that needs to change for grfcodec to go away
14:54:41 <LordAro> feels like it could be replicated in a small number of lines of python
14:54:47 <TallTyler> Oh right, I forgot about the actual GRFs that get shipped with OpenTTD
15:14:47 *** sla_ro|master has joined #openttd
15:19:33 <pickpacket> Can I play my 13.0 save games with 13.1?
15:19:45 <Brickblock1> yes
15:20:24 <pickpacket> 😃
15:20:47 <LordAro> you can play your TT saves with 13.1, if you so choose :p
15:23:01 *** nielsm has joined #openttd
15:29:17 <jfs-> being able to load and continue old savegames is a very high priority
15:30:50 <andythenorth> * "continue"
15:31:07 <andythenorth> "possibly with changed gameplay, see warranty for details"
15:38:44 *** HerzogDeXtEr has joined #openttd
15:44:28 *** Smedles_ has quit IRC (Read error: Connection reset by peer)
15:45:24 *** Smedles has joined #openttd
15:49:41 <petern> Unchanged gameplay if it's not too odious.
16:06:33 <jfs-> https://cdn.discordapp.com/attachments/1008473233844097104/1097916083249369158/image.png
16:06:33 <jfs-> https://cdn.discordapp.com/attachments/1008473233844097104/1097916083702345898/image.png
16:06:33 <jfs-> TallTyler: tried repositioning the windows like this
16:07:21 <jfs-> https://cdn.discordapp.com/attachments/1008473233844097104/1097916289177112606/image.png
16:07:21 <jfs-> 640x480
16:12:54 <andythenorth> we will have the challenge of equal visual weighting to solve
16:13:03 <andythenorth> and relative prominence of elements
16:13:25 <TallTyler> I'd put the primary element on top, personally
16:14:18 <jfs-> putting the central one on top is more difficult/annoying
16:14:22 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1097918054366716016/10970261-railroad-tycoon-ii-windows-main-screen-where-you-can-load-scenar.png
16:14:25 <frosch> i would have used these columns: [new game, play scenario, play heightmap, new game]; [load game, multiplayer, exit]; [game options, settings, online content, highscore]. basically "create game" on the left, most-often-used in the middle, and "details" on right. i dropped the newgrf/ai/gs settings which are already in new-game. the "exit" button could also be a fourth thing in the top-right. and the version could be a sub-title to the lo
16:14:42 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1097918135950119012/menu_16x9.png
16:14:46 <andythenorth> I miss the days of excess bevels
16:14:58 <andythenorth> used to be such great Photoshop plugins to generate them
16:15:31 <LordAro> RRTII <3
16:15:37 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1097918367492472925/image.png
16:15:37 <andythenorth> currently I see a lot of this
16:15:50 <andythenorth> settings on the left, big 'play' button
16:16:02 <andythenorth> and many red dots to get you to send your money to Belarus 😛
16:18:09 <DorpsGek> [OpenTTD/OpenTTD] nielsmh updated pull request #10671: Change title screen GUI: Move buttons to bottom edge of screen https://github.com/OpenTTD/OpenTTD/pull/10671
16:18:41 <andythenorth> jfs-: can you live stream programming on Twitch and we will all go in voice chat and make "helpful" suggestions?
16:18:50 <andythenorth> like doing commercial web design with a client there 😛
16:19:16 <jfs-> :no:
16:26:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10671: Change title screen GUI: Move buttons to bottom edge of screen https://github.com/OpenTTD/OpenTTD/pull/10671#issuecomment-1513464271
16:35:47 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on pull request #10671: Change title screen GUI: Move buttons to bottom edge of screen https://github.com/OpenTTD/OpenTTD/pull/10671#issuecomment-1513476355
16:39:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10671: Change title screen GUI: Move buttons to bottom edge of screen https://github.com/OpenTTD/OpenTTD/pull/10671#issuecomment-1513481368
16:40:48 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on pull request #9984: Add various user folders to the file browser windows https://github.com/OpenTTD/OpenTTD/pull/9984#issuecomment-1513482720
16:50:04 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on pull request #10671: Change title screen GUI: Move buttons to bottom edge of screen https://github.com/OpenTTD/OpenTTD/pull/10671#issuecomment-1513493640
16:53:58 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10671: Change title screen GUI: Move buttons to bottom edge of screen https://github.com/OpenTTD/OpenTTD/pull/10671#issuecomment-1513498192
17:02:14 <jfs-> should I try to integrate "play heightmap" and "play scenario" into the New Game (Generate World) window and remove them from the main menu?
17:10:41 <TallTyler> I would absolutely support that
17:10:56 <TallTyler> Either in your redesign or in the existing menu
17:11:18 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674
17:13:15 <petern> The file selector is pretty weird though, it already shows you things that are in tar files in separate directories. And the "heightmap" directory exists underneath "scenarios" but is empty in scenario mode 🙂
17:13:27 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513525127
17:14:54 <petern> In typical fashion, I'd done this one 4 months ago 😉
17:15:18 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513529674
17:17:56 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513534851
17:19:31 <andythenorth> `name 'elf' is not defined. Did you mean: 'self'?`
17:19:31 <andythenorth> ^ yes
17:23:09 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097935364225781870/image.png
17:23:09 <petern> Have you heard... of will self on a will self?
17:25:08 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513544171
17:25:30 <andythenorth> elf yourself
17:25:42 <andythenorth> a classic flash christmas game that was
17:31:22 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513554149
17:34:43 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513557813
17:47:28 *** nebulabc has quit IRC (Ping timeout: 480 seconds)
17:53:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675
17:55:19 <Rubidium_> petern: any progress on 10585? Would it make sense to just remove the change for rails from that PR?
17:58:29 <DorpsGek> [OpenTTD/OpenTTD] andythenorth commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513583708
17:58:45 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675#pullrequestreview-1390665412
17:59:50 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513585143
18:02:18 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675#pullrequestreview-1390670627
18:02:53 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
18:03:01 <DorpsGek> [OpenTTD/OpenTTD] aeonofdiscord commented on pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673#issuecomment-1513588745
18:04:51 *** WormnestAndroid has joined #openttd
18:32:13 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513624953
18:38:19 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673#issuecomment-1513631558
18:38:46 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675
18:39:13 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675
18:53:09 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675
19:00:44 *** Wolf01 has joined #openttd
19:01:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10676: Codechange: use string/fmt instead of print for strgen warnings/errors/fatals https://github.com/OpenTTD/OpenTTD/pull/10676
19:05:43 *** gelignite has quit IRC (Quit: Stay safe!)
19:19:39 <pickpacket> Can I make a station accept goods somehow?
19:20:24 <jfs-> find some town houses that accept goods nearby and extend the station's catchment area to cover those houses
19:20:50 <jfs-> if the town is too small maybe it doesn't have any buildings accepting goods
19:21:53 <pickpacket> in this case it doesn't have any :( It did, but no more
19:22:38 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
19:23:06 *** WormnestAndroid has joined #openttd
19:23:09 <DorpsGek> [OpenTTD/OpenTTD] aeonofdiscord updated pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673
19:31:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10677: Codechange: use string/fmt instead of printf for grfmsg https://github.com/OpenTTD/OpenTTD/pull/10677
19:32:07 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513691581
19:38:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674
19:48:17 <glx[d]> rubidium_: I have another vseprintf removal on https://github.com/OpenTTD/OpenTTD/compare/master...glx22:OpenTTD:str_fmt
19:51:12 <Rubidium_> did you work on the media stuff not using as much char* anymore?
19:51:48 <petern> I've got a patch that removes the buffer stuff for outputting lists of drivers/blitters/etc/etc
19:54:02 <petern> <https://github.com/OpenTTD/OpenTTD/compare/master...PeterN:OpenTTD:std-string-stuff>
19:59:30 <glx[d]> I just did a quick look at char* usage in media, doesn't seem to be a rabbit hole
19:59:49 <Rubidium_> well...
19:59:54 <glx[d]> but not touched anything
20:01:31 <petern> I did the simple side 😉
20:02:27 <petern> Some of it is wrong though.
20:06:34 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1097976489661694042/image.png
20:06:34 <andythenorth> well
20:06:40 <andythenorth> I made that work 😛
20:06:53 <andythenorth> probably should include the (size) suffix though
20:06:55 <petern> But at what cost?
20:07:00 <andythenorth> "the cost of time"
20:07:09 <andythenorth> due to having a massive complicated compile
20:08:07 <petern> Hmm, I guess 'interface scale' should probably go under Graphics, but... then that tab is long, and the general tab is mostly empty.
20:08:28 <TallTyler> It should definitely be under graphics
20:09:13 <TallTyler> Is there anything that should move from Settings to Options or vice versa?
20:09:23 <petern> It is, at least, super easy to move from one tab to another.
20:09:33 <jfs-> maybe the video settings can be compacted to half width (line breaking on the driver name)
20:09:50 <petern> Width is not the problem, height is.
20:09:55 <jfs-> yes
20:10:07 <jfs-> so you can place the interface scaling next to the video settings
20:10:16 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097977420390334534/image.png
20:10:16 <petern> That is now... empty
20:10:50 <glx[d]> oh there's space to put more units here
20:10:53 <jfs-> move those "localisation" settings in there to resolve https://www.tt-forums.net/viewtopic.php?t=90717
20:11:05 <petern> Heh
20:11:32 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097977739551707186/image.png
20:11:32 <petern> May still need re-ordering
20:11:40 <petern> I guess it's still shorter than it is currently
20:11:55 <jfs-> except apparently those are a world or company setting or something?
20:12:21 <petern> Maybe move base set to the bottom so it's more similar to the old layout, and matches the music layout better. Yeah.
20:12:25 <andythenorth> oof where do I get the length suffix from? 😛
20:12:29 <andythenorth> oh I have it in scope
20:12:31 <andythenorth> silly me
20:13:07 <petern> Units are not per-company.
20:13:19 <glx[d]> they are per client
20:13:36 <glx[d]> like language and currency
20:13:54 <jfs-> then they're perfect for the Options window
20:13:59 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097978355623677992/image.png
20:13:59 <petern> Anyway, Game Options is where tehy belong.
20:14:36 <glx[d]> driving side is per game
20:14:42 <JGR> Units are sort of in the save but not really, depending on context
20:14:58 <petern> I'm not adding all those, just showing that it was originally in options until we removed it.
20:15:19 <petern> "Localization Units" 😉
20:15:48 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097978810479165621/image.png
20:15:48 <petern> Okay, that does actually fit better
20:17:09 <petern> Also our options dropdown buttons are raised instead of sunken.
20:17:58 <petern> IIRC game options is the only place that has dropdowns in TTD, but we turned buttoned into dropdowns elsewhere...
20:19:42 <petern> <https://user-images.githubusercontent.com/639850/232852150-bc5c262d-3a94-4a1e-8de0-89dfae82abef.png> still less big than currently.
20:20:48 <jfs-> can it fit 640x480? 800x600?
20:21:12 <glx[d]> current version is almost too high for 480 IIRC
20:21:14 <petern> At 1x, yes.
20:21:19 <jfs-> (can we drop 640x480 as a resolution option?)
20:21:36 <petern> No, it fits fine.
20:21:53 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097980343270772766/image.png
20:21:53 <petern> 😄
20:21:57 <jfs-> the main game toolbar doesn't fit in 640
20:23:03 *** Venemo has quit IRC (Remote host closed the connection)
20:23:17 <petern> It's slightly squashed and the spacers are gone, but it's not quite "doesn't fit"
20:23:42 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097980800990974154/image.png
20:23:57 <petern> Don't open the build station or build object windows though!
20:24:47 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1097981072421175449/image.png
20:24:47 <petern> That needs some limits and some redesign.
20:25:04 <petern> jgrpp probably already fixes it.
20:25:45 *** Venemo has joined #openttd
20:26:13 <JGR> No, I'm happy for that window to be someone else's problem
20:26:56 <petern> 🙂
20:26:59 <petern> I have some ideas
20:27:02 <TallTyler> And the filter needs to search the object name, not just the category
20:27:06 <petern> Just not got around to it.
20:27:07 <petern> Yes, that to.
20:27:40 <petern> Mainly... do we actually need a preview, and then another list with more previews?
20:28:34 <petern> Maybe I will mock something up later 🙂
20:29:24 <petern> One thing this split-up Options window might allow is space for a font chooser.
20:29:28 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10678: Codechange: use std::string for base media filename/warning storage https://github.com/OpenTTD/OpenTTD/pull/10678
20:29:35 <petern> Probably as a separate tab, as there are 4 different fonts to set.
20:30:16 <TallTyler> A font chooser would be really nice
20:30:26 <TallTyler> And scratch an itch for a lot of people, I think
20:30:26 <petern> Phew, we didn't touch the same bits of those files.
20:30:49 *** nielsm has quit IRC (Ping timeout: 480 seconds)
20:30:57 <petern> When we start needing a scrollbar for the options tabs, then we need to start worrying.
20:31:19 <TallTyler> When you need a scrollbar, it's time for more tabs!
20:31:22 <petern> Although actually that is annoyingly common for modern games, just to show three and use left-right paddles to shift it.
20:31:29 <TallTyler> Until you need a scrollbar for your tabs 😛
20:31:36 <petern> Yeah, that's the one I meant.
20:31:57 <petern> Okay.. so move "Localization" too...?
20:32:36 <petern> That's slightly annoying as you need to reimplement things :/
20:33:07 <TallTyler> Didn't some Windows folder settings windows have two rows of tabs? It was quite annoying
20:36:04 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1097983910765404320/image.png
20:36:04 <andythenorth> here we go, suffixes
20:37:13 <glx[d]> looks nice
20:37:59 <andythenorth> think it's a good feature
20:38:05 <andythenorth> well a BAD FEATURE obviously
20:38:07 <andythenorth> but a good one
20:39:39 <petern> Bah, have to duplicate all the strings 😦
20:39:49 <petern> Well, the labels.
20:40:01 <glx[d]> because colour ?
20:40:51 <petern> No, in the settings window the value is included as a parameter.
20:41:57 <petern> The button is drawn before it, and it isn't a real widget.
20:42:11 <glx[d]> but if you move them from settings to options you can just change the strings to fit
20:42:32 <petern> Yes, that's basically the bit I'm "bah"ing at 🙂
20:45:32 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1097986292752584744/Screen_Recording_2023-04-18_at_21.45.08.mov
20:45:32 <andythenorth> oh 😄
20:45:39 <petern> Colour is fine these days btw, I switched the default "no colour" from blue to black quite a long time ago. I keep putting {BLACK} out of habit...
20:45:40 <andythenorth> maybe we found this before, I forget 🙂
20:46:51 <JGR> That behaviour is explicitly designed in
20:47:09 <petern> What am I looking at?
20:47:30 <petern> Oh it's a video.
20:47:54 <petern> Yes, it's by design.
20:47:57 <andythenorth> ok
20:48:06 <andythenorth> I enjoyed
21:04:58 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10674: Change: Split Game options into General, Base sets, Video tabs. https://github.com/OpenTTD/OpenTTD/pull/10674#issuecomment-1513793493
21:05:38 <LordAro> yellow definitely better than orange
21:07:45 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675#pullrequestreview-1390938998
21:08:39 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #10676: Codechange: use string/fmt instead of print for strgen warnings/errors/fatals https://github.com/OpenTTD/OpenTTD/pull/10676#pullrequestreview-1390940411
21:09:39 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10677: Codechange: use string/fmt instead of printf for grfmsg https://github.com/OpenTTD/OpenTTD/pull/10677#pullrequestreview-1390941584
21:10:28 *** keikoz has quit IRC (Ping timeout: 480 seconds)
21:13:19 <DorpsGek> [OpenTTD/OpenTTD] erenes commented on discussion #10648: How to log dedicated server on windows. https://github.com/OpenTTD/OpenTTD/discussions/10648
21:15:10 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10678: Codechange: use std::string for base media filename/warning storage https://github.com/OpenTTD/OpenTTD/pull/10678#pullrequestreview-1390944378
21:15:44 <petern> Pretty sure I'm doing something wrong here.
21:16:09 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #10677: Codechange: use string/fmt instead of printf for grfmsg https://github.com/OpenTTD/OpenTTD/pull/10677#pullrequestreview-1390948670
21:16:33 <petern> Adding 6 dropdowns to the game options window is certainly more tedious than the settings window...
21:20:58 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10676: Codechange: use string/fmt instead of print for strgen warnings/errors/fatals https://github.com/OpenTTD/OpenTTD/pull/10676
21:21:13 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10677: Codechange: use string/fmt instead of printf for grfmsg https://github.com/OpenTTD/OpenTTD/pull/10677
21:28:50 <DorpsGek> [OpenTTD/OpenTTD] Ufiby commented on issue #10574: [Bug]: Map generation fails when no town buildable roads are available https://github.com/OpenTTD/OpenTTD/issues/10574
21:28:55 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10667: Codechange: Use more std::string in fios land https://github.com/OpenTTD/OpenTTD/pull/10667#pullrequestreview-1390956877
21:29:50 <petern> Hmm, why is NC_EQUALSIZE not working...
21:40:22 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675#pullrequestreview-1390973933
22:00:51 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10678: Codechange: use std::string for base media filename/warning storage https://github.com/OpenTTD/OpenTTD/pull/10678#pullrequestreview-1390992596
22:25:08 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
22:26:19 *** WormnestAndroid has joined #openttd
22:29:05 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:29:06 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
22:29:19 *** WormnestAndroid has joined #openttd
22:30:54 <DorpsGek> [OpenTTD/OpenTTD] aeonofdiscord updated pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673
22:33:27 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on discussion #10648: How to log dedicated server on windows. https://github.com/OpenTTD/OpenTTD/discussions/10648
22:37:20 <DorpsGek> [OpenTTD/OpenTTD] aeonofdiscord updated pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673
22:37:41 <DorpsGek> [OpenTTD/OpenTTD] aeonofdiscord updated pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673
22:38:43 <DorpsGek> [OpenTTD/OpenTTD] aeonofdiscord updated pull request #10673: Fix #10665: CheckEngines should ignore wagons https://github.com/OpenTTD/OpenTTD/pull/10673
22:40:43 <petern> Weird, there are DorpsGek IRC hithub messages that aren't here.
22:40:51 <petern> And github...
22:41:19 <glx[d]> sometimes messages are delayed
22:43:48 <petern> It's #10673, there's been other messages. Hmm.
23:24:43 <TrueBrain> petern: Meh, that is annoying .. means the Discord webhook ate them .. possibly a rate limit?
23:25:16 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1098026494506311830/image.png
23:25:24 <TrueBrain> No, it is 673 ..
23:25:30 <TrueBrain> Maybe the word ignore?
23:25:44 <petern> Seems quite far apart.
23:25:48 <petern> Interesting.
23:26:24 <petern> No there were some earlier... from other people.
23:26:45 <petern> Or at least, yesterday.
23:26:59 <TrueBrain> And they did came through yesterday
23:27:25 <TrueBrain> That username maybe?
23:27:27 <petern> Oh yesterday is only a minbutes ago 🙂
23:27:34 <petern> Well, I was wondering about that too...
23:28:15 <TrueBrain> Seems so .. only comments from others are logged
23:28:59 <TrueBrain> Guess you can't name a bot anything with Discord in it
23:29:11 <TrueBrain> Makes a tiny bit sense, but weird
23:29:14 <petern> That... it interesting.
23:29:44 <TrueBrain> Owh well
23:29:51 <TrueBrain> Time for some zzz
23:30:13 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
23:41:15 <petern> Oh probably
23:45:41 *** WormnestAndroid has quit IRC (Remote host closed the connection)
23:45:53 *** WormnestAndroid has joined #openttd
23:56:53 <petern> Still need this AI to determine reasonable widths...
23:59:06 *** sla_ro|master has quit IRC ()
continue to next day ⏵
