-
Notifications
You must be signed in to change notification settings - Fork 813
feat: VSCode Extension for Gitingest Integration #242
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.
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a VS Code extension link and icon to the site navbar and updates the README to advertise the new VS Code extension.
- Added a VS Code Extension button (with inline SVG) to the navbar component
- Extended README link list and features section to include the VS Code extension
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
File | Description |
---|---|
src/server/templates/components/navbar.jinja | Inserted a new VS Code Extension link with inline SVG icon |
README.md | Appended VS Code Extension to links and feature list |
Comments suppressed due to low confidence (2)
README.md:16
- [nitpick] Use consistent product naming—change
VS-Code Extension
toVS Code Extension
to match other references.
[gitingest.com](https://gitingest.com) · [Chrome Extension](https://chromewebstore.google.com/detail/adfjahbijlkjfoicpjkhjicpjpjfaood) · [Firefox Add-on](https://addons.mozilla.org/firefox/addon/gitingest) · [VS-Code Extension](https://marketplace.visualstudio.com/items?itemName=iamshreydxv.gitingest)
README.md:29
- The nested bullets under "VS Code Extension" are not indented properly; adjust markdown indentation (e.g., two spaces before sub-items) for correct rendering.
- Fully Secure 🔐
<a href="https://marketplace.visualstudio.com/items?itemName=iamshreydxv.gitingest" | ||
target="_blank" | ||
rel="noopener noreferrer" | ||
class="text-gray-900 hover:-translate-y-0.5 transition-transform flex items-center gap-1.5"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add an aria-label (e.g., aria-label="VS Code Extension"
) to the anchor tag so screen readers clearly announce the link purpose.
class="text-gray-900 hover:-translate-y-0.5 transition-transform flex items-center gap-1.5"> | |
class="text-gray-900 hover:-translate-y-0.5 transition-transform flex items-center gap-1.5" | |
aria-label="VS Code Extension"> |
Copilot uses AI. Check for mistakes.
} | ||
</style> | ||
</defs> | ||
<title>1</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use a descriptive <title>
inside the SVG (e.g., VS Code Extension Icon
) instead of a generic "1" for better screen reader support.
<title>1</title> | |
<title>Navbar Icon</title> |
Copilot uses AI. Check for mistakes.
rel="noopener noreferrer" | ||
class="text-gray-900 hover:-translate-y-0.5 transition-transform flex items-center gap-1.5"> | ||
<div class="flex items-center"> | ||
<svg xmlns="http://www.w3.org/2000/svg" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The inline SVG is very long and duplicates markup; consider extracting it into a separate partial or using a shared icon component to improve readability and reuse.
Copilot uses AI. Check for mistakes.
We're discussing with @ShreyPurohit in order to properly migrate his extension into the up and coming github org for gitingest related projects In the meantime I'll make this a Draft |
Related Issue: #146
VSCode Extension: Gitingest Extension
Underlying GitHub Repo: gitingest-vsextension
Changes:
README.md
src/server/templates/components/navbar.jinja
Basic Usage
Setup Guide