Skip to content
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

[Bug]: data-disabled attribute is missing on NumberFieldRoot element #1211

Closed
michaelhthomas opened this issue Aug 10, 2024 · 0 comments · Fixed by #1212
Closed

[Bug]: data-disabled attribute is missing on NumberFieldRoot element #1211

michaelhthomas opened this issue Aug 10, 2024 · 0 comments · Fixed by #1212
Assignees
Labels
bug Something isn't working

Comments

@michaelhthomas
Copy link

Environment

Developement/Production OS: macOS Sonoma 14.5 (23F79)
Node version: 20.15.1
Package manager: [email protected]
Radix Vue version: 1.9.3
Vue version: 3.4.35
CSS framework: [email protected]
Client OS: macOS Sonoma 14.5 (23F79)
Browser: Firefox 129.0

Link to minimal reproduction

https://stackblitz.com/edit/ak9f9h?file=src%2FApp.vue

Steps to reproduce

  1. Open the reproduction
  2. Observe that the input background is not red, as the data-disabled attribute is missing from the NumberFieldRoot component.

Describe the bug

The documentation for the Number Field component states that the NumberFieldRoot element will have a data-disabled attribute, provided it is disabled. Unfortunately, this attribute is missing in practice.

Expected behavior

The data-disabled attribute is present on the root element of a disabled NumberField.

Context & Screenshots (if applicable)

No response

@michaelhthomas michaelhthomas added the bug Something isn't working label Aug 10, 2024
@michaelhthomas michaelhthomas changed the title [Bug]: [Bug]: data-disabled attribute is missing on NumberFieldRoot element Aug 10, 2024
@zernonia zernonia self-assigned this Aug 11, 2024
@zernonia zernonia linked a pull request Aug 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants