IRC Logs | #openttd | 2025-08-15

00:20:34 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#issuecomment-3190266834

01:41:38 <DorpsGek> [OpenTTD/OpenTTD] killerdevildog commented on issue #14485: [Bug]: Truck can't go via a bus stop https://github.com/OpenTTD/OpenTTD/issues/14485

01:47:43 <DorpsGek> [OpenTTD/OpenTTD] killerdevildog opened pull request #14518: Fix issue #14485: Allow road vehicles to use cross-type stops as wayp… https://github.com/OpenTTD/OpenTTD/pull/14518

01:49:11 <DorpsGek> [OpenTTD/OpenTTD] killerdevildog commented on pull request #14518: Fix issue #14485: Allow road vehicles to use cross-type stops as wayp… https://github.com/OpenTTD/OpenTTD/pull/14518#issuecomment-3190382127

02:07:35 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#issuecomment-3190402332

04:43:29 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)

05:08:59 <zsombi55> coobies: I believe it is, I canno seem to learn/remember the keys, so for me a button would be helpful.

05:58:47 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)

07:39:33 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14518: Fix issue #14485: Allow road vehicles to use cross-type stops as wayp… https://github.com/OpenTTD/OpenTTD/pull/14518#pullrequestreview-3123159682

07:41:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14518: Fix issue #14485: Allow road vehicles to use cross-type stops as wayp… https://github.com/OpenTTD/OpenTTD/pull/14518#issuecomment-3190858324

07:43:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#issuecomment-3190861903

08:22:29 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed pull request #10938: Feature: Option to disable activate-on-release behaviour of toolbar dropdown buttons. https://github.com/OpenTTD/OpenTTD/pull/10938

08:23:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN reopened pull request #10938: Feature: Option to disable activate-on-release behaviour of toolbar dropdown buttons. https://github.com/OpenTTD/OpenTTD/pull/10938

08:23:49 <peter1138> Meant to delete from a one remotely only, not the upstream :)

08:30:46 *** mindlesstux has quit IRC (Quit: The Lounge - https://thelounge.chat)

10:12:03 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)

11:21:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #10938: Feature: Option to disable activate-on-release behaviour of toolbar dropdown buttons. https://github.com/OpenTTD/OpenTTD/pull/10938

11:51:24 <peter1138> Try to split things off but they require bits from the big project, so those also need to be split off, and then... more...

11:52:34 <peter1138> Yeah but interfaces still need models, so do I consider the model as part of the interface, or another separate project... Hmm.

12:17:36 *** reldred has quit IRC (Quit: User went offline on Discord a while ago)

12:42:01 *** jinks has quit IRC (Remote host closed the connection)

13:16:03 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14518: Fix issue #14485: Allow road vehicles to use cross-type stops as wayp… https://github.com/OpenTTD/OpenTTD/pull/14518#pullrequestreview-3123817317

13:18:43 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#issuecomment-3191482178

13:32:00 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#issuecomment-3191510377

13:50:57 <jfkuayue> I had lamb stew, cheaper than British sandwiches.

14:11:35 <peter1138> Sandwiches are cheap, unless for reason you're hooked on supermarket meal deals and lack the intelligence to make your own.

14:15:16 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#issuecomment-3191606937

14:17:06 <talltyler> emperorjake: , you might be interested in looking at this PR about bridges: https://github.com/OpenTTD/OpenTTD/pull/14517

14:17:06 <talltyler> It looks like a huge improvement to me, just want to hear from someone who's made bridges in case we're missing something. 🙂

14:26:55 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10938: Feature: Option to disable activate-on-release behaviour of toolbar dropdown buttons. https://github.com/OpenTTD/OpenTTD/pull/10938#pullrequestreview-3123980027

14:29:10 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10938: Feature: Option to disable activate-on-release behaviour of toolbar dropdown buttons. https://github.com/OpenTTD/OpenTTD/pull/10938#pullrequestreview-3123996993

14:30:49 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10938: Feature: Option to disable activate-on-release behaviour of toolbar dropdown buttons. https://github.com/OpenTTD/OpenTTD/pull/10938#pullrequestreview-3124000761

14:34:21 <emperorjake> talltyler: Interesting, I'll take a closer look soon

14:35:03 <emperorjake> basically this adds a flag to bridge pillars that makes them disappear when they'd collide witha road or track?

14:36:08 <talltyler> Read the discussion in the comments, I needed clarification at first too and it helped 🙂

14:41:57 <peter1138> For reference, this is part of what was in the bridges-over-stations PR, where other reviewers requested this behaviour vs preventing construction. So read through #14477 as well.

14:42:46 <emperorjake> Yeah I've already been keeping my eye on that one

14:43:31 <emperorjake> I like the idea, especially if authors can control if pillars can disappear or not

14:43:53 <peter1138> Okay, Well this PR is simply the bridge flags part of that extracted out into a separate PR, as it can be applied to things other than station.s

14:46:45 <peter1138> The intention is not for authors to control if pillars should appear or not. The intention is for authors to tell the game if pillars exist in a particular position on a tile, and the game will then decide what to draw to avoid conflicts.

14:48:03 <emperorjake> But some bridge types should always draw pillars or they'll look weird

14:49:20 <peter1138> Okay, so should we prevent construction to avoid conflicts?

14:50:14 <peter1138> I don't really like that approach, it makes placing bridges an awkward experience.

14:50:30 <emperorjake> That would potentially be very confusing to players when bridges can't be built sometimes

14:52:29 <peter1138> Right. So my answer to "the bridge looks weird when I build parallel track underneath it" is "don't do that then".

14:54:09 <emperorjake> "looks weird" is either the current behaviour, or Locomotion-style floating bridges

14:54:51 <emperorjake> In that case I'd prefer to keep the current behaviour

14:55:54 <talltyler> I don't see why authors couldn't set pillars as "always draw this, even if it looks silly" or "draw this if there's space, otherwise it's fine to skip it"

14:56:10 <talltyler> (via something more official than the idea I accidentally gave people 🙂 )

14:57:36 <peter1138> They can absolutely set the pillar flags to pretend that there are no pillars if they really want to.

14:58:31 <talltyler> I think that's the best way to handle pillars that are needed for the bridge to not glitch out (suspension bridges, tall arch bridges, etc)

14:58:44 <peter1138> That does mean the flags would need to be set differently between JGRPP and vanilla, which means that nobody is going to bother setting them for vanilla.

14:58:45 <talltyler> Preventing construction entirely is definitely a bad idea 🙂

14:59:48 <emperorjake> why would they need to be different for JGRPP?

15:00:46 <talltyler> Oh, I misread the part about JGRPP compatibility.

15:01:40 <talltyler> Could the flags be for optional pillars, and the required pillars get no flags?

15:04:53 <peter1138> No. Pillars are never required. Forget about that idea.

15:05:52 *** k-man has quit IRC (Remote host closed the connection)

15:06:59 <talltyler> I guess this looks awful either way you handle it 😛

15:07:33 <talltyler> So, the better way for the bridge author to handle this would be a cutout in the pillar for the track to pass through, thereby only requiring them to set the corner pillar flags?

15:09:05 <peter1138> It looks bad without the track underneath, with glitchly pointy bits where to terrain slopes.

15:09:56 <talltyler> I imagine the number of bridge sets with full JGRPP support, for which this would change behavior for existing vanilla savegames, is pretty small. And maybe more likely to be maintained than ancient bridge GRFs.

15:11:10 <emperorjake> Bridge pillars are extremely limited, there's no way to make them conform to slopes

15:12:14 <peter1138> I imagine whatever we do with bridges would end up merged into JGRPP as it remains vanilla compatible.

15:15:13 <andythenorth> I am very +1 to “so don’t do that” if a bridge looks wrong in a specific location

15:16:51 <talltyler> I agree, I'm just sensitive to potential complaints of "you broke my savegame I've been playing for ten years" so I want to understand the ramifications of the change 🙂

15:17:19 <_jgr_> The only bridge set that I know of that does define the JGRPP properties for pillar flags is modular wider bridges. Not having the pillars ought to be fine for that set. If there is some wider compatibility issue I can deal with it in my branch, up to supporting both behaviours simultaneously if necessary.

15:30:26 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#pullrequestreview-3124166249

15:40:12 *** Wormnest has quit IRC (Read error: Connection reset by peer)

15:54:10 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#pullrequestreview-3124358670

15:56:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#pullrequestreview-3124363333

15:56:33 <yiffgirl> what's in the way of making a new bridges spec, anyhow?

15:59:41 <jfkuayue> where is the origin of “reverse n and u in the word bunny” meme

16:01:23 <yiffgirl> someone got excited after seeing a rabbit and their fingers slipped, and then it spread from there. or at least that's my theory

16:01:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#pullrequestreview-3124377393

16:02:26 <yiffgirl> peter1138: mission received: release 15.0 at all costs

16:03:38 <yiffgirl> i should update my cities pr, i think there was one actionable request

16:05:07 <yiffgirl> say, should ./cache be added to the gitignore?

16:09:18 <LordAro> that doesn't sound like anything our build system produces, so no

16:12:26 <peter1138> git has personal ignores as well, use that.

16:14:41 <DorpsGek> [OpenTTD/OpenTTD] Moth-Tolias commented on pull request #14504: Add: Identify cities on the main viewport by appending an icon to their names https://github.com/OpenTTD/OpenTTD/pull/14504#pullrequestreview-3124426179

16:14:55 *** Dead_Bush_Sanpai has quit IRC (Read error: Connection reset by peer)

16:17:33 <yiffgirl> should i squash the change of variable name or just have it as a new commit?

16:21:38 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#pullrequestreview-3124449264

16:21:57 <LordAro> yiffgirl: up to you. message still needs to conform :)

16:22:12 <LordAro> (and we'll probably end up squashing the whole PR on merge as well)

16:22:29 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#pullrequestreview-3124452149

16:23:35 *** Guest24227 has quit IRC (Ping timeout: 480 seconds)

16:28:53 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)

16:35:08 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517

16:59:42 <DorpsGek> [OpenTTD/OpenTTD] Moth-Tolias dismissed a review for pull request #14504: Add: Identify cities on the main viewport by appending an icon to their names https://github.com/OpenTTD/OpenTTD/pull/14504#pullrequestreview-3109761623

16:59:45 <DorpsGek> [OpenTTD/OpenTTD] Moth-Tolias updated pull request #14504: Add: Identify cities on the main viewport by appending an icon to their names https://github.com/OpenTTD/OpenTTD/pull/14504

17:10:25 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14504: Add: Identify cities on the main viewport by appending an icon to their names https://github.com/OpenTTD/OpenTTD/pull/14504#pullrequestreview-3124601877

17:11:16 <DorpsGek> [OpenTTD/OpenTTD] zephyris approved pull request #14504: Add: Identify cities on the main viewport by appending an icon to their names https://github.com/OpenTTD/OpenTTD/pull/14504#pullrequestreview-3124603598

17:15:31 <yiffgirl> oh, if the station signs don't switch i should definitely be doing that

17:19:52 <_zephyris> I might be wrong! I'm working from memory, from when I designed the TTF fonts, so I easily could have remembered incorrectly.

17:24:37 <peter1138> Stations don't switch between showing population and not show population.

17:41:48 *** Flygon has quit IRC (Remote host closed the connection)

17:47:28 <peter1138> > 17:22 < LordAro> (and we'll probably end up squashing the whole PR on merge as well)

18:01:40 <yiffgirl> peter1138: but do they switch between showing the icon and not showing the icon?

18:02:21 <yiffgirl> [not an actual question, i'll check myself in a minute]

18:04:23 <peter1138> They don't, but they probably should, as there's a spurious space character left over.

19:54:54 <DorpsGek> [OpenTTD/OpenTTD] killerdevildog closed pull request #14518: Fix issue #14485: Allow road vehicles to use cross-type stops as wayp… https://github.com/OpenTTD/OpenTTD/pull/14518

20:23:05 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)

20:46:24 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517#pullrequestreview-3125116162

20:49:29 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #14504: Add: Identify cities on the main viewport by appending an icon to their names https://github.com/OpenTTD/OpenTTD/pull/14504

21:21:00 <talltyler> Congrats yiffgirl (and thanks for the contribution!) 🙂

21:40:59 <kuhnovic> Nice work, it's a small detail but I like it a lot

21:42:12 <yiffgirl> i should probably start working on those followup prs, aha...

21:52:47 <kuhnovic> You don't have to do anything, all development is voluntary. But it would be much appreciated 😉

21:56:40 <DorpsGek> [OpenTTD/OpenTTD] ldpl opened pull request #14519: Fix: [Script] Invalid title on GSGoal.Question windows https://github.com/OpenTTD/OpenTTD/pull/14519

21:57:39 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)

22:07:40 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14519: Fix: [Script] Invalid title on GSGoal.Question windows https://github.com/OpenTTD/OpenTTD/pull/14519#pullrequestreview-3125265746

22:20:22 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14517: Change: Add bridge pillar exclusions for tiles under bridges. https://github.com/OpenTTD/OpenTTD/pull/14517

22:23:08 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14477: Feature: Allow stations under bridges and bridges over stations. https://github.com/OpenTTD/OpenTTD/pull/14477

22:55:13 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #14519: Fix: [Script] Invalid title on GSGoal.Question windows https://github.com/OpenTTD/OpenTTD/pull/14519

23:59:12 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)