Skip to content

Lock Hacklytics Notify me to the portal, sharp cyan CTA - #368

Open
aamoghS wants to merge 1 commit into
mainfrom
cursor/hacklytics-notify-cta-5140
Open

Lock Hacklytics Notify me to the portal, sharp cyan CTA#368
aamoghS wants to merge 1 commit into
mainfrom
cursor/hacklytics-notify-cta-5140

Conversation

@aamoghS

@aamoghS aamoghS commented Aug 28, 2026

Copy link
Copy Markdown
Member

What

Notify me on the Hacklytics 2027 marketing site is a portal handoff, not a form.

Every Notify me control (Navbar desktop + mobile, hero CTA, JSON-LD offer) already used INTEREST_URL from sites/hacklytics2027/lib/links.ts. That destination is unchanged:

https://datasciencegt.org/login?callbackUrl=%2Fhacklytics

The portal path is /hacklytics (sites/mainweb/app/(portal)/hacklytics/page.tsx). Login already honors callbackUrl=/hacklytics. No Typeform, no mailto, no /notify route, no new form page.

Pixel lock

  • Shared .pixel-btn is a sharp cyan rectangle with black/ink label.
  • Removed the 28px glow soup. Hard pixel edge only; bloom stays on sprites.
  • Hero caption: “Opens the DS@GT portal. We don't collect emails here.”

Out of scope

Navbar structure, Footer, sections, and PixelGarden were left alone. Hero was touched only for the Notify me control + handoff copy.

Open in Web Open in Cursor 

Every Notify me still opens INTEREST_URL (login → /hacklytics). The shared
pixel-btn drops the 28px glow so the control reads as a cyan rectangle with
black type. Hero copy states the site has no form.

Co-authored-by: Aamogh <aamoghS@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Visit the preview URL for this PR (updated for commit 3f2a0b6):

https://hacklytics2027--pr-368-itnknci7.web.app

(expires Fri, 04 Sep 2026 04:22:23 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c48ba34db61581e25fe2978355160b5eefe0e83f

@aamoghS
aamoghS marked this pull request as ready for review August 28, 2026 04:24
@greptile-apps

greptile-apps Bot commented Aug 28, 2026

Copy link
Copy Markdown

Greptile Summary

The PR clarifies that Hacklytics “Notify me” controls hand users off to the existing DS@GT portal and updates the shared CTA to a sharp cyan pixel style.

  • Adds shared handoff copy for visible and accessible descriptions.
  • Updates the hero and navbar Notify me controls without changing their destination.
  • Removes CTA bloom styling and adds a visible keyboard-focus treatment.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The existing portal destination remains unchanged, and the styling, visible caption, and accessible-label updates preserve the CTA navigation behavior.

Important Files Changed

Filename Overview
sites/hacklytics2027/app/globals.css Restyles shared pixel buttons with hard edges, reduced hover brightness, pressed-state shadows, and a focus-visible outline.
sites/hacklytics2027/app/page.tsx Adds visible portal-handoff guidance to the hero CTA and associates it through aria-describedby.
sites/hacklytics2027/components/Navbar.tsx Adds portal-handoff context to desktop and mobile Notify me links through accessible labels.
sites/hacklytics2027/lib/links.ts Exports shared handoff copy while preserving the existing encoded portal login URL.

Reviews (1): Last reviewed commit: "Lock Hacklytics Notify me to the portal ..." | Re-trigger Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants