Added C Language by Akshit-MMQH · Pull Request #193 · DhanushNehru/CustomCodeEditor

Akshit-MMQH · 2025-10-03T11:18:02Z

[#191]

Checklist before requesting a review

[y] I have pull latest changes from main branch
[ y] I have tested the changes locally
[ y] I assure there is no similar/duplicate pull request regarding same issue

vercel · 2025-10-03T11:18:08Z

@Akshit-MMQH is attempting to deploy a commit to the Dhanush Nehru's projects Team on Vercel.

A member of the Team first needs to authorize it.

vercel · 2025-10-03T13:30:37Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
custom-code-editor	Error			Oct 8, 2025 2:39pm

DhanushNehru · 2025-10-03T13:32:23Z

@Akshit-MMQH deployment has failed, please check your code well

Copilot

Pull Request Overview

Adds C language support to the app by introducing a C logo component and extending the language constants. Also updates Firebase SDK version.

Adds C language entry to language constants (name, logo, default, hello world).
Introduces CLogo SVG component.
Bumps firebase dependency to ^11.10.0.

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.

File	Description
src/constants/constants.js	Registers C language with logo component and Hello World snippet.
src/components/images/CLogo.js	New SVG component for the C language logo.
package.json	Updates firebase to ^11.10.0.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-10-03T13:33:20Z

+      y="0px"
+      width={width}
+      height={height}
+      viewBox="0 0 48 48"


The SVG lacks an accessible name or an explicit indication that it is decorative. Either add role="img" with aria-label/title (e.g., aria-label="C programming language logo") or, if purely decorative, mark it as aria-hidden="true" to improve screen reader experience.

Suggested change

viewBox="0 0 48 48"

viewBox="0 0 48 48"

role="img"

aria-label="C programming language logo"

Copilot · 2025-10-03T13:33:21Z

+      <linearGradient
+        id="Ey3AfYdg0JtJGx7I73Eu7a_TpULddJc4gTh_gr1"
+        x1="5"
+        x2="43"
+        y1="24"
+        y2="24"
+        gradientUnits="userSpaceOnUse"
+      >
+        <stop offset=".002" stopColor="#427fdb"></stop>
+        <stop offset=".397" stopColor="#2668cb"></stop>
+        <stop offset=".763" stopColor="#1358bf"></stop>
+        <stop offset="1" stopColor="#0c52bb"></stop>
+      </linearGradient>
+      <path
+        fill="url(#Ey3AfYdg0JtJGx7I73Eu7a_TpULddJc4gTh_gr1)"
+        fillRule="evenodd"
+        d="M22.903,3.286c0.679-0.381,1.515-0.381,2.193,0	c3.355,1.883,13.451,7.551,16.807,9.434C42.582,13.1,43,13.804,43,14.566c0,3.766,0,15.101,0,18.867	c0,0.762-0.418,1.466-1.097,1.847c-3.355,1.883-13.451,7.551-16.807,9.434c-0.679,0.381-1.515,0.381-2.193,0	c-3.355-1.883-13.451-7.551-16.807-9.434C5.418,34.899,5,34.196,5,33.434c0-3.766,0-15.101,0-18.867	c0-0.762,0.418-1.466,1.097-1.847C9.451,10.837,19.549,5.169,22.903,3.286z"
+        clipRule="evenodd"
+      ></path>


The hard-coded SVG IDs (e.g., Ey3AfY...gr1) can collide if multiple CLogo instances render on the same page, breaking gradient references. Use React's useId to generate unique IDs per instance and reference them in the url(#...) fills to avoid DOM ID collisions.

Copilot · 2025-10-03T13:33:21Z

+    LOGO: <CLogo />,
+    HELLO_WORLD: `#include <stdio.h>
+int main() {
+    printf("Hello World");


Consider adding a newline to ensure the output flushes consistently across environments: printf("Hello World\n");.

Suggested change

printf("Hello World");

printf("Hello World\n");

Copilot · 2025-10-03T13:33:22Z

+}`,
+  },
+  {
+    ID: LANGUAGE_ID_FOR_C, // Add this object for C


[nitpick] Avoid inline comments that restate obvious code intent; they add noise. Recommend removing the trailing comment.

Suggested change

ID: LANGUAGE_ID_FOR_C, // Add this object for C

ID: LANGUAGE_ID_FOR_C,

Akshit-MMQH · 2025-10-04T13:19:23Z

@DhanushNehru now check!!

Copilot

Pull Request Overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 8 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Akshit-MMQH · 2025-10-05T09:51:37Z

@DhanushNehru please review

Copilot

Pull Request Overview

Copilot reviewed 2 out of 4 changed files in this pull request and generated 2 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-10-08T14:39:07Z

+}`,
+  },
+  {
+    ID: LANGUAGE_ID_FOR_C, // Add this object for C


The constant LANGUAGE_ID_FOR_C is referenced but not defined. This will cause a ReferenceError at runtime.

Copilot · 2025-10-08T14:39:08Z

+
+
+


[nitpick] Remove unnecessary blank lines at the end of the file to maintain consistent code formatting.

Suggested change

DhanushNehru · 2025-10-08T14:41:23Z

Please resolve the code issues and CI/CD pipeline issues @Akshit-MMQH

Akshit-MMQH added 2 commits October 3, 2025 16:43

Added C Language

e3c63cf

removed my firebase keys

db69eb3

DhanushNehru requested a review from Copilot October 3, 2025 13:30

vercel Bot had a problem deploying to Preview October 3, 2025 13:31 Failure

DhanushNehru assigned Akshit-MMQH Oct 3, 2025

DhanushNehru added enhancement New feature or request hacktoberfest hacktoberfest-accepted labels Oct 3, 2025

Copilot AI reviewed Oct 3, 2025

View reviewed changes

fixed

8554931

DhanushNehru requested a review from Copilot October 5, 2025 03:28

Copilot AI reviewed Oct 5, 2025

View reviewed changes

corrected the version of firebase to match the repo

3be91d6

DhanushNehru requested a review from Copilot October 8, 2025 14:38

Copilot AI reviewed Oct 8, 2025

View reviewed changes

vercel Bot had a problem deploying to Preview October 8, 2025 14:39 Failure

Akshit-MMQH added 2 commits October 9, 2025 16:20

Delete package-lock.json

aebea96

Delete package.json

5200b52

Akshit-MMQH closed this Oct 9, 2025

	ID: LANGUAGE_ID_FOR_C, // Add this object for C
	ID: LANGUAGE_ID_FOR_C,

Uh oh!

Conversation

Akshit-MMQH commented Oct 3, 2025

Checklist before requesting a review

Uh oh!

vercel Bot commented Oct 3, 2025

Uh oh!

vercel Bot commented Oct 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

DhanushNehru commented Oct 3, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

Akshit-MMQH commented Oct 4, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Akshit-MMQH commented Oct 5, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Oct 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 8, 2025

Choose a reason for hiding this comment

Uh oh!

DhanushNehru commented Oct 8, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

vercel Bot commented Oct 3, 2025 •

edited

Loading