docs: Markenquellen unter docs/marke ablegen - #46
Conversation
Die Originale des Logo-Pakets lagen bisher nur im Downloads-Ordner. Damit war alles, was public/ nicht ausliefert, nach dem nächsten Aufräumen weg — auch die 1024er-Bildmarke, aus der die Icon-Größen entstehen. - docs/marke/ nimmt Master-, Bild- und Wortmarke als SVG und PNG auf, dazu das Entwurfsblatt zur Formfindung. - docs/marke/README.md ordnet die Dateien zu und hält die beiden Ableitungen fest, die keine Kopien sind: das mehrgrößige favicon.ico und das maskierbare Icon mit Sicherheitsrand — beide als ausführbarer Befehl. - README und designsystem.md verweisen darauf. docs/ steht in .dockerignore; das Abbild bleibt unverändert. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 43 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughDie Markenquellen und Ableitungsbefehle werden in ChangesMarkendokumentation
Estimated code review effort: 2 (Einfach) | ~10 Minuten 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 13fba2f2f3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # Marke füllt ihre Fläche fast randlos, deshalb hier auf 56 % verkleinert und | ||
| # auf die helle Fläche des Designsystems gesetzt. | ||
| magick public/logo-bildmarke.png -resize 288x288 \ | ||
| -background "#f1ebe0" -gravity center -extent 512x512 public/maskable-512.png |
There was a problem hiding this comment.
When this documented command is rerun, public/logo-bildmarke.png contains transparent pixels and ImageMagick's -background only supplies pixels added by -extent; it does not flatten transparency already present in the resized 288×288 canvas. The result therefore has a transparent central area instead of the fully opaque #F1EBE0 background of the committed maskable icon and will not reproduce the design-system asset. Set the background and remove/flatten alpha before extending the canvas.
AGENTS.md reference: AGENTS.md:L42-L44
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Geprüft und widerlegt — mit einem Zusatz, der die Absicht künftig explizit macht.
-extent hängt keine Ränder an, sondern legt eine neue Leinwand in -background an und komponiert das Bild mit over darauf. Die Transparenz der 288×288-Fläche wird dabei bereits gegen #F1EBE0 gerechnet, nicht durchgereicht. Ein separates Flatten wäre nur nötig, wenn Quelle und Zielgröße gleich sind — dann entsteht keine Leinwand.
Messung an den tatsächlichen Dateien (ImageMagick 7.1.2-27):
logo-bildmarke.png opaque=False alpha_min=0 # Prämisse stimmt: Quelle ist transparent
maskable-512.png opaque=True alpha_min=255 # committetes Icon ist vollständig deckend
Zentrumspixel p{256,256} = srgb(241,235,224) = #F1EBE0
Der dokumentierte Befehl neu ausgeführt ergibt compare -metric AE = 0 gegen die committete Datei — also pixelgleich, kein transparenter Bereich.
Übernommen ist trotzdem etwas: 59eb728 ergänzt -alpha remove -alpha off samt Begründung. Am Ergebnis ändert das nichts (erneut AE = 0), aber die Anforderung „ein maskierbares Icon muss deckend sein" hing bisher unausgesprochen am Verhalten von -extent.
Der Befehl war korrekt — `-extent` komponiert auf eine Leinwand in `-background`, die Transparenz der Quelle ist damit schon gegen #F1EBE0 gerechnet. Nachgebaut ist das Ergebnis pixelgleich zum committeten Icon (compare -metric AE = 0), auch mit dem Zusatz. Trotzdem `-alpha remove -alpha off` dazu, samt Begründung im Kommentar: Dass das Icon deckend sein muss, hing bisher unausgesprochen am Verhalten von `-extent`. Ein Review hat genau daran Anstoß genommen; die Frage soll sich nicht ein zweites Mal stellen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
docs/marke/README.md (1)
40-48: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick win
-compose overexplizit setzen.Der Befehl setzt
-background, aber nicht-compose.-extentverwendet die Kompositionseinstellung für die neue Leinwand. Setze-compose over, damit die dokumentierte Ableitung nicht vom impliziten Standard abhängt. (imagemagick.org)Vorgeschlagene Änderung
magick public/logo-bildmarke.png -resize 288x288 \ - -background "`#f1ebe0`" -gravity center -extent 512x512 \ + -background "`#f1ebe0`" -compose over -gravity center -extent 512x512 \ -alpha remove -alpha off public/maskable-512.png🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/marke/README.md` around lines 40 - 48, Update the ImageMagick command in the maskable icon generation example to explicitly set the composition mode to over before -extent, preserving the existing background, sizing, and alpha options.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/marke/README.md`:
- Around line 62-64: Vereinheitliche die Synchronisationshinweise an allen drei
Stellen: In docs/marke/README.md (Zeilen 62-64) Quelle, public/-Ableitungen und
src/components/Marke.tsx nennen; in README.md (Zeilen 434-435) die Neuerzeugung
der public/-Assets per ImageMagick ergänzen; in docs/designsystem.md (Zeile 47)
„Beide Fassungen“ durch einen Verweis auf diesen vollständigen Ableitungsprozess
ersetzen.
---
Nitpick comments:
In `@docs/marke/README.md`:
- Around line 40-48: Update the ImageMagick command in the maskable icon
generation example to explicitly set the composition mode to over before
-extent, preserving the existing background, sizing, and alpha options.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: cbb45391-fa9e-4898-807b-e807b8f59aa2
⛔ Files ignored due to path filters (7)
docs/marke/sitzplan-studio-bildmarke.pngis excluded by!**/*.pngdocs/marke/sitzplan-studio-bildmarke.svgis excluded by!**/*.svgdocs/marke/sitzplan-studio-logo.pngis excluded by!**/*.pngdocs/marke/sitzplan-studio-logo.svgis excluded by!**/*.svgdocs/marke/sitzplan-studio-refined-concept.pngis excluded by!**/*.pngdocs/marke/sitzplan-studio-wortmarke.pngis excluded by!**/*.pngdocs/marke/sitzplan-studio-wortmarke.svgis excluded by!**/*.svg
📒 Files selected for processing (3)
README.mddocs/designsystem.mddocs/marke/README.md
Der Hinweis nannte zwei Fassungen und übersprang die mittlere: Zwischen der Quelle in docs/marke und der Komponente liegen die Dateien in public/, von denen zwei nicht bloß Kopien sind. Wer nur die Enden ändert, hat ein Favicon aus der alten Form — und merkt nichts, weil jede Fassung für sich stimmig aussieht. Jetzt in docs/marke/README.md als nummerierter Weg, in README.md und designsystem.md entsprechend verwiesen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 participant
Nachtrag zu #45: dort sind nur die Auslieferungsassets nach
public/gewandert.Die Originale lagen weiterhin allein im Downloads-Ordner — nach dem nächsten
Aufräumen wären sie weg, samt der 1024er-Bildmarke, aus der die Icon-Größen
entstehen.
Inhalt
docs/marke/nimmt Master-, Bild- und Wortmarke als SVG und PNG auf, dazu dasEntwurfsblatt zur Formfindung (
sitzplan-studio-refined-concept.png, nicht zumEinsetzen).
docs/marke/README.mdordnet die Dateien mit Maßen und Verwendungszweck zu undhält die beiden Ableitungen fest, die keine reinen Kopien sind:
favicon.ico(16/32/48/64 statt der 173-KB-Einzelgröße)maskable-512.pngmit Sicherheitsrand für die Android-Icon-MaskeBeide stehen dort als ausführbarer
magick-Befehl, sind also reproduzierbar.Dazu der Hinweis auf die dritte Fassung der Bildmarke in
src/components/Marke.tsxund warum es sie gibt.README und
designsystem.mdverweisen auf den neuen Ordner.Zum Abbild
docs/steht in.dockerignore— der Container ändert sich durch diesen PRnicht, ein erneutes Ausrollen ist nicht nötig.
🤖 Generated with Claude Code
Summary by CodeRabbit