fix(landing): fix image rendering and navbar blog/docs navigation#3785
fix(landing): fix image rendering and navbar blog/docs navigation#3785waleedlatif1 merged 2 commits intostagingfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Updates the desktop navbar so top-level items with dropdowns ( Written by Cursor Bugbot for commit 98f0083. Configure here. |
Greptile SummaryThis PR fixes two production issues on the landing/home pages: (1) 400 errors from Confidence Score: 5/5Safe to merge — targeted bug fixes with no functional regressions; the only trade-off (unoptimised images) is intentional and well-understood. Both fixes are simple, well-scoped, and address real production issues. The navbar change correctly threads the needle between navigable links and hover dropdowns. The No files require special attention; Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User hovers over navbar item] --> B{Has dropdown?}
B -- No --> C[Highlight item only]
B -- Yes --> D[Open hover dropdown panel]
D --> E{Item is external?}
E -- "Yes 'Docs'" --> F["a target=_blank href: https://docs.sim.ai"]
E -- "No 'Blog'" --> G["Link href: /blog"]
F --> H[User clicks label → opens docs in new tab]
G --> I[User clicks label → client-side nav to /blog]
D --> J[User hovers into dropdown]
J --> K[Click dropdown item → navigate to specific page]
D --> L[Mouse leaves li boundary]
L --> M[scheduleClose: 100ms delay → close dropdown]
Reviews (2): Last reviewed commit: "fix(navbar): remove disclosure aria attr..." | Re-trigger Greptile |
apps/sim/app/(landing)/components/footer/components/compliance-badges.tsx
Show resolved
Hide resolved
|
@greptile |
|
@cursor review |
Summary
unoptimizedto all local PNG/JPG<Image>components across landing and home pages to fix 400 errors from/_next/imagein production/blogand "Docs" navigates to docs (previously only the dropdown items were clickable)Type of Change
Testing
Tested manually
Checklist