Improve Sorting Logic

[stramel]

Closes #28

Defaults the column.sort to byTextAscending and inverts the result for descending sorts.

[Buuntu]

This looks good, thanks! Do you mind adding a quick test case that covers the bug/case you found in #28? And using Angular commit convention so the version is automatically bumped? See https://github.com/Buuntu/react-final-table/blob/master/CONTRIBUTING.md

[stramel]

I don't think there is a bug to really cover in #28. Just improvements with the performance and code size reduction.

[codecov-io]

Codecov Report

Merging #29 (fb8d9cf) into master (250a068) will increase coverage by 1.34%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   97.70%   99.05%   +1.34%     
==========================================
  Files           4        4              
  Lines         218      211       -7     
  Branches       49       44       -5     
==========================================
- Hits          213      209       -4     
+ Misses          5        2       -3     

Impacted Files	Coverage Δ
src/hooks.tsx	98.85% <100.00%> (+1.61%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 250a068...fb8d9cf. Read the comment docs.

[Buuntu]

I don't think there is a bug to really cover in #28. Just improvements with the performance and code size reduction.

That's true, okay looks good to me!

[github-actions]

🎉 This PR is included in version 1.5.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀