Skip to content

Create railway/signal_box preset#1835

Merged
tordans merged 1 commit intomainfrom
kh/signal-box
Dec 14, 2025
Merged

Create railway/signal_box preset#1835
tordans merged 1 commit intomainfrom
kh/signal-box

Conversation

@k-yle
Copy link
Copy Markdown
Collaborator

@k-yle k-yle commented Nov 28, 2025

Description, Motivation & Context

This was blocked by #1728, which introduced fields that this preset needs.

Related issues

part of #1641

Links and data

Test-Documentation

Preview links & Sidebar Screenshots

w401698613:
image

Search

image

Info-i

see above. Also different help text for each tag value of railway:signal=*:

Mechanical Signal box with mechanical railway signalling systems.
Electric Signal box with electromechanical railway signalling systems.
Digital Signal box with track diagram interlocking.
Fully Automated Signal box with fully computerised interlocking system.

Wording

  • American English
  • name, aliases (if present) use Title Case
  • terms (if present) use lower case, sorted A-Z

@github-actions
Copy link
Copy Markdown

🍱 Your pull request preview is ready

Please use this preview to check your changes. Ideally use the test documentation template and document your test results by commenting on the PR. This will speed up the review process for everyone.

FYI, once this PR is merged, you can use the iD Editor Preview to test your changes in interaction with all other changes.

Comment thread data/presets/railway/signal_box.json
Copy link
Copy Markdown
Collaborator

@tordans tordans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tordans tordans changed the title create signal_box preset Create railway/signal_box preset Dec 14, 2025
@tordans tordans merged commit c05c7f8 into main Dec 14, 2025
8 checks passed
@k-yle k-yle deleted the kh/signal-box branch March 12, 2026 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants