-
Notifications
You must be signed in to change notification settings - Fork 81
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
Add ability to pass in css class to spinner and bar #178
base: main
Are you sure you want to change the base?
Conversation
I realized that the background was overwriting some styles in my css class, namely background-color. I'm guessing that most people are wanting to style it based on a class, so the loader probably shouldn't set background in that case. Also, my [ngClass] code was old and wrong, updated to work. |
could you please update readme https://github.com/aitboudad/ngx-loading-bar#customize-ngx-loading-bar |
@@ -6,12 +6,18 @@ import { LoadingBarService } from './loading-bar.service'; | |||
template: ` | |||
<ng-container *ngIf="value != null ? value : (value$ | async) as progress"> | |||
<div *ngIf="includeSpinner" class="ngx-spinner"> |
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.
spinnerClass
should be set here
I'm not sure if we do really need such inputs 🤔, to use |
@aitboudad Thx for the hint. Styling the loading bar with
works great and I can use Angular Material Theming that way. It did not work without the |
I have to revoke my previous comment. With proper Angluar 12 theming it does not work. This PR would be really great. |
Closes #141
Not sure what you'd like to call the inputs, feel free to change them.