-
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
docs: (platform) added stackBlitz url for button example files #1494
Conversation
Deploy preview for fundamental-ngx ready! Built with commit 276b706 |
Deploy preview for fundamental-ngx ready! Built with commit d20f2ad |
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.
Only one change form my site. Rest seems to be ok.
@@ -131,7 +133,11 @@ export class CodeExampleComponent implements OnInit, AfterViewInit { | |||
this.exampleFiles.forEach(example => { | |||
|
|||
if (example.fileName && example.component) { | |||
this.parameters.html_tag = 'fd-' + example.fileName; | |||
|
|||
if (example.fileName !== undefined && example.fileName.indexOf('platform') !== -1) { this.parameters.html_tag = 'fdp-' + example.fileName; } |
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.
Try to keep
if () {
} else {
}
structure of statements.
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.
updated
We should also stop depending on string containing |
7a38db1
to
d20f2ad
Compare
Please provide a link to the associated issue.
Please provide a brief summary of this pull request.
added stackBlitz url for button example files
If this is a new feature, have you updated the documentation?
yes
README.md