Skip to content

fix(trading): fix post-trade survey UI styling#26815

Open
sherpaPSX wants to merge 2 commits intodevelopfrom
fix/25343-dex-post-trade-survey-ui
Open

fix(trading): fix post-trade survey UI styling#26815
sherpaPSX wants to merge 2 commits intodevelopfrom
fix/25343-dex-post-trade-survey-ui

Conversation

@sherpaPSX
Copy link
Copy Markdown
Contributor

@sherpaPSX sherpaPSX commented Apr 17, 2026

Description

  • Use H2 component for survey title instead of Text with inline typography style
  • Apply intent="brand" and size="large" to the survey CTA button for proper visual prominence
  • Add intent to MessageSystemButton forwarded props and set priority="primary" on the inner Banner.Button
  • Text content has to edited in the MessageSystem

Notes for QA

  • Text does not matching Figma till is updated in MessageSystem

Screenshot

@matous-jemelka you can test it here: https://dev.suite.sldev.cz/suite-web/fix/25343-dex-post-trade-survey-ui/web/start

image

Related Issue

Resolve #25343

🔍 Currents Test Results

🔍 Suite desktop test results: View in Currents

🔍 Suite web test results: View in Currents

🔒 Quarantined E2E Tests

Trezor Suite (web) — 13 test(s)
Test Type
Quarantine test: "Onboarding - create wallet,Success (basic)" 🙋 manual
Quarantine test: "Database migration,Db migration between: release/22.5/web => develop/web" 🙋 manual
Passphrase with cardano > verify cardano address behind passphrase 🤖 auto
Discovery > go to wallet settings page, activate all coins and see that there is equal number of records on dashboard 🤖 auto
Account types suite > Add-account-types-non-BTC-coins 🤖 auto
Public Keys > Check ada XPUB 🤖 auto
Cardano > Basic cardano walkthrough 🤖 auto
Export transactions > Go to account and try to export all possible variants (pdf, csv, json) 🤖 auto
Quarantine test: "Trading - Sell inputs,Sell form % inputs and limits" 🙋 manual
Quarantine test: "Trading - Sell inputs,Sell form % inputs and limits" 🙋 manual
Quarantine test: "Multiple sessions,Session overtaken by another" 🙋 manual
Quarantine CANARY test: "Trading - Sell BTC" 🙋 manual
Quarantine test: "Recovery T2T1 - dry run,Recovery after partial recovery" 🙋 manual

Updated: 2026-04-17T11:46:51.345Z • 13 test(s) total

Trezor Suite (desktop) — 15 test(s)
Test Type
Public Keys > Check ada XPUB 🤖 auto
Analytics Events - Staking Navigate > Should log the event staking/navigate - ADA from account menu 🤖 auto
Quarantine test: "Receive transaction,Receive a ETH transaction" 🙋 manual
Quarantine test: "Receive transaction,Receive a ETH transaction" 🙋 manual
Quarantine test: "Global receive and send,Global receive" 🙋 manual
Bridge > App acquired device, EXTERNAL bridge is restarted, app reconnects 🤖 auto
Bridge > App spawns bundled bridge and stops it after app quit 🤖 auto
Bridge > App in daemon mode spawns node-bridge 🤖 auto
Onboarding - create wallet > Success (basic) 🤖 auto
Discovery > go to wallet settings page, activate all coins and see that there is equal number of records on dashboard 🤖 auto
Cardano > Basic cardano walkthrough 🤖 auto
Account types suite > Add-account-types-non-BTC-coins 🤖 auto
Quarantine test: "Multiple sessions,Session overtaken by another" 🙋 manual
Quarantine test: "Send Base,User can perform ethereum sending on base network" 🙋 manual
Quarantine CANARY test: "Use regtest to test pending transactions,Send couple of pending txs and check that they are pending until mined" 🙋 manual

Updated: 2026-04-17T11:45:47.626Z • 15 test(s) total

🌐 Preview deployments

🌐 Suite Web preview: https://dev.suite.sldev.cz/suite-web/fix/25343-dex-post-trade-survey-ui/web/

@sherpaPSX sherpaPSX added the trading Related to Trading label Apr 17, 2026
@sherpaPSX sherpaPSX self-assigned this Apr 17, 2026
@sherpaPSX sherpaPSX added the trading Related to Trading label Apr 17, 2026
@trezor-bot
Copy link
Copy Markdown
Contributor

trezor-bot bot commented Apr 17, 2026

✅ Previously successful run of [Test] PR Suite Desktop e2e tests workflow has been found.
⏭️ Skipping tests for this run.
💡 If you are unsure about your latest changes, please rerun the workflow manually. (Use the Re-run all jobs option)

@trezor-bot
Copy link
Copy Markdown
Contributor

trezor-bot bot commented Apr 17, 2026

✅ Previously successful run of [Test] PR Suite Web e2e tests workflow has been found.
⏭️ Skipping tests for this run.
💡 If you are unsure about your latest changes, please rerun the workflow manually. (Use the Re-run all jobs option)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

trading Related to Trading

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix bugs in DEX post-trade

1 participant