8000 Fix list inputs on component data input form by TobleroneSwordfish · Pull Request #23779 · goonstation/goonstation · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix list inputs on component data input form #23779

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

TobleroneSwordfish
Copy link
Contributor

About the PR

Refactors the /datum/proccall_editor data entry form to allow for DATA_INPUT_LIST_PROVIDED to work properly.

Why's this needed?

Some components use this and it doesn't work right now, also I want to use this form for things like admin add antag options.

Testing

image
(ignore the fact that the tool flags are still extremely cursed)

@TobleroneSwordfish TobleroneSwordfish added A-Admin Deals with administration tools or similar A-Internal Deals with major internal, generally non-player facing code - e.g. types labels Jun 8, 2025
@boring-cyborg boring-cyborg bot added the A-UI Modifies UI in some way. Automatically applied on a change to tgui/ label Jun 8, 2025
@github-actions github-actions bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict labels Jun 8, 2025
@TobleroneSwordfish TobleroneSwordfish merged commit 5716938 into goonstation:master Jun 11, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Admin Deals with administration tools or similar A-Internal Deals with major internal, generally non-player facing code - e.g. types A-UI Modifies UI in some way. Automatically applied on a change to tgui/ S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0