Skip to content

Fix run config deltas table overflow and truncation#132

Merged
tomvothecoder merged 2 commits into
mainfrom
codex/fix-run-config-deltas-layout
Mar 10, 2026
Merged

Fix run config deltas table overflow and truncation#132
tomvothecoder merged 2 commits into
mainfrom
codex/fix-run-config-deltas-layout

Conversation

@Codex
Copy link
Copy Markdown
Contributor

@Codex Codex AI commented Mar 9, 2026

Description

Long run config delta values overflowed their card and forced horizontal scroll; now cells truncate with ellipses and show full content on hover tooltips while keeping layout stable.

  • Table layout: Switched config deltas table to fixed layout with bounded column widths and container border to prevent overflow.
  • Value rendering: Added DiffCell helper that truncates keys/values and surfaces full text via title tooltip.
const DiffCell = ({ value }: { value: unknown }) => {
  const text = value == null ? '—' : String(value);
  return <div className="truncate" title={text}>{text}</div>;
};

Checklist

  • Code follows project style guidelines
  • Self-reviewed code
  • No new warnings
  • Tests added or updated (if needed)
  • All tests pass (locally and CI/CD)
  • Documentation/comments updated (if needed)
  • Breaking change noted (if applicable)

Deployment Notes (if any)

Original prompt

This section details on the original issue you should resolve

<issue_title>[Bug]: Fix run_config_deltas Layout Overflow</issue_title>
<issue_description>## Summary

Fix layout instability and horizontal overflow in the run_config_deltas display.

UI bugfix only.

Motivation

Long configuration values cause:

  • Horizontal scrolling
  • Card overflow
  • Layout breakage

This impacts usability and readability.

Screenshot:

Image

Scope

Stabilize rendering of configuration deltas:

Preferred:

  • Truncate long values with ellipsis
  • Show full value in tooltip

Alternative:

  • Proper word wrapping without layout breakage

Requirements:

  • No horizontal overflow
  • Stable rendering for long keys and values
  • No regression to existing diff logic

Non-Goals

  • No changes to diff computation
  • No backend changes
  • No schema changes
  • No visual redesign beyond overflow fix</issue_description>

Comments on the Issue (you are @codex[agent] in this section)

@Codex Codex AI changed the title [WIP] Fix layout overflow in run_config_deltas display Fix run_config_deltas overflow handling Mar 9, 2026
@Codex Codex AI changed the title Fix run_config_deltas overflow handling Fix run config deltas table overflow and truncation Mar 9, 2026
@tomvothecoder
Copy link
Copy Markdown
Collaborator

@codex[agent] In the Configuration Differences table, for truncated fields show the rest of the value with mouse over.

@chatgpt-codex-connector
Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

@Codex
Copy link
Copy Markdown
Contributor Author

Codex AI commented Mar 10, 2026

@codex[agent] In the Configuration Differences table, for truncated fields show the rest of the value with mouse over.

Added hover tooltips via the new DiffCell helper so truncated field/canonical/current values show full text on mouseover (commit de24b5f).

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Improves the readability of the “Configuration Differences” section in the simulation details view by making diff values consistently truncated with hover-tooltips and updating the table styling.

Changes:

  • Added a DiffCell helper to render potentially long diff values with truncation and a title tooltip.
  • Updated the configuration diff table markup/styles (bordered container, fixed table layout, muted header styling).

@tomvothecoder tomvothecoder merged commit 732da7e into main Mar 10, 2026
5 checks passed
@tomvothecoder tomvothecoder deleted the codex/fix-run-config-deltas-layout branch March 24, 2026 17:19
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.

[Bug]: Fix run_config_deltas Layout Overflow

3 participants