Skip to content

regression: fuselage-ui-kit's InfoCard element secondary background color token#37594

Merged
kodiakhq[bot] merged 1 commit intorelease-7.13.0from
fix/infoCardBg
Nov 26, 2025
Merged

regression: fuselage-ui-kit's InfoCard element secondary background color token#37594
kodiakhq[bot] merged 1 commit intorelease-7.13.0from
fix/infoCardBg

Conversation

@gabriellsh
Copy link
Member

@gabriellsh gabriellsh commented Nov 24, 2025

Proposed changes (including videos or screenshots)

Changed the background token to the proper one.

Issue(s)

VGA-72

Steps to test or reproduce

Further comments

Summary by CodeRabbit

  • Style
    • Updated the background color of the InfoCard component for improved visual consistency.

✏️ Tip: You can customize this high-level summary in your review settings.

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Nov 24, 2025

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

@changeset-bot
Copy link

changeset-bot bot commented Nov 24, 2025

⚠️ No Changeset found

Latest commit: ac97021

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 24, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The InfoCard component's background color property has been updated from surface-neutral to surface-tint to align with design specifications for the message block information row.

Changes

Cohort / File(s) Summary
Background Color Update
packages/fuselage-ui-kit/src/blocks/InfoCard.tsx
Changed InfoCard container background color from surface-neutral to surface-tint

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

stat: QA assured

Suggested reviewers

  • cardoso

Poem

🐰 A tint of new color, so fresh and so bright,
The InfoCard glows with a surface-tint light,
From neutral we've shifted to something more fair,
A hop, skip, and change floating through the UI air! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title uses 'regression' but the actual change is a bug fix (correction of background color), not a regression fix. Change 'regression' to 'fix' to accurately reflect that this corrects an existing issue rather than addressing a regression.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The code changes fully satisfy VGA-72's requirement to change the InfoCard background fill color from surface-neutral token to surface-tint token.
Out of Scope Changes check ✅ Passed All changes are in scope: the InfoCard background color update and the corresponding changeset entry directly address the linked issue requirement.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 004aa67 and ac97021.

📒 Files selected for processing (1)
  • packages/fuselage-ui-kit/src/blocks/InfoCard.tsx (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gabriellsh gabriellsh added this to the 7.14.0 milestone Nov 24, 2025
@gabriellsh gabriellsh added the stat: QA assured Means it has been tested and approved by a company insider label Nov 24, 2025
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Nov 24, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 25, 2025

📦 Docker Image Size Report

📈 Changes

Service Current Baseline Change Percent
sum of all images 1.2GiB 1.2GiB +12MiB
rocketchat 358MiB 346MiB +12MiB
omnichannel-transcript-service 132MiB 132MiB -1.1KiB
queue-worker-service 132MiB 132MiB -178B
ddp-streamer-service 127MiB 127MiB -265B
account-service 114MiB 114MiB -151B
stream-hub-service 111MiB 111MiB +800B
authorization-service 111MiB 111MiB +21B
presence-service 111MiB 111MiB +3.0KiB

📊 Historical Trend

---
config:
  theme: "dark"
  xyChart:
    width: 900
    height: 400
---
xychart
  title "Image Size Evolution by Service (Last 30 Days + This PR)"
  x-axis ["11/15 22:28", "11/16 01:28", "11/17 23:50", "11/18 22:53", "11/19 23:02", "11/21 16:49", "11/24 17:34", "11/26 13:02 (PR)"]
  y-axis "Size (GB)" 0 --> 0.5
  line "account-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "authorization-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "ddp-streamer-service" [0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12]
  line "omnichannel-transcript-service" [0.14, 0.14, 0.14, 0.14, 0.14, 0.13, 0.13, 0.13]
  line "presence-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "queue-worker-service" [0.14, 0.14, 0.14, 0.14, 0.14, 0.13, 0.13, 0.13]
  line "rocketchat" [0.36, 0.36, 0.35, 0.35, 0.35, 0.34, 0.34, 0.35]
  line "stream-hub-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
Loading

Statistics (last 7 days):

  • 📊 Average: 1.5GiB
  • ⬇️ Minimum: 1.2GiB
  • ⬆️ Maximum: 1.6GiB
  • 🎯 Current PR: 1.2GiB
ℹ️ About this report

This report compares Docker image sizes from this build against the develop baseline.

  • Tag: pr-37594
  • Baseline: develop
  • Timestamp: 2025-11-26 13:02:20 UTC
  • Historical data points: 7

Updated: Wed, 26 Nov 2025 13:02:21 GMT

MartinSchoeler
MartinSchoeler previously approved these changes Nov 26, 2025
@gabriellsh gabriellsh changed the title fix: fuselage-ui-kit's InfoCard element secondary background color token regression: fuselage-ui-kit's InfoCard element secondary background color token Nov 26, 2025
@gabriellsh gabriellsh modified the milestones: 7.14.0, 7.13.0 Nov 26, 2025
@gabriellsh gabriellsh removed stat: ready to merge PR tested and approved waiting for merge stat: QA assured Means it has been tested and approved by a company insider labels Nov 26, 2025
@gabriellsh gabriellsh changed the base branch from develop to release-7.13.0 November 26, 2025 13:05
@gabriellsh gabriellsh dismissed MartinSchoeler’s stale review November 26, 2025 13:05

The base branch was changed.

@gabriellsh gabriellsh requested review from a team as code owners November 26, 2025 13:05
@d-gubert d-gubert added the stat: QA assured Means it has been tested and approved by a company insider label Nov 26, 2025
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Nov 26, 2025
@kodiakhq kodiakhq bot merged commit d19bb8a into release-7.13.0 Nov 26, 2025
10 of 12 checks passed
@kodiakhq kodiakhq bot deleted the fix/infoCardBg branch November 26, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants