feat(datagrid): add ability to unsort columns

[valentin-mladenov]

PR Checklist

Please check if your PR fulfills the following requirements:

• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
• If applicable, have a visual design approval

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• Other... Please describe:

What is the current behavior?

Once turned on column sort can only be rotated between ASC and DESC when clicking on the column, but can't be turned off.

Issue Number: CDE-2416

What is the new behavior?

Column sort can be turned off by clicking third time on the column.

Does this PR introduce a breaking change?

• Yes
• No

Other information

[github-actions]

👋 @valentin-mladenov,

• 🙏 The Clarity team thanks you for opening a pull request
• 🎉 The build for this PR has succeeded
• 🔍 The PR is now ready for review
• 🍿 In the meantime, view a preview of this PR
• 🖐 You can always follow up here. If you're a VMware employee, you can also reach us on our internal Clarity Support space

Thank you,

🤖 Clarity Release Bot

[Jinnie]

LGTM

[kevinbuhmann]

I would reword the PR title and commit message subject to "add ability to unsort columns".

[valentin-mladenov]

@kevinbuhmann clrDgSortDisableUnsort input is added in order to turn on/off the unsorted state of a column.
Does the name make sense?

@dtsanevmw can you review the changes we disscussed separately?

[kevinbuhmann]

Has this gone through design? I'd like to see the proposed documentation/guidance before we release this. I'm not clear on why we need the ability to disable unsort. That might cause unnecessary UX differences in products. As a user I would not expect not to be able to unsort some grids/columns, but not all of them.

[kevinbuhmann]

See my comments.

All changes are resolved.

[github-actions]

Hi there 👋, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed PRs after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.