Skip to content

fix(tarko): disable share button during agent execution#1288

Merged
ulivz merged 8 commits into
mainfrom
fix/share-button-disabled-state
Aug 28, 2025
Merged

fix(tarko): disable share button during agent execution#1288
ulivz merged 8 commits into
mainfrom
fix/share-button-disabled-state

Conversation

@ulivz
Copy link
Copy Markdown
Member

@ulivz ulivz commented Aug 28, 2025

Summary

Disable share button during agent execution instead of hiding it completely.

img_v3_02pj_044389f0-fe6f-4d6b-83c3-2533f8e845bg

Checklist

  • Added or updated necessary tests (Optional).
  • Updated documentation to align with changes (Optional).
  • Verified no breaking changes, or prepared solutions for any occurring breaking changes (Optional).
  • My change does not involve the above items.

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 28, 2025

Deploy Preview for agent-tars-docs ready!

Name Link
🔨 Latest commit c273e49
🔍 Latest deploy log https://app.netlify.com/projects/agent-tars-docs/deploys/68b029f31edb570008a458a1
😎 Deploy Preview https://deploy-preview-1288--agent-tars-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

ulivz added 7 commits August 28, 2025 17:48
Add Material-UI Tooltip to ShareButton component with consistent styling
matching MessageFooter tooltips. Provides clear feedback when button
is disabled during agent execution.
Remove HTML disabled attribute from share button to allow tooltip
to display on hover. Disabled state is now handled purely through
CSS styling and click prevention logic.
Add proper placement and margin settings to tooltip to prevent
it from appearing too close to screen edges. Set default placement
to bottom with adequate spacing.
Update tooltip text to clearly instruct users to wait for agent
execution completion before sharing, providing clearer guidance
than the previous generic message.
Update tooltip to be more concise while maintaining clarity:
'Share unavailable during agent execution. Please wait.'
Combines status explanation with clear action guidance.
Remove 'Please wait' from tooltip as it feels too formal.
Keep simple and clear: 'Share unavailable during agent execution'
which naturally implies waiting without being directive.
Change tooltip to provide both status explanation and action guidance.
New message: 'Share unavailable during agent execution. Please wait
for agent execution to complete'
@ulivz ulivz merged commit ba4509b into main Aug 28, 2025
10 checks passed
@ulivz ulivz deleted the fix/share-button-disabled-state branch August 28, 2025 13:49
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.

2 participants