-
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
Some improvements to Tables and OverviewPage #990
base: main
Are you sure you want to change the base?
Conversation
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 #990 +/- ##
==========================================
+ Coverage 89.85% 90.19% +0.33%
==========================================
Files 249 249
Lines 13073 13090 +17
Branches 1328 1338 +10
==========================================
+ Hits 11747 11806 +59
+ Misses 1232 1190 -42
Partials 94 94 ☔ View full report in Codecov by Sentry. |
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-12. 👍 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. |
…small alert text color
…to input on polling station choice page
e72177b
frontend/app/component/form/data_entry/polling_station_choice/PollingStationChoiceForm.tsx
Show resolved
Hide resolved
frontend/app/component/form/data_entry/polling_station_choice/PollingStationsList.tsx
Show resolved
Hide resolved
…ould not have a footer in the data entry flow due to political groups navigation scrolling issues
…sistent with the footer location in sibling pages
Description
Footer
toElectionReportPage
Table.LinkRow
last Cell has increased padding to make sure the padding around the Chevron is as designedTable
functions accept aclassName
Table.Column
toTable.HeaderCell
since it can also be used in a rowscope="row" | "col"
with a default set to"col"
toTable.HeaderCell
to allow rowth
usageTable.HeaderCell
can have no children, equal to the implementation ofTable.Cell
OverviewPage
instead of throwing a harsh error, separate designs for typist and administrator are implementedOverviewPage
Alert
,ProgressList
andStatusList
and small alert text colorTable.LinkRow
was black instead of gray