-
Notifications
You must be signed in to change notification settings - Fork 7
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
Radio/checkbox: Include description in label, set correct label styling #1031
Radio/checkbox: Include description in label, set correct label styling #1031
Conversation
Sigrid maintainability feedback✅ You wrote maintainable code and achieved your objective of 3.5 stars Show detailsSigrid compared your code against the baseline of 2025-02-17. 👍 What went well?
👎 What could be better?
📚 Remaining technical debt
View this system in Sigrid to explore your technical debt ⭐️ Sigrid ratings
💬 Did you find this feedback helpful?We would like to know your thoughts to make Sigrid better. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #1031 +/- ##
==========================================
+ Coverage 90.19% 90.26% +0.07%
==========================================
Files 249 249
Lines 13090 13181 +91
Branches 1338 1341 +3
==========================================
+ Hits 11806 11898 +92
+ Misses 1190 1187 -3
- Partials 94 96 +2 ☔ View full report in Codecov by Sentry. |
Closes #1002
This change is for all radio buttons and checkboxes in the application, for example: