fix: "Directory" heading uses accent color on all themes#454
Conversation
…theme Agent-Logs-Url: https://github.com/NextCommunity/NextCommunity.github.io/sessions/5c02d44f-df5c-4f3b-8b91-9f456a405396 Co-authored-by: jbampton <418747+jbampton@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Fixes a theme-specific styling issue on profile pages where the “Directory” heading color depended on --text-main (white in dark mode) and only appeared correct on hover.
Changes:
- Update the “Directory”
<h1>to usetext-accentinstead oftext-[var(--text-main)]plus a hover override. - Remove the
group-hover/back:text-accentclass from the “Directory” heading since the accent color is now applied consistently.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/gemini review |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the styling of the 'Directory' heading in the header-details template. The heading color is now set to 'text-accent' by default, and the hover-specific color transition has been removed to simplify the visual state. I have no feedback to provide as there are no review comments to evaluate.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@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 1 comment.
💡 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 simplifies the styling of the 'Directory' heading in the header details template by setting a constant accent color and removing hover-specific transitions. Feedback suggests re-adding a transition class, specifically transition-colors, to ensure smooth visual transitions during theme switches and maintain consistency with other headings.
Agent-Logs-Url: https://github.com/NextCommunity/NextCommunity.github.io/sessions/1d382a16-63f3-4334-8aed-690b06dd8f72 Co-authored-by: jbampton <418747+jbampton@users.noreply.github.com>
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request updates the styling of the 'Directory' header link in src/_includes/header-details.njk, changing the text color to a constant accent color and refining the transition property. A review comment suggests adding a specific transition duration to ensure the header's color transition is synchronized with the rest of the page during theme switches.
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.
Agent-Logs-Url: https://github.com/NextCommunity/NextCommunity.github.io/sessions/04479227-6935-4fbf-9f9d-d06850865898 Co-authored-by: jbampton <418747+jbampton@users.noreply.github.com>
transition-colorsto Directory<h1>(previous session)duration-300to Directory<h1>to match body transition duration per Gemini review