fix(plugin-chart-handlebars): Sort-By and Sort-By-Descending control not work

[stephenLYZ]

SUMMARY

This PR fixes the problem that sort-by control not working in the handlebar chart.

• use normalizeOrderBy to normalize the value of sort-by control
• only make sort-descending control visible when sort-by control has values
• change the order of the sort-descending control

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

before

2022-08-07.10.35.18.mov

after

2022-08-07.10.33.15.mov

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #21005 (4ceac51) into master (e214e1a) will decrease coverage by 0.00%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master   #21005      +/-   ##
==========================================
- Coverage   66.34%   66.34%   -0.01%     
==========================================
  Files        1767     1767              
  Lines       67312    67313       +1     
  Branches     7144     7144              
==========================================
  Hits        44656    44656              
- Misses      20828    20830       +2     
+ Partials     1828     1827       -1     

Flag	Coverage Δ
javascript	52.11% <33.33%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...lugin-chart-handlebars/src/plugin/controlPanel.tsx	50.00% <ø> (ø)
...n-chart-handlebars/src/plugin/controls/orderBy.tsx	28.57% <0.00%> (-11.43%)	⬇️
...s/plugin-chart-handlebars/src/plugin/buildQuery.ts	100.00% <100.00%> (+25.00%)	⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[rusackas]

/testenv up

[rusackas]

I have a handlebars chart I'll test this with asap...

[github-actions]

@rusackas Ephemeral environment spinning up at http://35.162.149.229:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.