fix(DropdownMenu/ContextMenu): correct bindings of checkbox items

[EvertonWingert]

🔗 Linked issue

Resolves #2892

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This PR fixes the DropdownMenu.CheckboxItem, which now uses the modelValue property instead of :checked and @update:checked. According to the official documentation (https://reka-ui.com/docs/components/dropdown-menu#checkboxitem), the :checked and @update:checked properties are no longer available.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[pkg-pr-new]

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

commit: ba9654f

[benjamincanac]

Thanks for the fix! Would you mind doing the same thing on the ContextMenu in the same PR? 😊

[benjamincanac]

I'm wondering if we should not use the same syntax as well: :model-value="item.modelValue" @update:model-value="item.onUpdateModelValue". What do you think?

I missed that when migrating to Reka UI but I would have probably renamed it.

[EvertonWingert]

I also agree to change it so that it is the same as the renka ui, it will make maintenance easier in the future

[benjamincanac]

Can you take care of it or do you want me to do it?

[EvertonWingert]

You can do it, I won't have time now to make all the changes and test it

[benjamincanac]

Now that I think of it, I'm not really sure we should rename all of this since it only applies to items with type checkbox. Let's merge this for now to fix the issue and figure it out later!