Skip to content

Fix snake_case decision point truncation#3235

Open
zackcl wants to merge 1 commit into
devfrom
feature/3234-consistent-decision-point-truncation
Open

Fix snake_case decision point truncation#3235
zackcl wants to merge 1 commit into
devfrom
feature/3234-consistent-decision-point-truncation

Conversation

@zackcl

@zackcl zackcl commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Resolves #3234

Changes

  • Added a reusable break-all utility class.
  • Applied character-level word breaking to decision point displays in the Payloads table.
  • Applied the same behavior to the Decision Points table for consistency.
  • Preserved the existing line clamp and overflow tooltip behavior.
  • Left other line-clamped content unchanged to avoid affecting natural-language and multi-line text.

@zackcl
zackcl requested review from bcb37 and Copilot July 15, 2026 12:24
@zackcl zackcl self-assigned this Jul 15, 2026

Copilot AI 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.

Pull request overview

Updates decision point text rendering so long snake_case/kebab-case targets can break at character boundaries before the single-line clamp applies, preventing premature ellipsis truncation in the experiment details tables.

Changes:

  • Added a global .break-all utility class (word-break: break-all).
  • Applied .break-all alongside existing line-clamp-1 usage for decision point displays in the Payloads and Decision Points tables.
  • Kept the existing overflow-tooltip + line-clamp behavior unchanged for other clamped fields.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
packages/frontend/projects/upgrade/src/styles.scss Adds a reusable .break-all utility to allow character-level breaking.
packages/frontend/projects/upgrade/src/app/features/dashboard/experiments/pages/experiment-details-page/experiment-details-page-content/experiment-payloads-section-card/experiment-payloads-table/experiment-payloads-table.component.html Applies break-all to the decision point cell text to avoid early truncation in the Payloads table.
packages/frontend/projects/upgrade/src/app/features/dashboard/experiments/pages/experiment-details-page/experiment-details-page-content/experiment-decision-points-section-card/experiment-decision-points-table/experiment-decision-points-table.component.html Applies the same break-all behavior for consistency in the Decision Points table.

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

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.

Ensure snake_case decision point targets truncate consistently

3 participants