[ML] Explain Log Rate Spikes: fix chart showing as empty when filter matches field/value pair in hovered row

[alvarezmelissa87]

Summary

Related issue: #138975

This PR fixes the issue of the main histogram chart showing as empty when the search filter matches field/value pair in hovered the hovered row. The main histogram now reflects the mini histogram.

• chart no longer shows as empty when there is data
• dual brush not moving all the way to the left
• brushes should not move on hover
  • this will be addressed in a follow up

Checklist

Delete any items that are not applicable to this PR.

• Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
• Documentation was added for features that require explanation or tutorials
• Unit or functional tests were updated or added to match the most common scenarios
• Any UI touched in this PR is usable by keyboard only (learn more about keyboard accessibility)
• Any UI touched in this PR does not create any new axe failures (run axe in browser: FF, Chrome)
• If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
• This renders correctly on smaller devices using a responsive layout. (You can test this in your browser)
• This was checked for cross-browser compatibility

[elasticmachine]

Pinging @elastic/ml-ui (:ml)

[peteharverson]

Not related to the changes here, but did we work out why the mini histogram in the row is incorrect in cases like this, where the field / value matches the filter:

[peteharverson]

LGTM. Testing with partial-.ds-cluster-kibana-filebeat-* I can no longer reproduce the original issue.

[alvarezmelissa87]

@elasticmachine merge upstream

[alvarezmelissa87]

Not related to the changes here, but did we work out why the mini histogram in the row is incorrect in cases like this, where the field / value matches the filter:

Not yet - looking into it 👍

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 5bca9c1

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
aiops	766.6KB	766.9KB	+217.0B

History

• 💚 Build #80232 succeeded 0068ff5
• 💚 Build #79616 succeeded 098f2b0fe634ca0acea46c846177c890332d46d9
• 💚 Build #79493 succeeded b5d6a4b34f1e40e2892bad93500b1d6801846357
• 💚 Build #78831 succeeded 0eb38b0ed12c5f138bfc29550ee51f4749aec5d1
• 💚 Build #78804 succeeded 936d99dbef83111f2a3547a2aa42a65951a93781
• 💔 Build #78526 failed 4a1fe4a374cfea2d06ab0795f415e8c263b0212f

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @alvarezmelissa87

[walterra]

Code & test LGTM. Sounds good to pick up additional fixes in follow ups and get this in.

Noticed the following:

When there's no "overall" data at all, we might still want to create a time series with all zeroes and pass it on to the chart. Otherwise we end up with a tooltip that's not showing a 0 for overall data.