-
Notifications
You must be signed in to change notification settings - Fork 833
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
Added examples for progress control #1809
base: develop
Are you sure you want to change the base?
Conversation
FabienLelaquais
commented
Sep 19, 2024
- removed 'color' property from progress control
- added CSS class names for value and title in the control
- minor refman issues
+ removed 'color' property from progress control + added CSS class names for value and title in the control + minor refman issues
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
---|---|---|---|
🟢 | Statements | 87.65% (+0.01% 🔼) |
3193/3643 |
🟡 | Branches | 68.9% (-0.12% 🔻) |
2173/3154 |
🟢 | Functions | 82.67% (+0.02% 🔼) |
582/704 |
🟢 | Lines | 88.22% (+0.01% 🔼) |
2957/3352 |
Test suite run success
640 tests passing in 43 suites.
Report generated by 🧪jest coverage report action from d3b758b
("width", PropertyType.string_or_number), | ||
] | ||
) | ||
._set_propagate(), |
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.
don't think propagate
is needed here as there's no variable modification from the frontend
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.
Let's discuss this rapidly. I'm sure you're right.
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
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.
lgtm