-
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: adds links to headers #1260
docs: adds links to headers #1260
Conversation
|
Deploy preview for fundamental-ngx ready! Built with commit f875f9c |
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.
Hi @stefanoScalzo
The scroll functionlity should be in docs-section-title.component
, every instance of this component can read the fragment
from url and check with assigned id. Also you should change naming of fragment
to idFromUrl
or something like that.
}); | ||
} | ||
|
||
ngAfterViewInit(): void { |
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.
I would recommend putting the scroll functionality in docs-section-title.component
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 also to avoid using for
loop in the code, instead of it you can use .forEach
, or find / filter
.
d04e391
to
15e63d1
Compare
15e63d1
to
ddd8aee
Compare
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.
The leftovers are in 26 examples. Remove it and we can approve this PR :)
private fragment: any; | ||
@ViewChildren(DocsSectionTitleComponent, { read: ElementRef }) myList: QueryList<ElementRef>; | ||
|
||
constructor(private route: ActivatedRoute) {} |
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.
Hi @stefanoScalzo, please remove these leftovers.
c0867b4
to
56a5a58
Compare
Please provide a link to the associated issue.
fixes #1182