AccessKit / accesskit

Accessibility infrastructure for UI toolkits
https://accesskit.dev
BSD 3-Clause "New" or "Revised" License
1.05k stars 53 forks source link

refactor!: Drop `SortDirection::Unsorted` #391

Closed mwcampbell closed 5 months ago

mwcampbell commented 5 months ago

This continues the pattern of eliminating states that don't have a clear meaning. In this case, according to the MDN documentation for aria-sort, the default value for that ARIA property is none. This means that specifying aria-sort="none" is synonymous with not specifying it at all. So, like other AccessKit enums such as AutoComplete, I think it's best to not have a None value, which is what SortDirection::Unsorted really is; instead, the user just shouldn't set the property at all.