Skip to content

fix: prevent inline code white-space: normal from applying inside table cells#706

Merged
IEvangelist merged 2 commits intomainfrom
copilot/fix-small-option-columns
Apr 13, 2026
Merged

fix: prevent inline code white-space: normal from applying inside table cells#706
IEvangelist merged 2 commits intomainfrom
copilot/fix-small-option-columns

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 13, 2026

Option name columns on configuration pages were too narrow because the inline-code wrapping rule (white-space: normal) was overriding the natural nowrap behaviour of code inside table cells.

Change

Add :not(td):not(th) to the parent exclusion list in the inline-code white-space rule in site.css:

/* Before */
:root:not(:has(.topics-sidebar[data-api-ref])) .sl-markdown-content :not(pre) > code:not(:where(.not-content *)) { … }

/* After */
:root:not(:has(.topics-sidebar[data-api-ref])) .sl-markdown-content :not(pre):not(td):not(th) > code:not(:where(.not-content *)) { … }

Code that is a direct child of a td or th now falls through to the browser's default behaviour (no wrapping), keeping option names on a single line.

Result

/dashboard/configuration/

Dashboard configuration table

/app-host/configuration/

AppHost configuration table

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • telemetry.astro.build
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/aspire.dev/aspire.dev/src/frontend/node_modules/.bin/../astro/bin/astro.mjs dev --host 0.0.0.0 --port 4321 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Fix small option columns on configuration pages fix: prevent inline code white-space: normal from applying inside table cells Apr 13, 2026
Copilot AI requested a review from IEvangelist April 13, 2026 12:53
@IEvangelist IEvangelist marked this pull request as ready for review April 13, 2026 12:56
Copilot AI review requested due to automatic review settings April 13, 2026 12:56
Copy link
Copy Markdown
Contributor

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

Adjusts the site-wide inline-code wrapping CSS so that inline code inside table cells is no longer forced to wrap, fixing overly-narrow “Option” columns on configuration pages.

Changes:

  • Excludes td/th parents from the inline-code white-space: normal rule to preserve table-cell inline code nowrap behavior.

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

Copy link
Copy Markdown
Contributor

@sebastienros sebastienros left a comment

Choose a reason for hiding this comment

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

Looks good overall; I did not find any high-confidence issues in this selector change. The carve-out matches the Markdown table shape this PR is fixing ( / ) and avoids broadening the override further than necessary.review

@IEvangelist IEvangelist merged commit 06b1581 into main Apr 13, 2026
9 checks passed
@IEvangelist IEvangelist deleted the copilot/fix-small-option-columns branch April 13, 2026 17:27
JamesNK pushed a commit that referenced this pull request Apr 16, 2026
…le cells (#706)

* Initial plan

* fix: exclude table cell code from white-space: normal rule

Agent-Logs-Url: https://github.com/microsoft/aspire.dev/sessions/b334da08-ed97-4d18-add1-b2df52531f8f

Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
JamesNK pushed a commit that referenced this pull request Apr 21, 2026
…le cells (#706)

* Initial plan

* fix: exclude table cell code from white-space: normal rule

Agent-Logs-Url: https://github.com/microsoft/aspire.dev/sessions/b334da08-ed97-4d18-add1-b2df52531f8f

Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
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.

Small option columns on configuration pages

4 participants