style(Teacher Tools): Improve component summary layout#2303
Conversation
|
Coverage Impact Unable to calculate total coverage change because base branch coverage was not found. Modified Files with Diff Coverage (1)
🤖 Increase coverage with AI coding...🚦 See full report on Qlty Cloud » 🛟 Help
|
hirokiterashima
left a comment
There was a problem hiding this comment.
LGTM. 👍
I had two thoughts for a future refactoring task:
- component-summary.component.html is getting logic-heavy and harder to read. I'm not sure what the solution is, but just wanted to bring this up.
- ai-summary.component is only rendered when there is student work (if I recall correctly), so we shouldn't have to check for
hasStudentResponsesin the code and in the template?
|
🎉 This PR is included in version 5.225.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |

Changes
Before:

After:

Before:

After:

Before:

After:

Test