Skip to content

Conversation

@thomasyuill-livekit
Copy link
Contributor

@thomasyuill-livekit thomasyuill-livekit commented Jan 16, 2026

Summary

Fixes #1249 (comment)

  • Reverts the audio bar visualizer class name back to audio-bar-visualizer across styles and the BarVisualizer component

Summary by CodeRabbit

  • Style
    • Renamed the CSS class used by the audio visualizer to standardize styling hooks; no functional changes to behavior or layout.
  • Revert
    • Added a patch-level change to revert an unintended classname change, restoring the expected styling selector scopes.

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

@changeset-bot
Copy link

changeset-bot bot commented Jan 16, 2026

🦋 Changeset detected

Latest commit: 62fe1b7

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

This PR includes changesets to release 6 packages
Name Type
@livekit/components-react Patch
@agents-ui Patch
@livekit/component-example-next Patch
@livekit/components-js-docs Patch
@livekit/component-docs-storybook Patch
@livekit/components-docs-gen Patch

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 16, 2026

📝 Walkthrough

Walkthrough

Replaced the audio visualizer root CSS class from lk-audio-visualizer-bar to lk-audio-bar-visualizer in the React component and updated corresponding SCSS selectors and a control-bar descendant selector to match.

Changes

Cohort / File(s) Summary
React component
packages/react/src/components/participant/BarVisualizer.tsx
Updated root element className from lk-audio-visualizer-barlk-audio-bar-visualizer.
Component styles
packages/styles/scss/components/participant/_audio-visualizer.scss
Renamed selector .audio-visualizer-bar.audio-bar-visualizer.
Prefab/control-bar styles
packages/styles/scss/prefabs/control-bar.scss
Adjusted descendant selector under .agent-control-bar from & .audio-visualizer-bar& .audio-bar-visualizer.
Release metadata
.changeset/tame-readers-hope.md
Added changeset noting the classname correction for a patch release.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

Poem

🐰 I hopped through CSS in search of a sign,
Swapped the words so the classes align.
Bars now parade in tidy array,
A ripple of styles that cheerfully play.
Hooray for a tidy classname day! 🎉

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: reverting an unintentional classname change to BarVisualizer across react and styles packages.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.



📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e28a64a and 62fe1b7.

📒 Files selected for processing (1)
  • .changeset/tame-readers-hope.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: test
🔇 Additional comments (1)
.changeset/tame-readers-hope.md (1)

1-5: No additional changeset is needed. Git verification confirms that the styles package (@livekit/components-styles) was not modified in this PR. The changeset correctly lists only @livekit/components-react as the affected package.

Likely an incorrect or invalid review comment.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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

@github-actions
Copy link
Contributor

github-actions bot commented Jan 16, 2026

size-limit report 📦

Path Size
LiveKitRoom only 6 KB (0%)
LiveKitRoom with VideoConference 32.46 KB (0%)
All exports 43.37 KB (-0.07% 🔽)

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
packages/styles/scss/prefabs/control-bar.scss (1)

19-23: Fix SCSS selectors to match the React component's prefixed class names.

The component applies lk-audio-bar-visualizer, lk-audio-bar, and lk-highlighted classes, but these selectors target .audio-bar-visualizer and .audio-bar.highlighted without the lk- prefix. Update selectors to .lk-audio-bar-visualizer and .lk-audio-bar.lk-highlighted for the styles to apply correctly.

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 776dd1d and e28a64a.

📒 Files selected for processing (3)
  • packages/react/src/components/participant/BarVisualizer.tsx
  • packages/styles/scss/components/participant/_audio-visualizer.scss
  • packages/styles/scss/prefabs/control-bar.scss
🧰 Additional context used
🧬 Code graph analysis (1)
packages/react/src/components/participant/BarVisualizer.tsx (1)
packages/react/src/utils.ts (1)
  • mergeProps (14-19)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: test
🔇 Additional comments (2)
packages/react/src/components/participant/BarVisualizer.tsx (1)

115-115: LGTM!

The class name lk-audio-bar-visualizer follows the LiveKit naming convention and aligns with the coordinated rename across the SCSS style files.

packages/styles/scss/components/participant/_audio-visualizer.scss (1)

32-70: LGTM!

The selector rename to .audio-bar-visualizer is consistent with the React component and control-bar prefab changes. The internal structure and nested selectors remain intact.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

@thomasyuill-livekit thomasyuill-livekit changed the title fix(react,styles): fix(react,styles): Revert unintentional classname change to BarVisualizer Jan 16, 2026
@thomasyuill-livekit thomasyuill-livekit merged commit 7b3d1f0 into main Jan 16, 2026
7 checks passed
@thomasyuill-livekit thomasyuill-livekit deleted the ty/fix-component-react-styles branch January 16, 2026 17:30
@github-actions github-actions bot mentioned this pull request Jan 16, 2026
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.

3 participants