Skip to content

Fix "Row-Reverse" Styling Bug#263

Merged
Satoshi-Sh merged 4 commits intomainfrom
fix/two-column-component
Jul 28, 2025
Merged

Fix "Row-Reverse" Styling Bug#263
Satoshi-Sh merged 4 commits intomainfrom
fix/two-column-component

Conversation

@Satoshi-Sh
Copy link
Copy Markdown
Member

@Satoshi-Sh Satoshi-Sh commented Jul 22, 2025

Web Dev Path
262

Have you updated the CHANGELOG.md file? If not, please do it.

Yes

What is this change?

The styling bug was introduced in my last PR #258 . Reverting the Container prop name(styles) fixed the issue.

If necessary, please describe how to test the new feature or fix.

Please check the styling both home and about page. Two columns with row-reverse .

Screenshot from 2025-07-22 06-54-52

When should this be merged?

After review

@Satoshi-Sh Satoshi-Sh requested a review from a team July 22, 2025 11:56
@Satoshi-Sh Satoshi-Sh self-assigned this Jul 22, 2025
@Satoshi-Sh Satoshi-Sh added the bug Something isn't working label Jul 22, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 22, 2025

Deploy Preview for webdevpathstage ready!

Name Link
🔨 Latest commit 3658758
🔍 Latest deploy log https://app.netlify.com/projects/webdevpathstage/deploys/68876e52fa456800085d0b57
😎 Deploy Preview https://deploy-preview-263--webdevpathstage.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Satoshi-Sh Satoshi-Sh force-pushed the fix/two-column-component branch from 732ed88 to cedca55 Compare July 22, 2025 14:00
Copy link
Copy Markdown
Member

@cherylli cherylli left a comment

Choose a reason for hiding this comment

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

Thanks for the fix

Copy link
Copy Markdown
Member

@briangesteban briangesteban left a comment

Choose a reason for hiding this comment

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

Looks good, bug is fixed. Thanks @Satoshi-Sh !

Comment thread components/containers/Container/index.js
Copy link
Copy Markdown
Member

@oluwatobiss oluwatobiss left a comment

Choose a reason for hiding this comment

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

LGTM

@Satoshi-Sh Satoshi-Sh merged commit e979738 into main Jul 28, 2025
4 checks passed
@Satoshi-Sh Satoshi-Sh deleted the fix/two-column-component branch July 28, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants