-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: (Platform) fix combobox examples for stackblitz #4469
Conversation
Deploy preview for fundamental-ngx ready! Built with commit 6289ebf |
@@ -89,7 +84,7 @@ export class ComboboxComponent extends BaseCombobox implements OnInit, AfterView | |||
|
|||
/** @hidden */ | |||
ngOnInit(): void { | |||
const providers = this.providers.size === 0 ? this._comboboxConfig.providers : this.providers; | |||
const providers = this.providers && this.providers.size === 0 ? this._comboboxConfig.providers : this.providers; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we a using TS greater than 3.7 it could be simplified:
const providers = this.providers && this.providers.size === 0 ? this._comboboxConfig.providers : this.providers; | |
const providers = this.providers?.size === 0 ? this._comboboxConfig.providers : this.providers; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are modifying combobox.component.ts
in the lib so the PR must be a fix
not a docs
Some examples are not working:
c331207
to
6289ebf
Compare
Please provide a link to the associated issue.
Part of #4392
Please provide a brief summary of this pull request.
All Combobox examples work on StackBlitz
Please check whether the PR fulfills the following requirements
https://github.com/SAP/fundamental-ngx/blob/main/CONTRIBUTING.md
https://github.com/SAP/fundamental-ngx/wiki/PR-Review-Checklist
Documentation checklist: