Skip to content

Fix TS1117 duplicate background property in FineTune JobCard#5

Merged
Jacobcdsmith merged 1 commit into
mainfrom
copilot/fix-duplicate-property-error
Apr 11, 2026
Merged

Fix TS1117 duplicate background property in FineTune JobCard#5
Jacobcdsmith merged 1 commit into
mainfrom
copilot/fix-duplicate-property-error

Conversation

Copilot AI commented Apr 11, 2026

Copy link
Copy Markdown
Contributor

The JobCard component's inline style object contained two background declarations in the same object literal, causing TypeScript error TS1117: An object literal cannot have multiple properties with the same name.

Change

Removed the redundant static background: "var(--surface)" property, retaining only the conditional one that correctly handles the selected state:

// Before (broken)
<div onClick={onSelect} style={{
  background: "var(--surface)",   // ← duplicate (first)
  border: "1px solid var(--border)",
  ...
  background: selected ? "rgba(88,166,212,0.05)" : "var(--surface)",  // ← TS1117 here
}}>

// After (fixed)
<div onClick={onSelect} style={{
  background: selected ? "rgba(88,166,212,0.05)" : "var(--surface)",  // ← single property
  border: "1px solid var(--border)",
  ...
  borderRadius: "var(--radius)"
}}>

The conditional already covers the default ("var(--surface)") case, so no visual behavior changes.

Copilot AI changed the title [WIP] Fix duplicate property error in FineTune component Fix TS1117 duplicate background property in FineTune JobCard Apr 11, 2026
Copilot AI requested a review from Jacobcdsmith April 11, 2026 13:28
@Jacobcdsmith Jacobcdsmith marked this pull request as ready for review April 11, 2026 13:29
Copilot AI review requested due to automatic review settings April 11, 2026 13:29
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Jacobcdsmith Jacobcdsmith merged commit 818320b into main Apr 11, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants