Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(ui): show sync variant on connections page #3588

Merged
merged 2 commits into from
Feb 27, 2025

Conversation

TBonnin
Copy link
Collaborator

@TBonnin TBonnin commented Feb 26, 2025

https://linear.app/nango/issue/NAN-2726/[design]-sync-variant-feature-quick-one

Show sync variant in a grey bubble next to the sync name
Only show variant bubble if there is more than the base variant
Long variant names are truncated. Full name can be seen in a tooltip

Screenshot 2025-02-26 at 15 13 38

Copy link

linear bot commented Feb 26, 2025

@TBonnin TBonnin requested a review from bodinsamuel February 26, 2025 20:20
<div className="w-36 max-w-3xl truncate">{sync.name}</div>
<div className="w-36 max-w-3xl truncate">
{sync.name}
{showSyncVariant && ' '}
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can remove that use flex wrapper

<div className="flex gap-2">{...}</div>

@TBonnin TBonnin requested a review from bodinsamuel February 27, 2025 15:17
https://linear.app/nango/issue/NAN-2726/[design]-sync-variant-feature-quick-one
Show sync variant in a grey bubble next to the sync name
Only show variant bubble if there is more than the base variant
Long variant name are truncated
Full name can be seen in a tooltip
@TBonnin TBonnin force-pushed the tbonnin/nan-2418/sync-variant-9 branch from 90a7328 to bdc011e Compare February 27, 2025 18:54
@TBonnin TBonnin enabled auto-merge (squash) February 27, 2025 18:54
@TBonnin TBonnin merged commit 867a63d into master Feb 27, 2025
16 checks passed
@TBonnin TBonnin deleted the tbonnin/nan-2418/sync-variant-9 branch February 27, 2025 18:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants