Repository navigation
docs(gi): add the marketing package for GI and myPolitics - #132
gieniek-ai wants to merge 4 commits into
Conversation
Avatars, covers and banners, link previews and e-mail signatures for the social channels of the foundation and of myPolitics, made from the logo files without changes, with a README giving each platform's official size and safe area as checked on 2026-10-18. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![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: 6ad59bd90e
ℹ️ 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".
| ctx.fillStyle = ACCENT; | ||
| ctx.fillText(label, PAD, firstBaseline - size * 0.72 - 28); | ||
| } | ||
| const fontOk = document.fonts.check('700 64px Poppins'); |
There was a problem hiding this comment.
When this file is opened offline, the Google stylesheet is unavailable, and Poppins is not installed locally, no Poppins face is registered: both document.fonts.load(...) calls resolve with an empty list, while FontFaceSet.check(...) is true when there is no matching face. Consequently fontOk enables both buttons and the canvas silently uses Arial, contrary to the package's offline warning; the identical myPolitics template needs to require a returned Poppins face (and handle a rejected load) before exporting.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Right - with the stylesheet blocked, document.fonts.load resolves to an empty list and nothing stopped the export. Fixed in e8f9ebe for both templates: Poppins counts as loaded only when both loads (bold for the title, medium for the label) return a face, and a rejected load counts as none. Checked in Chromium with fonts.googleapis.com and fonts.gstatic.com blocked: both buttons stay off and the page shows the offline note; with the fonts reachable they are on and a JPG with a Polish title saves.
| | `*-og-template.svg` | The same layout with the title as a text element, for Figma or Inkscape. It needs Poppins installed | | ||
| | `*-og-example*.png` | Examples made with the HTML template | | ||
|
|
||
| The template needs an internet connection to load Poppins from Google Fonts; without it the save buttons stay off and the page says why. Prefer the JPG: WhatsApp does not show previews larger than 300 KB, and the PNG from the template is usually larger than that. Keep the title and the logo away from the edges as the template does - a few apps crop the picture to a square in the middle. |
There was a problem hiding this comment.
In the apps described here that center-crop a 1200 × 630 image to a square, only x=285–915 survives, but the HTML templates place the logo at x=64–328 for GI and x=64–357 for myPolitics, and start the title at x=64. Those previews therefore remove most of the brand mark and the beginning of the title despite this guidance saying the template keeps them safe; provide a center-square-safe layout or reposition the fixed content.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Right about the fact: the sentence claimed the template keeps the logo and title inside a centred square, and it does not. Changed the text, not the layout, in e8f9ebe: a square-safe title would have to fit in 630 px and be much smaller in the full 1.91:1 preview that Facebook, LinkedIn, X and Discord show. The README now says which part survives a centred square (x 285 to 915), that the default picture keeps its whole logo there (GI 352-848, myPolitics 356-844), that a title picture does not, and to use the default picture for links that mostly travel as thumbnails.
The link preview templates now count Poppins as loaded only when the font load returns a face, so offline they no longer save a picture set in a fallback typeface. The README states which part of a 1200 x 630 picture survives a centred square thumbnail. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
…he avatars flat black - Remove "gi.org.pl" from the GI covers (Facebook, LinkedIn, YouTube, Discord banner), the default link preview and the link preview template (HTML and SVG); the logo now stands alone, centred. - In the GI e-mail signature the logo links to gi.org.pl instead of the separate address line. - GI avatars (master and every platform export): pure #000000 background, no glow; the logomark stays at 1/2 of the circle. - Re-render the GI link preview example and the GI preview sheets; update the README. - No myPolitics file changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2e65ca5249
ℹ️ 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".
|
|
||
| <label>Label above the title (optional)<input id="label" maxlength="40" placeholder="e.g. Wydarzenie"></label> | ||
| <label>Title<textarea id="title" maxlength="140">Tytuł strony</textarea></label> | ||
| <div class="buttons"><button id="savejpg" type="button">Save JPG</button><button id="save" type="button" class="secondary">Save PNG (lossless, larger)</button></div> |
There was a problem hiding this comment.
When either standalone template is opened on a slow Google Fonts connection, or an edit triggers a new font subset fetch, these buttons remain enabled while draw() awaits the font loads and image decode, so clicking them exports the blank or previous canvas before Poppins and the current text have been validated. The fresh evidence in the current code is that neither button is initially disabled, and draw() only updates .disabled after every await; disable them in the markup and at the start of each render, then re-enable only after the current render succeeds.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Valid, but not part of this round: the save buttons behave the same in the myPolitics template (shared generator), and the president asked for no change to any myPolitics file. Fixing only the GI template would make the two diverge. I will fix both templates together in a follow-up once the myPolitics freeze is lifted.
The brand book rule is about the width of the logomark, not its height. Re-render all GI and myPolitics avatar masters and exports with the logomark 540 px wide on 1080 px, and the preview sheets that show avatars. State the rule as a width in the GI and myPolitics usage pages and in the marketing package README. Backgrounds are unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Round 2 (2e65ca5)
After review by the president: "remove the website link from the GI logo templates", "in the avatars use a pure #000 background, as the brand book says", myPolitics "change nothing".
marketing-package/gi/and the README).Round 3 (194184b)
Correction from the president: the avatar rule is half of the avatar's width, not height.
What this adds
A marketing package (pakiet marketingowy) for the social channels of the foundation and of myPolitics, in
gi/docs/visual-identity/marketing-package/, withgi/andmypolitics/subfolders of the same shape:avatar/- a square master (SVG and 1080 x 1080 PNG) with the logomark width at 1/2 of the avatar width, centred, and one PNG per platform: Facebook, Instagram, LinkedIn, YouTube, X (myPolitics only), GitHub organisation, Discord server icon, Luma, Tally. myPolitics has a colour and a universal blue master.cover/- SVG master and PNG for the Facebook cover, LinkedIn page cover, YouTube banner, X header (myPolitics only), Discord server banner and Discord invite background. The primary logo sits alone in the middle.og/- a default link preview (1200 x 630) and a title template: an HTML page that sets the title in Poppins Bold and saves a JPG or PNG in the browser, the same layout as SVG with a text element for Figma or Inkscape, and examples made with the template.email/- the signature logo (on white and on black, at 2x) and an HTML signature snippet with{{NAME}}and{{ROLE}}placeholders.previews/- sheets for checking, not for upload: the circular crop at real sizes, the safe areas drawn over each cover, the covers on a 390 px phone, the signature.README.md- what each file is for, where it goes, which variant and why, the official sizes with links and the date they were checked, how to set up the signature, and who uploads.gi/docs/visual-identity/README.mdand a link inmypolitics/docs/visual-identity/README.md(an absolute GitHub link, becausegi/is not part of the site).Sizes (checked 2026-10-18 on the platforms' help pages)
Decisions
giproject, as asked, with a subfolder per brand. Themypolitics/assets/rule for images is for files shown on the myPolitics site, and nothing here is.<svg>that only sets x, y, width and height. There is no type in the SVG masters.prez-header.svg), scaled to each format: brand red for GI at 2x the template's opacity, universal blue #0092B8 for myPolitics at 3x. At the template's own 4% and 5% the glow cannot be seen on a phone.Pictures
All taken on commit 6ad59bd, rendered with headless Chromium; they are the preview sheets committed in the package. The later commit e8f9ebe changes no picture.
GI avatar - circular crop at 196, 98, 40 and 32 px on light and dark interfaces:
GI covers on a 390 px phone, with the profile picture in place:
GI link preview made with the template:
myPolitics covers on a 390 px phone:
Seven more: the safe-area sheets, the myPolitics avatars and link previews, both signatures
GI covers with each platform's safe areas drawn over them:
myPolitics covers with the safe areas:
myPolitics avatars, colour and universal blue:
myPolitics link previews, colour and universal blue:
E-mail signatures with example text:
Verification
On commit 6ad59bd, locally, unless a line says otherwise:
mkdocs buildcould not be run locally (Python crashes on this machine); the CIbuildcheck of the site passed on e8f9ebe. The only change undermypolitics/is one paragraph with an absolute link.Open points
@codex reviewfor e8f9ebe got "usage limits reached", so the fix commit is not reviewed yet; ask again when the quota is back. Blocks nothing in the files.🤖 Generated by Gieniek with Claude Code