Feat: Enrollment Update Time Tooltip #1315
Conversation
vicksey
changed the title
| <Help fontSize="small" /> | ||
| <Box display="flex" alignItems="center"> | ||
| {props.label} | ||
| {!isMobile && props.formattedTime && ( |
There was a problem hiding this comment.
How come we hide on mobile? iirc there was a discussion about how tricky it was to make responsive was it apart of that?
There was a problem hiding this comment.
I think because it's a hover tooltip and such a small feature that's really only used when someone's actively enrolling that it can just be waived for mobile. Do you think its needed for mobile?
There was a problem hiding this comment.
That's true. I think as a rule of thumb the feature should be supported in some capacity. I think the added panel needs some redesign maybe it's something to loop ui/ux in with.
There was a problem hiding this comment.
Yea, but I think that's something outside of the scope of this feature. I can easily include it if you want though, itll just be a tap to see the tooltip then - which might get finicky (like accidentally tapping it).
apps/antalmanac/src/components/RightPane/SectionTable/EnrollmentColumnHeader.tsx
Outdated
Show resolved
Hide resolved
|
Sorry for neglecting this. I wanted to loop in UI/UX because I want to do a redesign of the Added panel so we'll keep this in account |
Summary
Added time icon tooltip that displays the last time the enrollment status of a course was updated (last websoc call).
Moved the explanation of NOR (previously tooltip on enrollment header) to be a tooltip on the actual NOR text.
Test Plan
Issues
Closes #933