codex - 💡(How to fix) Fix Full-line red/green backgrounds for diff reviewer (or make configurable) in Codex VS Code extension [1 comments, 2 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
openai/codex#18911Fetched 2026-04-22 07:50:51
View on GitHub
Comments
1
Participants
2
Timeline
4
Reactions
0
Timeline (top)
labeled ×3commented ×1

The current diff reviewer in the Codex VS Code extension only highlights the line numbers/gutter with red or green for changes, rather than using full-line red/green background fills. This makes it noticeably harder to scan diffs compared to the traditional stronger background style that most diff tools and previous versions used.

Screenshot for reference:

<img alt="Codex reviewer diff highlighting, only gutter colored" width="1091" src="https://github.com/user-attachments/assets/733a7522-fdbc-4bb7-985b-51f6f71998b1" />

Root Cause

Full-line highlights make it significantly easier to spot changes, especially in large or visually dense diffs. Many users are accustomed to, and expect, the more strongly-highlighted red/green lines as seen in traditional diff tools and GitHub's old PR UI.

If performance or accessibility triggered this change, please consider at least making the behavior configurable.

Thanks for considering this request!

RAW_BUFFERClick to expand / collapse

Summary

The current diff reviewer in the Codex VS Code extension only highlights the line numbers/gutter with red or green for changes, rather than using full-line red/green background fills. This makes it noticeably harder to scan diffs compared to the traditional stronger background style that most diff tools and previous versions used.

Screenshot for reference:

<img alt="Codex reviewer diff highlighting, only gutter colored" width="1091" src="https://github.com/user-attachments/assets/733a7522-fdbc-4bb7-985b-51f6f71998b1" />

Request

  • Option 1: Restore the previous behavior where the entire line for insertions and deletions is highlighted in green/red, not just the line-number gutter.
  • Option 2: Make this behavior a configuration option, so users can choose between full line backgrounds and the current subtle accent-on-numbers style.

The current approach is much less readable for large diffs or for users with color-accommodation needs.

Why this matters

Full-line highlights make it significantly easier to spot changes, especially in large or visually dense diffs. Many users are accustomed to, and expect, the more strongly-highlighted red/green lines as seen in traditional diff tools and GitHub's old PR UI.

If performance or accessibility triggered this change, please consider at least making the behavior configurable.

Thanks for considering this request!

extent analysis

TL;DR

The most likely fix is to make the diff highlighting behavior configurable, allowing users to choose between full-line backgrounds and the current subtle accent-on-numbers style.

Guidance

  • Consider adding a configuration option to the Codex VS Code extension to allow users to select their preferred diff highlighting style.
  • Review the current implementation of the diff reviewer to determine the feasibility of restoring the previous behavior of highlighting the entire line for insertions and deletions.
  • Evaluate the potential performance and accessibility implications of changing the diff highlighting behavior.
  • Gather feedback from users to determine the most popular diff highlighting style and prioritize development efforts accordingly.

Example

No code snippet is provided as the issue does not contain sufficient technical details to create a concrete example.

Notes

The issue lacks information about the technical implementation of the diff reviewer and the reasons behind the change in highlighting behavior. Therefore, the guidance provided is focused on exploring possible solutions and gathering more information.

Recommendation

Apply workaround: Make the behavior a configuration option, allowing users to choose between full-line backgrounds and the current subtle accent-on-numbers style, as this approach provides the most flexibility for users with different preferences and needs.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING