Skip to content

fix(webapp): use monospace font for debugging output textboxes#1368

Open
natoscott wants to merge 1 commit into
ai-dynamo:mainfrom
natoscott:fix/webapp-debug-monospace
Open

fix(webapp): use monospace font for debugging output textboxes#1368
natoscott wants to merge 1 commit into
ai-dynamo:mainfrom
natoscott:fix/webapp-debug-monospace

Conversation

@natoscott

@natoscott natoscott commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Overview:

The webapp debugging textboxes display CLI output (PrettyTable tables, Pareto frontier charts) that assumes a fixed-width font. Without monospace styling, columns are misaligned and ASCII charts are unreadable.

Details:

  • Added .debug-output textarea { font-family: monospace !important; } CSS rule to src/aiconfigurator/webapp/main.py
  • Applied elem_classes=["debug-output"] to all 5 existing debugging textboxes across the webapp tabs (static, agg, agg_pareto, disagg_pareto, disagg_pd_ratio)

Where should the reviewer start?

  • src/aiconfigurator/webapp/main.py — the CSS rule (4 lines)
  • Any of the 5 tab component files — identical one-line change each

Related Issues:

  • None

Signed-off-by: Nathan Scott <nathans@redhat.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@natoscott
natoscott requested review from a team as code owners July 15, 2026 08:01
@copy-pr-bot

copy-pr-bot Bot commented Jul 15, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the fix label Jul 15, 2026
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

Changes

Debug output styling

Layer / File(s) Summary
Tag debugging textboxes
src/aiconfigurator/webapp/components/*_tab.py
Debugging textboxes across five tabs now use the shared debug-output Gradio element class.
Apply monospace styling
src/aiconfigurator/webapp/main.py
Application CSS now applies a monospace font to textareas with the debug-output class.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

Debug boxes donned a class today,
In monospace they type away.
Five tabs align, the styles unite,
Logs now march in neat delight.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title accurately summarizes the main change: applying monospace styling to debugging output textboxes.
Description check ✅ Passed The description follows the template and covers overview, details, reviewer starting points, and related issues.

Comment @coderabbitai help to get the list of available commands.

@Arsene12358 Arsene12358 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verified coverage is complete: grep -rn 'label="Debugging"' src/aiconfigurator/webapp/ on main returns exactly 5 textboxes and this PR tags all 5 with elem_classes=["debug-output"]. The .debug-output textarea selector is scoped and can't leak to other components, and the !important is justified to override Gradio's theme-scoped textarea font. LGTM.

Heads-up for rebase order: #1373 relocates the Blocks(css=...) block this rule lands in, and #1369 independently adds this identical CSS rule + convention for its new tab — whichever of the three merges later needs a small rebase rather than a double-add of the rule.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants