opened
aurel-l
username_0
## Purpose
Add a third state to a column header where it can be visually marked as sortable even though it is not sorted.

## Approach
Visual change, opacity 0.7 on a sortable column

## Testing
Updated snapshots, checked story

## Stories
data table story

## Checklist
- [x] My PR is scoped properly, and “does one thing only”
- [x] I have reviewed my own code
- [x] I have checked that linting checks pass and type safety is respected
- [x] I have checked that tests pass and coverage has at least improved, and if not explained the reasons why
- [x] Were all the edited/created files in TypeScript? If not, please list the files you left in JavaScript and the reason to not update them.
- [ ] For the stories you created/updated, are the props modifiables through knobs?
add a visual cue that a column is sortable even though it is not sorted
issue
created
aurel-l
username_0
Be the trigger of what? And do you mean the whoooole table header (as in, the whole set of column headers?)?
comment
created
xwatkins
username_1
Actually it is already, it's just that I think `cursor: pointer;` should be on `&--sortable` not just the `:after`
comment
created
aurel-l
username_0
Ah, I see. I wasn't sure what you were refering to as the behaviour was there already. I have now modified the cursor style to cover the whole column header.
comment