Deck: fix 5 layout bugs + fade transitions - #3
Open
pixie-chan wants to merge 2 commits into
Open
Conversation
Layout fixes (no wording/numbers touched, pure geometry + contrast): - S2: remove -2deg rotate on the 3 stat cards so orange bars line up in a straight row - S3 (research machine): nudge title down 0.07in so it no longer hugs the top edge; brighten card footer (8A8A8A -> B0B0B0) so source/freshness is readable - S3 (solution): drop bullet font 11->10.5 + lineSpacing 22->20 to kill dangling-word widows - S6 (moat): shrink advisory quote 13.5->12.5pt so it fits its box (was cut off) - S8 (market): pull last rail line up + shrink 12->11pt so 'record' no longer clips the box Animations: inject_transitions.py adds a medium fade transition to all 13 slides post-build (pptxgenjs cannot author transitions natively). Run it after node build-krishi-setu.js. Verified: rebuilt, 13 slides, gate tokens intact (FF5A1F/0A0A0A/Arial Black), 0 em dash, 13 transitions injected, all 5 fixes confirmed by render.
User directive: revert the black/white/orange theme back to green. Content, numbers, 13 slides, layout fixes, and transitions all preserved. Palette swap (all hardcoded tones recolored, no text/number touched): - near-black 0A0A0A -> forest green 1F3D26 (dark backgrounds + cards) - hazard orange FF5A1F -> moss green 97BC62 (accent bars, quote, circles) - gray/edge tones -> muted sage greens; card fills -> pale greens - extrusion shadows 2A2A2A -> deep green 142B1A; strips 555555 -> 3E5C42 Gate updated: check-deck.sh now asserts 97BC62 + 1F3D26 instead of FF5A1F + 0A0A0A. Verified: gate PASSES (13 slides, 0 em dash, 0 banned words), 13 fade transitions re-injected after gate rebuild.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Layout-only pass, zero wording/number changes.
Fixes (all verified by rendering the rebuilt pptx):
Added: fade slide transitions on all 13 slides via scaffold/deck/inject_transitions.py (run after node build-krishi-setu.js). pptxgenjs can't do animations natively, so this is a post-build step.
Gate check passes: 13 slides, FF5A1F + 0A0A0A + Arial Black present, 0 em dashes, 0 banned words.