Slash command popout should show option name instead of value

[ghost]

Description

I registered a slash command with an integer argument with the following options:

"choices": [
	{ "name": "first",  "value": 1 },
	{ "name": "second", "value": 2 },
	{ "name": "third",  "value": 3 },
],

if I run a command and select one of those options, then copy paste the command text from the popup and try to re-run the command, it prompts me to select an option again instead of filling in the value.

Steps to Reproduce

• Register a command with an integer argument, and have options with string names
• Run the command and select option "second"
• Click the "used /command" and copy the text
• Paste it into the message box to run the command again

Expected Behavior

The option is correctly filled out.

Current Behavior

The option is filled out with the number value instead of the string name and the user is prompted to pick the option again.

Screenshots/Videos

Client and System Information

Stable 96355 (c244e32) Google Chrome 92.0.4515.159 Windows 10 64-Bit

[ghost]

Why was this moved to a discussion?

[typpo]

The slash command popout is working as intended, but it's a good suggestion

[ghost]

so the intention is to not be able to copy-paste the command to run it again? that seems like really shitty user experience

[msciotti]

Hostility is not necessary.

The intention of the pop-out is to show the command that was run. The way that this data model is built, the message does not have access to the "human readable" name of the option type. Just the value.

The side effect of that architecture is that copy/paste doesn't work in this case. Which admittedly, is an experience we should support better in general.

That can done by making the change suggested here, or making another way to re-run previous commands.

[ghost]

I could've phrased that better. I assumed that having the text in the popout be selectable was made so users could copy-paste the text to run the command again - and my assumption was apparently incorrect. Sorry about that.

[PintTheDragon]

Maybe we can accept both the name and value? There shouldn't be conflicts with that, right?

[psibean]

I'd agree showing the name in the popout is much nicer. For the user, as has been mentioned, they can copy the command to run it again if they need to, or in the case of a non-ephemeral response, other users can learn how a command was used in a particular way.

[yonilerner]

The name in the popout will now show on Desktop Canary. iOS is coming, Android will be delayed a bit