Skip to content

change legion card styling #35

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 29, 2025
Merged

change legion card styling #35

merged 3 commits into from
Apr 29, 2025

Conversation

dgomie
Copy link
Owner

@dgomie dgomie commented Apr 29, 2025

No description provided.

@dgomie dgomie requested a review from Copilot April 29, 2025 01:31
@dgomie dgomie self-assigned this Apr 29, 2025
Copy link

vercel bot commented Apr 29, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
legionoftones ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 29, 2025 1:34am

@dgomie dgomie linked an issue Apr 29, 2025 that may be closed by this pull request
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the legion card styling by adding a new currentRound property and refactoring the card layout and appearance.

  • Adds currentRound property in DashboardFeed and page components.
  • Refactors the DashboardCard layout with a redesigned structure, new styling classes and images.

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

File Description
src/components/dashboardFeed/DashboardFeed.js Adds currentRound prop to DashboardCard component.
src/components/dashboardCard/DashboardCard.js Refactors card structure, adds currentRound and backgroundColor props, and updates styling with new layout classes and images.
src/app/legions/page.js Adds currentRound prop when rendering legion cards.
Files not reviewed (1)
  • src/components/dashboardCard/DashboardCard.module.css: Language not supported
Comments suppressed due to low confidence (1)

src/components/dashboardCard/DashboardCard.js:13

  • The 'backgroundColor' prop is declared but not used in the component; if it's intended to be applied, please update the styling accordingly or remove it if unnecessary.
backgroundColor,

@dgomie dgomie merged commit ba9b797 into main Apr 29, 2025
1 of 2 checks passed
@dgomie dgomie deleted the dashboard-card-styles branch April 29, 2025 01:34
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.

Update dashboard card styling
1 participant