fix(FileUpload): add aria-disabled attribute when disabled

[innocenzi]

❓ Type of change

• 👌 Enhancement

📚 Description

Adds aria-disabled to the element acting as a button inside FileUpload. My use case is supporting this:

:root[data-use-pointer-cursors="true"]
	:where(a[href], button, [role="button"], input[type="button"], input[type="submit"], input[type="reset"], label[for], summary):not(:disabled):not([aria-disabled="true"]) {
	cursor: pointer;
}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 39e4d287-c69b-4819-8c3b-0e8ef900102e

📥 Commits

Reviewing files that changed from the base of the PR and between c6ad4ff and 37690a8.

⛔ Files ignored due to path filters (2)

• test/components/__snapshots__/FileUpload-vue.spec.ts.snap is excluded by !**/*.snap
• test/components/__snapshots__/FileUpload.spec.ts.snap is excluded by !**/*.snap

📒 Files selected for processing (1)

• src/runtime/components/FileUpload.vue

✅ Files skipped from review due to trivial changes (1)

• src/runtime/components/FileUpload.vue

---

📝 Walkthrough

Walkthrough

In FileUpload.vue, the root element now conditionally sets aria-disabled based on variant. The attribute is omitted for the button variant and otherwise reflects the computed disabled state.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the change: FileUpload now sets aria-disabled when disabled.
Description check	✅ Passed	The description is directly related to the change and explains the aria-disabled use case.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[pkg-pr-new]

npm i https://pkg.pr.new/@nuxt/ui@6653

commit: 37690a8

[benjamincanac]

I think aria-disabled is redundant on the button variant since native disabled already covers it. The variant that actually needs it is area (a <div role="button">). Could you flip it?

:aria-disabled="variant === 'button' ? undefined : (disabled || undefined)"