Skip to content

Conversation

@Ruzihm
Copy link
Contributor

@Ruzihm Ruzihm commented Jan 5, 2026

Adds the ability to set the text value of an input via winset.

Test Setup

  1. Starting with TestGame, add the following verb to mob:
	verb/set_text()
		set name = "Set Text"
		winset(src,"outputwindow.input","text=asdf")
  1. Compile and start server
  2. Click "Set Text" verb in Verbs tab (test case 1)
  3. Clear input box of any content, then type set-text in input box and press enter (test case 2)
  4. Clear input box of any content, then type .winset "outputwindow.input.text="asdf"" in text box and press enter (test case 3)

Test Results

master OD:
The text box will be empty for all test cases

PR:
The text box will be set to asdf for all test cases

@boring-cyborg boring-cyborg bot added the Client Involves the OpenDream client label Jan 5, 2026
@github-actions github-actions bot added the size/S label Jan 5, 2026
@wixoaGit
Copy link
Member

wixoaGit commented Jan 5, 2026

Could you instead reset the text before running the command?

@Ruzihm
Copy link
Contributor Author

Ruzihm commented Jan 5, 2026

Could you instead reset the text before running the command?

You know, I must have had a typo in an earlier test where I had something like:

.winset "outputwindow.input.text="" ? outputwindow.input.text="empty" : outputwindow.input.text="not empty""

because now when I try it in byond, it shows as empty. so yes that should work. 👍

@Ruzihm Ruzihm marked this pull request as draft January 5, 2026 23:18
@github-actions github-actions bot added size/XS and removed size/S labels Jan 5, 2026
@Ruzihm Ruzihm marked this pull request as ready for review January 5, 2026 23:24
@wixoaGit wixoaGit merged commit 0d80ee2 into OpenDreamProject:master Jan 6, 2026
20 checks passed
@Ruzihm Ruzihm deleted the winset-text-override-reset branch January 8, 2026 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client Involves the OpenDream client size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants