perf(stark-ui): set ChangeDetectionStrategy.OnPush to UI components to prevent Angular from running unnecessary change detection cycles

[christophercr]

ISSUES CLOSED: #1331

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/NationalBankBelgium/stark/blob/master/CONTRIBUTING.md#-commit-message-guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #1331

What is the new behavior?

All UI components from stark-ui use ChangeDetectionStrategy.OnPush except these ones because they are more complex or they implement the ControlValueAccessor API:

• AppSidebar
• DatePicker
• DateRangePicker
• DateTimePicker
• Dropdown
• GenericSearch

This PR just applies to components and NOT directives.

Does this PR introduce a breaking change?

[X] Yes (to be confirmed)
[ ] No

[coveralls]

Coverage decreased (-0.03%) to 92.85% when pulling dff48b9 on christophercr:feature/ui-components-performance into d7f5fac on NationalBankBelgium:master.

[christophercr]

@SuperITMan please don't merge this PR yet because I'm still working on it, trying to enable the onPush strategy in as much components as I can 😉

I'll let you know when I finish 👍

[christophercr]

@SuperITMan @carlo-nomes @nicanac I've updated this PR. I've enabled the OnPush strategy to almost all components.

I didn't enable it in the components that implement the ControlValueAccessor API and neither in the AppSidenav and GenericSearch components. I think we should implement this in a separate PR because it might be tricky to implement and we should thoroughly test that change in those components to avoid regressions.

So this PR can already be reviewed and merged if that is ok for you guys ;)