Skip to content

fix(opal): extract interactive container styles to CSS#8307

Merged
raunakab merged 2 commits intomainfrom
fix/interactive-container
Feb 10, 2026
Merged

fix(opal): extract interactive container styles to CSS#8307
raunakab merged 2 commits intomainfrom
fix/interactive-container

Conversation

@raunakab
Copy link
Copy Markdown
Contributor

@raunakab raunakab commented Feb 10, 2026

Description

Extract inline Tailwind classes from InteractiveContainer into a dedicated CSS class (.interactive-container) and move the border prop to a data-border data attribute, consistent with the existing data-attribute styling pattern used throughout styles.css. Also adds self-stretch to the container.

Screenshots

PATs (in the User Settings)

image

File Cards (in the main Chat Input Bar)

image

How Has This Been Tested?

  • Verified build passes (bun run types:check)
  • Visual inspection of interactive components

Additional Options

  • [Required] I have considered whether this PR needs to be cherry-picked to the latest beta branch.
  • [Optional] Override Linear Check

Summary by cubic

Extracted InteractiveContainer inline Tailwind styles into a reusable CSS class and switched border styling to a data attribute. Also restores self-stretch for more consistent layout.

  • Refactors
    • Added .interactive-container class with flex, center alignment, and self-stretch.
    • Moved border prop to data-border="true" to follow the data-attribute styling pattern.

Written for commit 3405142. Summary will update on new commits.

raunakab and others added 2 commits February 10, 2026 13:18
…er attribute

Move inline Tailwind classes to a CSS class and use data-border attribute
for border styling, consistent with the existing data-attribute pattern.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@raunakab raunakab requested a review from a team as a code owner February 10, 2026 21:23
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Feb 10, 2026

Greptile Overview

Greptile Summary

Extracted inline Tailwind classes (flex items-center justify-center self-stretch) from InteractiveContainer into a dedicated .interactive-container CSS class in styles.css. Migrated the border prop from an inline conditional class to a data-border="true" data attribute with corresponding CSS selector, following the existing data-attribute pattern used throughout the file (like data-disabled, data-static, data-transient, etc.).

  • Consolidates styling into CSS for better maintainability and consistency
  • Restores self-stretch to the container (previously added in commit 4b0569f)
  • Follows the established data-attribute styling convention in the opal interactive system
  • No functional changes - purely a refactoring for better code organization

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • Clean refactoring that extracts inline styles to CSS and adopts the existing data-attribute pattern. No logic changes, no new dependencies, and follows established conventions in the codebase.
  • No files require special attention

Important Files Changed

Filename Overview
web/lib/opal/src/core/interactive/components.tsx Refactored inline Tailwind classes to use CSS class .interactive-container and migrated border prop to data-border attribute
web/lib/opal/src/core/interactive/styles.css Added .interactive-container class with flex layout and self-stretch, plus data-border attribute selector

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@raunakab raunakab added this pull request to the merge queue Feb 10, 2026
Merged via the queue into main with commit 6fab710 Feb 10, 2026
83 of 93 checks passed
@raunakab raunakab deleted the fix/interactive-container branch February 10, 2026 21:47
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