Skip to content

Let a long auth type truncate on the connector card - #1315

Merged
helpcodeai merged 2 commits into
HelpCode-ai:mainfrom
sapatmohit:fix/connector-card-auth-truncate
Oct 28, 2026
Merged

helpcodeai merged 2 commits into
HelpCode-ai:mainfrom
sapatmohit:fix/connector-card-auth-truncate

Conversation

@sapatmohit

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #1303, as asked there: on a connector card, a long auth type next to a setup or sign-in pill pushed the pills past the card's right edge. The Auth value already had truncate, but its flex parents had no min-w-0, so it could never shrink.

Changes

  • app/connectors/page.tsx: min-w-0 on the Tools/Auth group and on the Auth column, so a long auth type truncates with an ellipsis and the pills stay on the card.

English changes too: "Connection String" and "OAuth 2.0" now truncate ("Connection …", "OAuth …") when a card also shows two pills, because the row was only about 13px from overflowing. Cards with one pill, or a short auth type, look the same as before.

Before After
English
German
Italian

Type

  • Bug fix

Testing

No new test: the existing i18n tests for /connectors already render these two cards, and still pass, since the truncated text is the same text node.

Directory Command Result
packages/frontend npm run lint 0 errors, 49 warnings (same as main)
packages/frontend npx tsc --noEmit -p tsconfig.json pass
packages/frontend npx playwright test tests/e2e/i18n.spec.ts tests/e2e/connector-export-import.spec.ts 33 passed
  • Added/updated tests
  • Tested manually. Checked in the browser with mocked API data (the screenshots above).

Checklist

  • My code follows the existing code style
  • I have tested my changes locally
  • All existing tests pass
  • This PR has a descriptive title

Related Issues

Follow-up to #1303 (part of #1271)

Without min-w-0 on the Tools/Auth column, a long auth type next to a
setup or sign-in pill pushed the pills past the card edge.

@helpcodeai helpcodeai 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.

Thanks @sapatmohit. Checked: min-w-0 on the flex parents is the right fix for the truncate, CSS-only change, CI green. Merging.

@helpcodeai
helpcodeai enabled auto-merge (squash) October 28, 2026 17:27
@helpcodeai
helpcodeai merged commit 85a5d72 into HelpCode-ai:main Oct 28, 2026
14 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants