style: chat issues

[coodos]

Description of change

Issue Number

Type of change

• Breaking (any change that would cause existing functionality to not work as expected)
• New (a change which implements a new feature)
• Update (a change which updates existing functionality)
• Fix (a change which fixes an issue)
• Docs (changes to the documentation)
• Chore (refactoring, build scripts or anything else that isn't user-facing)

How the change has been tested

Change checklist

• I have ensured that the CI Checks pass locally
• I have removed any unnecessary logic
• My code is well documented
• I have signed my commits
• My code follows the pattern of the application
• I have self reviewed my code

Summary by CodeRabbit

• Improvements
  • Enhanced mobile chat experience: Blab button now hidden during chat sessions, maximizing available screen space for conversation
  • Improved message clarity: Sender identification now consistently displayed across all message types with updated visual styling for better distinction

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

The PR modifies UI component logic across two platforms: the Blabsy sidebar now hides the Blab button on mobile chat pages by detecting the current route, while the Pictique ChatMessage component now displays sender headers for both own and other messages with updated styling.

Changes

Cohort / File(s)	Summary
Conditional Rendering (Route-Based) platforms/blabsy/src/components/sidebar/sidebar.tsx	Added Next.js useRouter hook to detect current pathname and conditionally hide the Blab button when on mobile chat pages (isMobile && isOnChatPage).
Message Header Logic & Styling platforms/pictique/src/lib/fragments/ChatMessage/ChatMessage.svelte	Changed sender header condition from isHeadNeeded && !isOwn && sender to isHeadNeeded && sender to display headers for own messages; updated styling from text-black-400 to text-black-800 and font-medium to font-semibold.
Indentation Adjustment platforms/pictique/src/routes/(protected)/messages/[id]/+page.svelte	Re-indented block handling regular user messages; no functional changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• ChatMessage condition change — Verify the intent of displaying sender headers for own messages and ensure this aligns with messaging UX expectations
• Blabsy route detection logic — Confirm the pathname check correctly identifies chat pages across all relevant routes
• Styling adjustments — Ensure color (text-black-800) and weight (font-semibold) changes are intentional and visually consistent

Possibly related PRs

• feat: messages #174 — Modifies ChatMessage rendering logic for isHeadNeeded, isOwn, and sender header visibility, directly related to the condition refactor in this PR.

Suggested reviewers

• sosweetham
• JulienAuvo

Poem

🐰 A sidebar hides on mobile chat's stage,
While message headers now shine on every page,
The sender's name grows bold and dark,
No need to hide—just leave a mark!
Small tweaks, but oh, what grace they bring! ✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch style/chat-ui-on-socials

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between acbce56 and 084dd8e.

📒 Files selected for processing (3)

• platforms/blabsy/src/components/sidebar/sidebar.tsx (3 hunks)
• platforms/pictique/src/lib/fragments/ChatMessage/ChatMessage.svelte (1 hunks)
• platforms/pictique/src/routes/(protected)/messages/[id]/+page.svelte (1 hunks)

---

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 and usage tips.}