fix: remove unnecessary prop, add required aria attribute

[felix-ico]

this should fix #2190

and i removed the optional ariacheckedstate, i guess it can be derived by the selected prop

[netlify]

✅ Deploy Preview for marvelous-moxie-a6e2fe ready!

Name	Link
🔨 Latest commit	0f2daeb
🔍 Latest deploy log	https://app.netlify.com/sites/marvelous-moxie-a6e2fe/deploys/65857ed4866feb0008bccdeb
😎 Deploy Preview	https://deploy-preview-2192--marvelous-moxie-a6e2fe.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[acstll]

Instead of the ternary below when defining the role, why not add a default value to the prop? would that work?

@Prop() ariaRoleTitle?: 'switch' | 'radio' | ... = 'switch';

and I would use these as possible types, according to mdn:

• checkbox
• menuitemcheckbox
• menuitemradio
• option
• radio
• switch

---

btw it's a shame this is called ariaRoleTitle instead of innerAriaRole, no? 😕

[felix-ico]

Instead of the ternary below when defining the role, why not add a default value to the prop? would that work?

this is fixed

btw it's a shame this is called ariaRoleTitle instead of innerAriaRole, no? 😕

yes but i guess changing this would be a breaking change if people are using the old prop name...