Skip to content

fix(stories): update Box stories to correctly render colors and layouts#1732

Closed
Aryan9inja wants to merge 1 commit intoRocketChat:mainfrom
Aryan9inja:fix_1695
Closed

fix(stories): update Box stories to correctly render colors and layouts#1732
Aryan9inja wants to merge 1 commit intoRocketChat:mainfrom
Aryan9inja:fix_1695

Conversation

@Aryan9inja
Copy link
Contributor

@Aryan9inja Aryan9inja commented Jul 30, 2025

Proposed changes (including videos or screenshots)

  • Fixed the rendering of various color variants (bg='light', bg='tint', etc.) in the Box stories.
  • Ensured layout-related props like padding, margin, border, and size are visually represented.
  • Wrapped the stories in a flex container to showcase variants side-by-side.

Video proof of issue fix-

Recording.2025-07-30.151314.mp4

Issue(s)

Closes #1695

Further comments

Used SurfaceContext.Provider to ensure background color tokens are interpreted correctly when rendering Box variants. Considered alternatives like hardcoded background colors but chose context to align with design token usage across Fuselage.

@CLAassistant
Copy link

CLAassistant commented Jul 30, 2025

CLA assistant check
All committers have signed the CLA.

@Aryan9inja
Copy link
Contributor Author

@aleksandernsilva Can you pls review this bug fix.

@Anxhul10
Copy link
Contributor

Anxhul10 commented Sep 1, 2025

Thankyou for your contribution.
Can you use specific branch name for this pr

@Aryan9inja Aryan9inja closed this Sep 1, 2025
@Aryan9inja Aryan9inja deleted the fix_1695 branch September 1, 2025 16:43
@Aryan9inja Aryan9inja restored the fix_1695 branch September 1, 2025 16:44
@Aryan9inja Aryan9inja reopened this Sep 1, 2025
@Aryan9inja Aryan9inja closed this Sep 1, 2025
@Aryan9inja Aryan9inja reopened this Sep 1, 2025
@Aryan9inja Aryan9inja closed this Sep 1, 2025
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.

Box: Colour and Layout are not visible

3 participants