Add href support for default actions in Basic Table

[thefaded]

Summary

Fixes : #1792

EuiBasicTable default action accepts support both href or onClick

Checklist

• Check against all themes for compatibility in both light and dark modes
• Checked in mobile
• Checked in IE11 and Firefox
• Props have proper autodocs
• Added documentation examples
• Added or updated jest tests
• Checked for breaking changes and labeled appropriately
• Checked for accessibility including keyboard-only and screenreader modes
• A changelog entry exists and is marked appropriately

[kibanamachine]

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

[cla-checker-service]

💚 CLA has been signed

[cchaos]

Thanks @thefaded , Can you please sign the CLA and describe in the PR summary the actions you took to fix the issue?

[thefaded]

@cchaos, sure. I'm also updated changelog.

[chandlerprall]

Should I add samples to docs with href? (from #1792 (comment))

Yes, updating one or two specific actions with href would be great. Would also love to see a third action added to src/components/basic_table/collapsed_item_actions.test.tsx so we can better prevent future regressions.

[chandlerprall]

jenkins test this

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_3115/

[chandlerprall]

Changes LGTM; pulled & tested locally

[cchaos]

@thefaded Can you also update this doc example's action item from using an onClick to an href: '#'?

eui/src-docs/src/views/tables/actions/actions.js

Lines 188 to 194 in 2bc9917

	name: 'Share',
	isPrimary: true,
	description: 'Share this user',
	icon: 'share',
	type: 'icon',
	onClick: () => {},
	'data-test-subj': 'action-share',

[cchaos]

Jenkins test this

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_3115/

[cchaos]

Sorry for the delay on this. I just realized there is a docs example added already. I've fixed merge conflicts and once the build passes, I'll merge. Thanks @thefaded !