Enhances the reviewer role list with additional, reviewer specific detail.#4018
Conversation
…27 (when middleware caches settings fetch)
joemull
left a comment
joemull
left a comment
There was a problem hiding this comment.
This is great--I can imagine how useful it will be!
A couple small changes that might be nice:
- Put a default value, like an empty string, in the template on the cells for interests, number of assignments, last completed reviews, and average score, so they don't show up as "None"
- Adjust the widths of the first three columns so they don't take up so much space, or maybe so that they adjust automatically as needed?
- Clicking on the Edit User button and then saving the user results in being sent back to a different page, the Enrolled Users list. Is that by design? Would it be possible to return users back to the same page? Or is it not worth doing since we need to rethink how these user lists relate do one another?
I think the issue here is that the email address won’t break because it has no spaces, it makes the email column wide. |
It doesn't have to do with the |
Ah yeah, for Fname and Lname this makes a difference, for email not so much. |
|
I've updated this PR with changes. The |
3 participants
Closes #3039