-
Notifications
You must be signed in to change notification settings - Fork 142
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
Text field is unusable when placed inside a radio group #504
Comments
Hey @gabrielBusuioc! Could you provide a bit more information on what the issue you're seeing is? In particular, what does it mean that "the text field will lose focus whenever clicked upon"? I ask because I tried to reproduce this issue and whenever I click on the text field it focuses and remains in focus until clicked away, which is all correct behavior as far as I can tell. Extra note: Not sure if this might be contributing to this issue or not, but as a heads up there's an issue with your sample code. There is a known issue where <vscode-radio-group orientation="vertical">
<label slot="label">What meals do you prefer?</label>
<vscode-radio value="pasta">Pasta</vscode-radio> <!-- Note `value` attribute -->
<vscode-radio value="pizza">Pizza</vscode-radio> <!-- Note `value` attribute -->
<vscode-radio value="other">Others...</vscode-radio> <!-- Note `value` attribute -->
<vscode-text-field placeholder="Fish, Burgers...">ss</vscode-text-field>
</vscode-radio-group> |
At a higher level, I'm also curious about what your use case is for putting a text field inside a radio group. I can't say I've seen anyone try that before, so I'd love to know more about what you're hoping to accomplish. |
Hey @hawkticehurst, Regarding the use case, I just want to achieve a form-like radio group that will allow one to give input other than the ones specified. This is a basic HTML example that would highlight what I am trying to do here. Please let me know if you need additional input from me. |
Ahhh nice, I totally see what you're going for now. Thank you for clarifying! Also good to know that adding As for the focus issue, I'm still unable to reproduce the issue you've shown in the gif you shared (thank you btw!) –– could you create and share a small reproduction of the issue so I can see more details of what you're doing? |
Hi, I've attached a zip file that contains the source code for an extension where this issue is reproducible. To run the extension just follow the steps from the readme file. Here is some additional information that might be useful for reproducing this issue: Hope it helps :D |
Hi @hawkticehurst, Were you able to reproduce the issue? |
Hey @gabrielBusuioc, apologies for the radio silence! I split my time between maintaining the toolkit and a handful of other projects at Microsoft so I usually only have a few days a month where I can dedicate my full attention to toolkit issues before I have to move on to the next thing. I'm still deep in the weeds on other projects but I at least wanted to say hi, that I haven't forgotten about this issue, and will return to it soon-ish.. 😅 With that said, if you're open to publishing your sample to a github repo and sending me the link (versus using a ZIP file) that would actually be really helpful. I won't be able to do a proper debugging session for a while but I can likely review the code (via GitHub's UI) if/when I find myself with some free pockets of time over the next couple of weeks and might (??) be able provide some suggestions/answers sooner versus later. |
Hi @hawkticehurst, I totally understand that time is not on your side. It`s the same for me as well these days xD. Really appreciate that you took the time to get back to me! I published the sample on this github repo. Hope it helps! |
Thanks @gabrielBusuioc! From a quick pass can you also make sure to include your |
Hi @hawkticehurst, My mistake... I forgot to add the hidden files :D |
Describe the bug
When placing a tag of type
<vscode-text-field>
inside a tag of type<vscode-radio-group>
, the text field will lose its focus whenever clicked upon, thus making it unusable.Code sample:
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: