Skip to content

Replace hardcoded link color with CSS variable in Certifications.jsx#22

Merged
Yasar2019 merged 2 commits into
mainfrom
copilot/replace-hardcoded-link-color
Mar 18, 2026
Merged

Replace hardcoded link color with CSS variable in Certifications.jsx#22
Yasar2019 merged 2 commits into
mainfrom
copilot/replace-hardcoded-link-color

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 18, 2026

The CertificationCard anchor color was hardcoded to #00d084, making it inconsistent with the CSS variable theming system introduced for the rest of the app and unresponsive to theme changes.

Change

  • src/Components/Certifications.jsx — replace hardcoded color with var(--accent), already defined in GlobalStyles.js for both light and dark themes and used across all other components.
 a {
-  color: #00d084;
+  color: var(--accent);
   font-size: 1.2rem;
   text-decoration: none;
 }
Original prompt

This section details on the original issue you should resolve

<issue_title>Replace hardcoded #00d084 link color in Certifications.jsx with a CSS custom property</issue_title>
<issue_description>Source: Copilot review comment on PR #8src/Components/Certifications.jsx line 34 (suppressed low-confidence comment)

Problem

This link color is still hardcoded (#00d084). Since the PR introduces CSS variables for theming, this hardcoded color is inconsistent with the theming approach and won't adapt to theme changes.

Suggested Fix

Replace the hardcoded color with a semantic CSS variable (e.g. --link-accent or reuse --accent) so it remains consistent across themes:

a {
  color: var(--accent);
  font-size: 1.2rem;
  text-decoration: none;
}

Review thread: Suppressed comment from Copilot review on PR #8</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…fications.jsx

Co-authored-by: Yasar2019 <60102921+Yasar2019@users.noreply.github.com>
Copilot AI changed the title [WIP] Replace hardcoded #00d084 link color in Certifications.jsx Replace hardcoded link color with CSS variable in Certifications.jsx Mar 18, 2026
Copilot AI requested a review from Yasar2019 March 18, 2026 00:52
Copy link
Copy Markdown
Owner

@Yasar2019 Yasar2019 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good

@Yasar2019 Yasar2019 marked this pull request as ready for review March 18, 2026 00:55
@github-actions
Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://witty-mud-031b23a0f-22.eastus2.2.azurestaticapps.net

@Yasar2019 Yasar2019 merged commit 1197de2 into main Mar 18, 2026
2 checks passed
@Yasar2019 Yasar2019 deleted the copilot/replace-hardcoded-link-color branch March 18, 2026 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace hardcoded #00d084 link color in Certifications.jsx with a CSS custom property

2 participants