-
Notifications
You must be signed in to change notification settings - Fork 105
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
A readable example #147
Comments
@sheikalthaf Thanks, that's much better, but still hard to grasp for a beginner, the best would just be the config object, followed by the array of images, and the template binding. Just those core concepts would be perfect, adding change detection and everything else can come in as an advanced part of documentation |
I think an example with only the simple example as well, leave out routing etc, just an isolated simple carousel. Thank you for all the hard work, I can't even begin to imagine how much work you've put in to this, just to help others, I'm deeply grateful. |
Something as simple as this would be very welcoming https://stackblitz.com/edit/ngu-carousel-demo-n3rfvf Then later in the docs you can add more complicated examples |
This issue has been automatically marked as stale because it has not had recent activity for 6 months. It will be closed if no further activity occurs. Thank you for your contributions. |
Closing this issue due to no activity for 6 months. |
Looking into the usage example hits you with a barely formatted wall of text for the carousel template, which is very hard to read, especially when you at first jump between it and the component JS code a lot to see what goes where.
Could you please extract the template into a separate code block and apply formatting and language highlight to it?
The text was updated successfully, but these errors were encountered: