Conversation
Agent-Logs-Url: https://github.com/NextCommunity/NextCommunity.github.io/sessions/17998eeb-b35f-412e-85dc-ec4382e6ab9b Co-authored-by: jbampton <418747+jbampton@users.noreply.github.com>
|
/gemini review |
There was a problem hiding this comment.
Pull request overview
Adds a missing Nunjucks attribution button to the footer alongside existing framework/tool buttons, keeping the footer’s “Built with” section consistent with the site’s tech stack.
Changes:
- Added a new “🔧 Nunjucks” footer link between the existing 11ty and Tailwind links.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
|
/gemini review |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Code Review
This pull request adds a link to Nunjucks and includes rel="noopener" attributes to external links in the footer. Feedback suggests refactoring the links into a Nunjucks loop to improve maintainability, adding aria-label attributes for better accessibility, and using rel="noopener noreferrer" for enhanced security and privacy.
…el=noopener noreferrer Agent-Logs-Url: https://github.com/NextCommunity/NextCommunity.github.io/sessions/ec46b339-e1b6-4c1b-9bcb-608018487de6 Co-authored-by: jbampton <418747+jbampton@users.noreply.github.com>
|
/gemini review |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Code Review
This pull request refactors the footer links in src/_includes/footer-details.njk by replacing hardcoded anchor tags with a Nunjucks loop. This change introduces a new link for Nunjucks and improves accessibility and security by adding aria-label and rel="noopener noreferrer" attributes to the links. I have no feedback to provide.
forlooprel="noopener noreferrer"to all three linksaria-labelattributes for accessibility (replaces accessible name; not a duplication)