Should it take a `SortMode` instead of a String?