[Enterprise Search] Updated product_selector to match new No Data screens

[cchaos]

Pulled work out of original PR #107709

Mainly visual changes to the overview product selector screen which now uses KibanaPageTemplate directly.

And updated the product images to match the new UI.

Checklist

• Unit or functional tests were updated or added to match the most common scenarios
• Any UI touched in this PR does not create any new axe failures (run axe in browser: FF, Chrome)
• This renders correctly on smaller devices using a responsive layout. (You can test this in your browser)
• This was checked for cross-browser compatibility

[cchaos]

Hey Enterprise Search team, sorry for the double-review. I've pulled this out of the other PR so it wouldn't get prolonged by the updates from the other solutions.

[scottybollinger]

LGTM 🚀

[cee-chen]

🎉 Wahoo! Changes look great and super clean! I think @daveyholler or @johnbarrierwilson has to approve as well, but this should be good to go code wise!

[cchaos]

Thank you both!! 💟

[daveyholler]

HUZZAH!

[cee-chen]

Argh, sorry about that failing test Caroline. Definitely a good lesson in not hooking our test assertions into CSS classes (vs. data-test-subj) 🙈 🤦‍♀️

You should be able to fix it by replacing this line:

https://github.com/elastic/kibana/blob/master/x-pack/plugins/enterprise_search/public/applications/index.test.tsx#L67

with

      expect(mockContainer.querySelector('.kbnPageTemplate')).not.toBeNull();

[cchaos]

No worries. Thanks for the tip @constancecchen . I'll be sure to run the test locally too.

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: 35be122
• Storybooks Preview

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
enterpriseSearch	1482	1472	-10

Async chunks

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

id	before	after	diff
enterpriseSearch	2.2MB	2.1MB	-19.3KB

Unknown metric groups

miscellaneous assets size

id	before	after	diff
enterpriseSearch	886.8KB	1.0MB	+142.8KB

History

• 💔 Build #145393 failed 1d60e16
• 💔 Build #145374 failed d1c1c7e

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

[kibanamachine]

💚 Backport successful

Status	Branch	Result
✅	7.x

This backport PR will be merged automatically after passing CI.

[gchaps]

@daveyholler, @scottybollinger Here are a few suggestions for the UI copy

Change intro sentence?

Instead of "Choose a product to set up and get started", which I think users can figure out, how about doing this instead:

Welcome to Elastic Enterprise Search
Add search to your app or organization with Elastic App Search and Workplace Search.

Then you would have just 1 line here:

Enterprise-grade features
Watch our videos to see what you can do when search is made easy.

suggest "our videos because it is friendlier

Add a comma after experiences

Enterprise-grade features
Enterprise authentication via SAML, document-level permission and authorization support, custom search experiences**,** and more with a Platinum license.

[scottybollinger]

@daveyholler, @scottybollinger Here are a few suggestions for the UI copy

Change intro sentence?

Instead of "Choose a product to set up and get started", which I think users can figure out, how about doing this instead:

Welcome to Elastic Enterprise Search
Add search to your app or organization with Elastic App Search and Workplace Search.

Then you would have just 1 line here:

Enterprise-grade features
Watch our videos to see what you can do when search is made easy.

suggest "our videos because it is friendlier

Add a comma after experiences

Enterprise-grade features
Enterprise authentication via SMAL, document-level permission and authorization support, custom search experiences**,** and more with a Platinum license.

@jonasll @zumwalt @aznick thoughts?

[jonasll]

Looks good to me.

Small typo SMAL -> SAML

[gchaps]

Good catch. Updated the text.