Skip to content

ui: preview: Remove useless download button from dialog actions#1121

Closed
035966-L3 wants to merge 1 commit intohydro-dev:masterfrom
035966-L3:master
Closed

ui: preview: Remove useless download button from dialog actions#1121
035966-L3 wants to merge 1 commit intohydro-dev:masterfrom
035966-L3:master

Conversation

@035966-L3
Copy link

@035966-L3 035966-L3 commented Jan 24, 2026

503

Summary by CodeRabbit

  • Chores
    • Removed the download button from the preview dialog, leaving copy and confirm actions available to users.

✏️ Tip: You can customize this high-level summary in your review settings.

Removed the download button from the dialog actions.
@github-actions
Copy link
Contributor

github-actions bot commented Jan 24, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@coderabbitai
Copy link

coderabbitai bot commented Jan 24, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

The change removes the "Download" button from the dialog actions in the preview page component. Previously, the dialog provided three available user actions: copy, download, and ok. After this modification, only two actions remain available: copy and ok. This involves a single-line deletion in the preview component's configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@035966-L3
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

1 similar comment
@035966-L3
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@pandadtdyy pandadtdyy closed this Jan 24, 2026
@pandadtdyy
Copy link
Member

pandadtdyy commented Jan 24, 2026

If you don't know how to fix it, don't create a PR to just remove it. That's actually useless.

@035966-L3
Copy link
Author

(How? I confirmed the "Ok" button on the side did exactly what that one ought to do. That one's actually useless.)

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