Skip to content

fix: admin stats width#139

Merged
desoindx merged 1 commit into
developfrom
fix/name
Apr 9, 2026
Merged

fix: admin stats width#139
desoindx merged 1 commit into
developfrom
fix/name

Conversation

@desoindx

@desoindx desoindx commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 9, 2026 08:55

Copilot AI left a comment

Copy link
Copy Markdown

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 addresses layout issues in the Admin stats table by switching to the project’s Table wrapper component and adjusting link styling to avoid forcing overly wide table columns.

Changes:

  • Replace direct DSFR Table usage in admin stats with the shared src/components/Table/Table wrapper.
  • Remove width: max-content / display: block from admin stats links to prevent column width expansion.
  • Add consistent hover transition to link elements (admin stats + brands list).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/components/Brand/BrandsList.module.css Adds a transition on .link to smooth hover margin/icon movement.
src/components/Admin/StatsTable.tsx Uses the shared Table wrapper (dynamic, client-only DSFR table) for consistent rendering/width behavior.
src/components/Admin/StatsTable.module.css Removes width/display rules that could force wide columns; adds .link transition for smoother hover.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@desoindx desoindx merged commit a0ea531 into develop Apr 9, 2026
9 checks passed
@desoindx desoindx deleted the fix/name branch April 9, 2026 09:08
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