add defaultOrder field to tablecard header

[rrennick]

Fixes #2136

This PR adds a defaultOrder field to the headers prototype of the TableCard component to allow specifying whether the default sort column is initially sorted in ascending or descending order.

Screenshots

Screenshot showing query is an empty object, header defaultOrder set, chevron showing ascending:

Detailed test instructions:

• Use my example component Table sorting/filtering action-scheduler-admin#6
• Alternative would be to test by modifying the example component in this repo

[rrennick]

I noticed today that clicking the chevron the first time updates the query object from empty to "asc" even when it's already set to default "asc".

[rrennick]

clicking the chevron the first time updates the query object from empty to "asc" even when it's already set to default "asc".

This was fixed in 2ef82bc

[psealock]

LGTM, you'll just want to add a note to the changelog to go along with this.

https://github.com/woocommerce/woocommerce-admin/blob/master/packages/components/CHANGELOG.md

[rrennick]

you'll just want to add a note to the changelog to go along with this.

Added. I'm getting a linter error after merging in master:

.../wp-content/plugins/woocommerce-admin/client/dashboard/store-performance/index.js
  202:5  error  'hiddenIndicators' is not defined  no-undef

[psealock]

Weird how that slipped through. This line

woocommerce-admin/client/dashboard/store-performance/index.js

Line 202 in 9908770

hiddenIndicators,

Should be hiddenIndicators: [],. cc @Aljullu just to confirm. I tried it out and it works as expected with no indicators.

[psealock]

I took care of the build error here #2184. This one is ready to 🚢

[rrennick]

@psealock Thanks :)