Skip to content

feat(DAO-2183): use AlwaysEnabledButton for mobile tooltip on disabled actions#2099

Open
Freshenext wants to merge 2 commits into
mainfrom
DAO-2183
Open

feat(DAO-2183): use AlwaysEnabledButton for mobile tooltip on disabled actions#2099
Freshenext wants to merge 2 commits into
mainfrom
DAO-2183

Conversation

@Freshenext

Copy link
Copy Markdown
Collaborator

Summary

  • Replace disabled Button + Tooltip with AlwaysEnabledButton (wrapping ConditionalTooltip) for the BTC Vault deposit/withdraw actions
  • On mobile, tapping a blocked button now shows a tooltip explaining why (e.g., "Your address is not whitelisted for deposits"), instead of showing a greyed-out button with no explanation
  • CTA is guarded via onClick so the modal never opens when the action is blocked

@Freshenext Freshenext self-assigned this Apr 14, 2026
@Freshenext Freshenext requested a review from a team April 14, 2026 02:26
@github-actions

github-actions Bot commented Apr 14, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA edf4df0.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

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