Skip to content

feat: update Flux banner#5607

Open
therealemjy wants to merge 1 commit into
mainfrom
feat/flux-banner-update
Open

feat: update Flux banner#5607
therealemjy wants to merge 1 commit into
mainfrom
feat/flux-banner-update

Conversation

@therealemjy
Copy link
Copy Markdown
Member

Jira ticket(s)

VPD-1315

Changes

  • update Flux banner

@therealemjy therealemjy requested a review from cuzz-venus May 29, 2026 07:32
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 29, 2026

🦋 Changeset detected

Latest commit: 75cc919

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@venusprotocol/evm Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented May 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dapp-preview Ready Ready Preview May 29, 2026 10:55am
dapp-testnet Ready Ready Preview May 29, 2026 10:55am
venus.io Ready Ready Preview May 29, 2026 10:55am

Request Review

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 29, 2026

Greptile Summary

This PR updates the Venus Flux ad banner with new copy ("$50K USDT rewards on Venus Flux", "30 days only · Live May 28") and a redesigned layout that consolidates two breakpoint-specific images into a single illustration positioned differently on mobile and desktop.

  • New illustration replaces the old mobileIllustration.png / tabletIllustration.png pair, now positioned to the right on mobile and left on sm+ screens.
  • Banner text and button layout restructured; responsive breakpoints on typography classes updated from lg: to sm:; all seven locale files updated with the new copy.
  • Snapshot regenerated to match the new markup; target=\"_blank\" was already handled by the Link container component and its removal from props has no behavioral effect.

Confidence Score: 5/5

Safe to merge — changes are scoped to a visual ad banner update with no data fetching, auth, or business logic involved.

The change touches only a promotional banner component, translation strings, a new image asset, and its snapshot. No logic paths, API calls, or state management are modified.

No files require special attention.

Important Files Changed

Filename Overview
apps/evm/src/containers/AdBanner/VenusFluxBanner/index.tsx Banner layout refactored: two separate mobile/tablet images replaced with a single illustration positioned at right (mobile) and left (sm+); text hierarchy and button layout restructured; responsive breakpoints updated from lg to sm on typography lines. The touched typography lines still use blocklisted Tailwind utilities instead of design tokens.
apps/evm/src/libs/translations/translations/en.json Banner copy updated: title changed to "$50K USDT rewards on Venus Flux" and description updated to "30 days only · Live May 28"; all other locale files updated consistently.
.changeset/funny-dingos-relax.md New minor changeset entry for the Flux banner update.
apps/evm/src/pages/Dashboard/tests/snapshots/index.spec.tsx.snap Snapshot updated to reflect the banner layout and copy changes.

Reviews (2): Last reviewed commit: "feat: update Flux banner" | Re-trigger Greptile

@therealemjy
Copy link
Copy Markdown
Member Author

@greptile

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026

Coverage Report for ./apps/evm

Status Category Percentage Covered / Total
🔵 Lines 81.32% 45828 / 56353
🔵 Statements 81.32% 45828 / 56353
🔵 Functions 62.35% 651 / 1044
🔵 Branches 72.33% 5172 / 7150
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
apps/evm/src/containers/AdBanner/VenusFluxBanner/index.tsx 100% 0% 100% 100%
Generated in workflow #13530 for commit 75cc919 by the Vitest Coverage Report Action

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