Skip to content

refactor: containers to use Tailwind CSS - #5761

Open
therealemjy wants to merge 1 commit into
mainfrom
refactor/tailwind-containers
Open

refactor: containers to use Tailwind CSS#5761
therealemjy wants to merge 1 commit into
mainfrom
refactor/tailwind-containers

Conversation

@therealemjy

Copy link
Copy Markdown
Member

Changes

  • refactor containers to use Tailwind CSS instead of Emotion CSS and MUI components

@therealemjy
therealemjy requested a review from cuzz-venus August 3, 2026 15:53
@changeset-bot

changeset-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: f220e63

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
dapp-preview Ready Ready Preview Aug 3, 2026 3:54pm
dapp-testnet Ready Ready Preview Aug 3, 2026 3:54pm
venus.io Ready Ready Preview Aug 3, 2026 3:54pm

Request Review

@greptile-apps

greptile-apps Bot commented Aug 3, 2026

Copy link
Copy Markdown

Greptile Summary

The PR migrates several EVM application containers from Emotion and MUI presentation components to Tailwind CSS while retaining their existing structure and behavior.

  • Replaces MarketTable and ProposalCard Emotion styles with Tailwind utilities.
  • Converts remaining responsive, typography, visibility, spacing, and alignment rules in AccountOverview, NavBar, and TokenListWrapper.
  • Removes obsolete container-specific Emotion style modules.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete behavioral, responsive, or build regression identified in the changed containers.

The Tailwind conversions preserve the configured MUI typography, colors, spacing, breakpoints, and layout behavior, while the removed MarketTable descendant selector does not target an element rendered by the current table implementation.

Important Files Changed

Filename Overview
apps/evm/src/containers/MarketTable/index.tsx Removes the container-level Emotion hook and obsolete descendant styling without changing table data, controls, navigation, or actions.
apps/evm/src/containers/MarketTable/useColumns/index.tsx Replaces icon and borrow-limit alignment styles with equivalent Tailwind spacing and responsive flex utilities.
apps/evm/src/containers/ProposalCard/index.tsx Reimplements the proposal card’s responsive grid, spacing, borders, title clamp, and typography with equivalent Tailwind utilities.
apps/evm/src/containers/AccountOverview/index.tsx Rewrites the period selector’s responsive width rule with equivalent Tailwind breakpoint behavior.
apps/evm/src/containers/Layout/NavBar/MenuItem/index.tsx Reorders equivalent responsive visibility classes for the beta badge.
apps/evm/src/containers/TokenListWrapper/index.tsx Replaces MUI Typography with a semantic span carrying the corresponding text size and explicit color.

Reviews (1): Last reviewed commit: "refactor: containers to use Tailwind CSS" | Re-trigger Greptile

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Coverage Report for ./apps/evm

Status Category Percentage Covered / Total
🔵 Lines 80.82% 51928 / 64245
🔵 Statements 80.82% 51928 / 64245
🔵 Functions 74.35% 1574 / 2117
🔵 Branches 86.13% 7624 / 8851
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
apps/evm/src/containers/AccountOverview/index.tsx 97.32% 82.53% 75% 97.32% 234, 237, 279-283, 292
apps/evm/src/containers/Layout/NavBar/MenuItem/index.tsx 96.47% 73.68% 60% 96.47% 81-84
apps/evm/src/containers/MarketTable/index.tsx 93.75% 93.1% 71.42% 93.75% 92-101
apps/evm/src/containers/MarketTable/useColumns/index.tsx 81.02% 77.17% 75% 81.02% 135-137, 148-156, 173-178, 183-197, 379-387, 406-407, 410-411, 414-419, 437-453
apps/evm/src/containers/ProposalCard/index.tsx 100% 100% 100% 100%
apps/evm/src/containers/TokenListWrapper/index.tsx 90.9% 97.5% 100% 90.9% 156-170
Generated in workflow #14023 for commit f220e63 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