[ayushgade06]

Description

This PR fixes #8158.

Removes the stray white border around the "Join the Conversation" Discuss card, and the white box behind the logo that makes it hard to read. Affects all pages rendering DiscussCallout:

• https://layer5.io/community
• https://layer5.io/company/faq
• https://layer5.io/learn/learning-paths/mastering-meshery/introduction-to-meshery
• https://layer5.io/learn/learning-paths/mastering-service-meshes-for-developers/introduction-to-service-meshes
• https://layer5.io/learn/learning-paths/mastering-service-meshes-for-developers/advance-concepts-of-service-mesh
• https://layer5.io/learn/learning-paths/mastering-kubernetes-for-engineers/ambassador-edge-stack-api-gateway-with-meshery
• https://layer5.io/learn/learning-paths/mastering-kubernetes-for-engineers/explore-dapr-with-meshery
• https://layer5.io/learn/learning-paths/mastering-kubernetes-for-engineers/scalable-postgres-with-cloudnativepg
• https://layer5.io/learn/learning-paths/mastering-kubernetes-for-engineers/deploying-wordpress-and-mysql-with-persistent-volumes-with-meshery

Notes for Reviewers (After)

Signed Commits

• Yes, I signed my commits.

Summary by CodeRabbit

• Style
  • Corrected a styling issue in the discussion callout. No visible interface changes.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 5cc68fef-4785-4285-80a1-22185b6ca59a

📥 Commits

Reviewing files that changed from the base of the PR and between 70ce445 and 60cccd8.

📒 Files selected for processing (1)

• src/sections/Discuss-Callout/discuss.style.js

💤 Files with no reviewable changes (1)

• src/sections/Discuss-Callout/discuss.style.js

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

---

📝 Walkthrough

Walkthrough

The extra closing brace after .explain was removed from DiscussWrapper. The following button and media-query styles are no longer preceded by that brace.

Changes

Discuss callout styling

Layer / File(s)	Summary
Keep following styles in scope src/sections/Discuss-Callout/discuss.style.js	The extra closing brace after .explain was removed. The button and media-query styles that follow are no longer preceded by it.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix · Severity of issue fixed: Low

Merge Risk: ⚪ Minimal · up to 60ccc

The Discuss callout styling change is ready to merge after normal checks.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The whole-PR diff for #8158 removes the extra closing brace in src/sections/Discuss-Callout/discuss.style.js. This keeps the button rule in the DiscussWrapper styled-components scope and prevent…
Out of Scope Changes check	✅ Passed	The whole-PR diff changes only the extra brace in DiscussWrapper. This change directly addresses #8158. No unrelated changes are present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title identifies the main change: removing the stray white border from the “Join the Conversation” card. The reference to a white box is broader than the stated objective, but the title remains cl…

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[ayushgade06]

@PARTH-TUSSLE as discussed in meet I've made the changes to border, will look into the button (Layer 5 Discuss) as per what the figma file suggests

[github-actions]

Preview deployment for PR #8162 removed.

This PR preview was automatically pruned because we keep only the 3 most recently updated previews on GitHub Pages to stay within deployment size limits.

If needed, push a new commit to this PR to generate a fresh preview.

[ayushgade06]

@PARTH-TUSSLE identified the issue, an extra bracket "}" inside discuss.style.js which caused the rules after that bracket to fall out of scope and not apply, so the button used the browser's default styles instead

[ayushgade06]

@PARTH-TUSSLE could you look into the following PR

[ayushgade06]

Hi @PARTH-TUSSLE , just checking in on this one. The fix is a single-line change in discuss.style.js. It removes the stray } that pushed the following rules out of scope. That removes the white border and the white box behind the logo, and the "Layer5 Discuss" button gets its styles back. It touches every page that renders DiscussCallout. The preview was pruned, so I can push a commit to regenerate it if that helps the review. Thanks!

[Mahakgupta108]

lgtm! but just put the video once, as the pr previewed is not available to get this changes verified across all vp and devices.

[ayushgade06]

@Mahakgupta108 Thanks for the suggestion, I have added the video for reference
@Maanvi212006 @PARTH-TUSSLE Could you take a look at the PR