[ML] Redesign file-based Data Visualizer

[qn895]

Summary

This PR is a follow up of #85726 which updates the file based data visualizer to follow the new design. It also updates the functional tests for file based data visualizer.

Before

After

Todo/Follow up PRs

• Better support for geo fields (will do in a follow up PR)

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
• Unit or functional tests were updated or added to match the most common scenarios

[pheyos]

One of my test files (ts_head.log) crashes the ML app on data visualizer load (blank page, have to switch to a different Kibana app and come back to ML again). The console shows the following error:

[mdefazio]

I think this is quite close! Just a few comments:

1. It looks like we have different gutter sizes set on the flex group for the expanded row depending on the field type.
   Keyword:
   

Number:

2. It might be nice to force the stats table to have a max-width and then the subsequent columns will start at the same point

With a width set (instead of a percent value for the columns).

[qn895]

Thanks @mdefazio I have updated the gutter size to all match in the latest changes as well as having a fixed pixel number for the Document stats column

[peteharverson]

Tested latest edits and LGTM

[jgowdyelastic]

To be using useUiSettings in a hook like this, shouldn't it be watching for changes in the settings?
Like this function:

kibana/x-pack/plugins/ml/public/application/components/color_range_legend/use_color_range.ts

Line 192 in f7fdda5

export function useCurrentEuiTheme() {

Otherwise it doesn't need to be in a hook.

[qn895]

You're right. This is now updated here e09dc6f

[jgowdyelastic]

LGTM

[mdefazio]

Tested locally and this all looks good.

[darnautov]

LGTM 🚀

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: f46544e

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
ml	1694	1712	+18

Async chunks

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

id	before	after	diff
ml	7.1MB	7.0MB	-23.1KB

History

• 💚 Build #99613 succeeded e09dc6f
• 💚 Build #99248 succeeded e69af6a
• 💚 Build #99158 succeeded e296ac0
• 💔 Build #99017 failed a640650
• 💔 Build #98426 failed 514f4a9

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