Skip to content

improvement(chat): give wsres resource links the clickable chip treatment#5719

Merged
waleedlatif1 merged 1 commit into
stagingfrom
chat-resource-chip-spacing
Jul 16, 2026
Merged

improvement(chat): give wsres resource links the clickable chip treatment#5719
waleedlatif1 merged 1 commit into
stagingfrom
chat-resource-chip-spacing

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • #wsres- resource links in assistant chat messages had no visible affordance at rest — plain text + icon, only the cursor hinted they were clickable
  • applied the canonical clickable-resource chip recipe from WorkspaceResourceDisplay (special-tags.tsx): rounded-[5px] bg-[var(--surface-5)] px-[5px] with hover-hover:bg-[var(--surface-6)] + transition-colors
  • follow-up to fix(chat): align resource mention icon spacing with mention chips #5718 — the same resource now renders identically through both markdown paths

Type of Change

  • Improvement

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@greptile-apps

greptile-apps Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR gives workspace-resource links in assistant chat messages the established clickable-chip appearance. The main changes are:

  • Adds a resting surface, rounded corners, and horizontal padding.
  • Adds hover feedback and a color transition.
  • Limits the new styling to recognized workspace-resource links.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The update only adds established local styling classes to the existing resource-link branch.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/chat-content/chat-content.tsx Adds local chip styling and hover feedback to recognized workspace-resource links without changing parsing, navigation, or click handling.

Reviews (1): Last reviewed commit: "improvement(chat): give wsres resource l..." | Re-trigger Greptile

@waleedlatif1
waleedlatif1 merged commit 01ffacc into staging Jul 16, 2026
16 of 18 checks passed
@waleedlatif1
waleedlatif1 deleted the chat-resource-chip-spacing branch July 16, 2026 22:49
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

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.

1 participant