chore(site): replace hero CTA with license link and scroll arrow#440
chore(site): replace hero CTA with license link and scroll arrow#440
Conversation
Replace "See How It Works" button with "License & Usage Restrictions" linking to the license section — more useful for first-time visitors. Add a subtle bounce arrow below the CTAs as scroll-down hint.
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request enhances the landing page experience by refining the primary call-to-action in the hero section to better serve first-time visitors seeking licensing information. It also improves user navigation and discoverability of content by adding a visual scroll hint and ensuring direct access to the license details via an anchor link. Highlights
Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughSummary by CodeRabbit
WalkthroughUpdated the hero section's primary CTA to link to "#license-usage" instead of "#how-it-works" and changed its label accordingly. Added a new scroll hint element with a downward chevron that links to the "how-it-works" section. Added an id attribute to the LICENSE & USAGE section container to enable direct anchor linking. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
📝 Coding Plan
Comment |
There was a problem hiding this comment.
Code Review
This pull request successfully replaces the hero call-to-action button to link to the license section and adds a new scroll-down arrow as a visual hint. The changes are implemented correctly by updating the link's href and text, adding an id to the target section, and including a new anchor link with an SVG icon. My feedback includes one suggestion to improve code maintainability by componentizing the new SVG icon, which aligns with best practices for Astro projects.
| <a href="#how-it-works" class="mt-12 inline-block text-gray-500 hover:text-gray-300 transition-colors animate-bounce" aria-label="Scroll down for more details"> | ||
| <svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="m6 9 6 6 6-6"/></svg> | ||
| </a> |
There was a problem hiding this comment.
For better maintainability and reusability, it's a good practice to extract SVG icons into their own Astro components. This keeps the markup cleaner and makes icons easier to manage, especially if this icon or others are used elsewhere on the site.
For example, you could create a component src/components/icons/ChevronDown.astro:
// src/components/icons/ChevronDown.astro
<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" {...Astro.props}>
<path d="m6 9 6 6 6-6"/>
</svg>And then use it in this file:
---
import ChevronDown from '../components/icons/ChevronDown.astro';
---
<a href="#how-it-works" class="mt-12 inline-block text-gray-500 hover:text-gray-300 transition-colors animate-bounce" aria-label="Scroll down for more details">
<ChevronDown />
</a>
Greptile SummaryThis PR makes a focused hero-section UX update: it swaps the secondary CTA from a
Confidence Score: 5/5
Important Files Changed
Last reviewed commit: fefbf41 |
🤖 I have created a release *beep* *boop* --- ## [0.2.4](v0.2.3...v0.2.4) (2026-03-15) ### Bug Fixes * attach cosign signatures and provenance bundle to release assets ([#438](#438)) ([f191a4d](f191a4d)) * create git tag explicitly for draft releases ([#432](#432)) ([1f5120e](1f5120e)) * docker healthcheck, CI optimization, and container hardening ([#436](#436)) ([4d32bca](4d32bca)) * ensure security headers on all HTTP responses ([#437](#437)) ([837f2fc](837f2fc)) * make install scripts usable immediately without terminal restart ([#433](#433)) ([b45533c](b45533c)) * migrate pids_limit to deploy.resources.limits.pids ([#439](#439)) ([66b94fd](66b94fd)) ### Refactoring * redesign release notes layout ([#434](#434)) ([239aaf7](239aaf7)) ### Maintenance * **site:** replace hero CTA with license link and scroll arrow ([#440](#440)) ([56af41c](56af41c)) * **web:** adopt @vue/tsconfig preset ([#435](#435)) ([7d4b214](7d4b214)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
🤖 I have created a release *beep* *boop* --- ## [0.2.4](v0.2.3...v0.2.4) (2026-03-15) ### Bug Fixes * attach cosign signatures and provenance bundle to release assets ([#438](#438)) ([f191a4d](f191a4d)) * create git tag explicitly for draft releases ([#432](#432)) ([1f5120e](1f5120e)) * docker healthcheck, CI optimization, and container hardening ([#436](#436)) ([4d32bca](4d32bca)) * ensure security headers on all HTTP responses ([#437](#437)) ([837f2fc](837f2fc)) * make install scripts usable immediately without terminal restart ([#433](#433)) ([b45533c](b45533c)) * migrate pids_limit to deploy.resources.limits.pids ([#439](#439)) ([66b94fd](66b94fd)) * use cosign --bundle flag for checksums signing ([#443](#443)) ([19735b9](19735b9)) ### Refactoring * redesign release notes layout ([#434](#434)) ([239aaf7](239aaf7)) ### Maintenance * **main:** release 0.2.4 ([#431](#431)) ([63b03c4](63b03c4)) * remove stale v0.2.4 changelog section from failed release ([#446](#446)) ([769de10](769de10)) * reset version to 0.2.3 for re-release ([#444](#444)) ([8579993](8579993)) * **site:** replace hero CTA with license link and scroll arrow ([#440](#440)) ([56af41c](56af41c)) * **web:** adopt @vue/tsconfig preset ([#435](#435)) ([7d4b214](7d4b214)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Summary
#license-usagesection — more useful for first-time visitors evaluating the projectid="license-usage"anchor to the License & Usage sectionTest plan
#how-it-works