Skip to content

feat: style discussion section in modal with heading and background#323

Merged
rdmueller merged 1 commit intoLLM-Coding:mainfrom
raifdmueller:fix/feedback-styling
Mar 18, 2026
Merged

feat: style discussion section in modal with heading and background#323
rdmueller merged 1 commit intoLLM-Coding:mainfrom
raifdmueller:fix/feedback-styling

Conversation

@raifdmueller
Copy link
Contributor

@raifdmueller raifdmueller commented Mar 18, 2026

Summary

Visually separates community discussion comments from anchor content in the modal:

  • Heading: "Community Discussion" / "Community-Diskussion" with chat icon
  • Background: Light gray container with rounded corners and subtle border
  • Only visible when comments exist — no empty box
┌─ Modal ─────────────────────────────┐
│                                     │
│  (anchor content in normal style)   │
│                                     │
│  ┌─ 💬 Community Discussion ─────┐ │
│  │  👤 rdmueller                  │ │
│  │  works best for me to clarify  │ │
│  └────────────────────────────────┘ │
│                                     │
├─────────────────────────────────────┤
│ Is this anchor useful?  [Vote] [💬] │
└─────────────────────────────────────┘

Test plan

  • Anchor with comments (Socratic Method): gray box with heading visible
  • Anchor without comments: no feedback section shown
  • Dark mode: appropriate contrast

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Neue Funktionen

    • Neue Überschrift "Community-Diskussion" im Feedback-Bereich mit visuellem Icon-Element hinzugefügt.
  • Design

    • Feedback-Sektion überarbeitet: Abstände optimiert, abgerundete Ecken und Hintergrundfarben (Light- und Dark-Modus) erweitert sowie Rahmen verbessert.

- Add "Community Discussion" heading with chat icon
- Light gray background to visually separate from anchor content
- Only shown when comments exist
- i18n: EN "Community Discussion" / DE "Community-Diskussion"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Mar 18, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

Das Pull Request fügt einen neuen visuellen Abschnitt mit einer Überschrift und einem SVG-Icon über den Feedback-Kommentaren hinzu, ergänzt um entsprechende CSS-Stile, Translations und Komponentenlogik zur Renderung des neuen Elements.

Changes

Cohort / File(s) Summary
UI-Komponente und Styling
website/src/components/anchor-modal.js, website/src/styles/main.css
Injektion eines neuen Discussion-Heading-Blocks mit SVG-Icon und übersetztem Text über den Feedback-Kommentaren; Umgestaltung des feedback-section-Containers mit aktualisiertem Padding, Border-Styling und Hintergrundfarbe; Einführung neuer .feedback-heading-Klasse mit Flex-Layout und Typografie.
Übersetzungen
website/src/translations/en.json, website/src/translations/de.json
Hinzufügung neuer Übersetzungsschlüssel "feedback.discussionHeading" für Englisch ("Community Discussion") und Deutsch ("Community-Diskussion").

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • feat: anchor feedback via GitHub Discussions #318: Beide PRs modifizieren website/src/components/anchor-modal.js zur Erweiterung der Feedback- und Discussion-UI-Funktionalität, wobei dieser PR die übersetzte Diskussions-Überschrift über Kommentaren hinzufügt.
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Der Titel beschreibt präzise die Hauptänderung: Styling der Discussion-Section im Modal mit Heading und Hintergrund, was den Änderungen in CSS, HTML-Struktur und Übersetzungen entspricht.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
📝 Coding Plan
  • Generate coding plan for human review comments

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@rdmueller rdmueller merged commit 7ff651d into LLM-Coding:main Mar 18, 2026
6 of 7 checks passed
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