fix(core,platform): add ability to show summary step

[N1XUS]

Please provide a link to the associated issue.

Closes #6132

Please provide a brief summary of this pull request.

Added ability to specify if summary step should be visible in progress bar for core wizard and platform wizard generator.

Please check whether the PR fulfills the following requirements

• the commit message follows the guideline:
  https://github.com/SAP/fundamental-ngx/blob/main/CONTRIBUTING.md
• tests for the changes that have been done
• all items on the PR Review Checklist are addressed :
  https://github.com/SAP/fundamental-ngx/wiki/PR-Review-Checklist
• Run npm run build-pack-library and test in external application

Documentation checklist:

• n/a update README.md
• Breaking Changes Wiki
• Documentation Examples
• Stackblitz works for all examples

[netlify]

✔️ Deploy Preview for fundamental-ngx ready!

🔨 Explore the source changes: 51f1274

🔍 Inspect the deploy log: https://app.netlify.com/sites/fundamental-ngx/deploys/6116a149dc446d00074128c5

😎 Browse the preview: https://deploy-preview-6244--fundamental-ngx.netlify.app

[github-actions]

This pull request is stale because it has been open 2 days with no activity. Remove stale label or comment or this will be closed in 3 days

[InnaAtanasova]

prob not related to your PR, but the branching example is not working because the dropdown to select the type is not working:

[platon-rov]

Not sure if this should be included in current PR but the animation for switching steps for the mobile mode too fast, it blinks.

[droshev]

@N1XUS can you rebase as maybe the e2e were fixed?

[mikerodonnell89]

Not sure if this should be included in current PR but the animation for switching steps for the mobile mode too fast, it blinks.

@platon-rov Looks like this is happening to the progress bar but good catch. Should be considered a separate issue. New issue here: #6294

[mikerodonnell89]

prob not related to your PR, but the branching example is not working because the dropdown to select the type is not working:

Can confirm not unique to this pr @InnaAtanasova new issue here #6295

[mikerodonnell89]

@N1XUS I see that the summary is listed as the last step for wizard in dialog on the platform generator but not on core

[N1XUS]

@N1XUS I see that the summary is listed as the last step for wizard in dialog on the platform generator but not on core

@mikerodonnell89 please check this link: https://deploy-preview-6244--fundamental-ngx.netlify.app/#/core/wizard#visible-summary

[mikerodonnell89]

@N1XUS I see that the summary is listed as the last step for wizard in dialog on the platform generator but not on core

@mikerodonnell89 please check this link: https://deploy-preview-6244--fundamental-ngx.netlify.app/#/core/wizard#visible-summary

@N1XUS after discussing with @droshev we can confirm that this should be default behavior - so we should also be seeing it in this example: https://deploy-preview-6244--fundamental-ngx.netlify.app/#/core/wizard#dialog

[mikerodonnell89]

I do not think this needs to be an input. It can be a hidden property of the wizard-step component, and the wizard can iterate through ContentChildren to set this property when [displaySummaryStep]="true" on the wizard component.

[mikerodonnell89]

Looks good, just two small things, see above comments

[lgtm-com]

This pull request introduces 1 alert when merging 80e4b05 into 17a6cb5 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class