feat(GradeByStep): Show teams on node#2264
Conversation
|
Coverage Impact Unable to calculate total coverage change because base branch coverage was not found. Modified Files with Diff Coverage (2)
🛟 Help
|
Removed period text from NavItemProgress and TeamsOnNode tooltips.
breity
left a comment
There was a problem hiding this comment.
Looks good. 👍
I updated styles and removed the period name from NavItemProgress and TeamsOnNode tooltips.
|
🎉 This PR is included in version 5.204.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |

Notes
Please style as you see fit. The person icon is not quite centered vertically. :p
I'm wondering if we need to show the "(Period 1)" and "(All Periods)" in the tooltip. It clutters up the tooltip text a bit and makes it hard to read. Would the teachers usually know which period they're currently looking at and can get this info by looking at the period select drop-down?
Changes
Test