Skip to content

TIG-207: implement info icon that shows important links #516

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.

Sign up for GitHub

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

Merged
angelina-ji merged 4 commits into from
Apr 1, 2026
Merged

TIG-207: implement info icon that shows important links #516

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2a9c69f
Added info button, decluttered popover with just two links
niyathikukkapalli Mar 25, 2026
79cf914
Update .env.production.example
niyathikukkapalli Mar 25, 2026
4a4a7c2
Updated Info Button, Fixed Broken Info, Changed Names
niyathikukkapalli Apr 1, 2026
ef6383b
Merge branch 'master' into info_box_button
angelina-ji Apr 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions frontend/src/components/Requirements.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@ export default function Requirements({ onChange, requirements, schedule }) {
reqLabel,
popoverInstance,
{
hideDelayMs: 0,
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot Apr 1, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Restore a nonzero hover hide delay for requirement popovers

Using hideDelayMs: 0 causes bindManualHoverPopover to run the hide check immediately on trigger mouseleave, so when the pointer crosses the trigger→popover gap and relatedTarget is null, the popover is hidden before users can reach interactive content. This regresses existing behavior for sub-requirement popovers (notably the Find Satisfying Courses button) and can also affect the new header-link popovers; a small delay is needed to make hover-to-popover transitions reliable.

Useful? React with 👍 / 👎.

onShow: (popoverEl) => {
if (!popoverEl) return;
const reqPath = reqLabel.getAttribute('reqpath');
Expand Down
7 changes: 7 additions & 0 deletions frontend/src/styles/Requirements.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,13 @@
margin-top: 5px;
}

/* Main-req popover only (paired classes in Requirements.jsx); resets scroll/height from .popoverContentContainer */
.popoverContentContainer.main-req-popover-wrap {
max-height: none;
margin-top: 0;
overflow: visible;
}

.fa-exclamation-circle {
color: #e74c3c;
}
Expand Down
Loading