Repository navigation
Conversation
…de pptx - title: echte Npuls-titelachtergrond (Slide15) en volledig horizontaal logo op blauw - beeldmerk: het echte stippenring-beeldmerk in plaats van het indicatieve - nieuw slidetype timeline: blokken op schaal met tijden, uitleg, tags en legenda - edit.py: bestaande pptx lezen, tekst vervangen, logo wisselen, afbeeldingen vervangen, slides klonen en verplaatsen (python-pptx) - assets met beeldmerk en horizontaal logo (wit en zwart); versie 0.2.0 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…plaats van uit de skill De skill draait alleen in clidev-presentaties, waar de andere Npuls-assets al staan. Dat maakt de skill 0,5 MB lichter. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Contributor
Author
|
Hoi @StevenRamondt, fijn dat je meekijkt! 🙂 Even wat context, zodat je weet waar je naar kijkt:
Voor deze review is dus vooral de skill interessant, niet de inhoud van de DAIR-decks. Alle opmerkingen, vragen of ideeën zijn welkom, en bedankt alvast voor de moeite! |
This branch has not been deployed
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
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.
Wat
De
powerclaude-skill (nu 0.2.0) maakt een deck in één keer goed, zodat een nabewerking niet meer nodig is. Aanleiding: voor het DAIR-workshopdeck moesten we nabuild.mjsnog drie scripts draaien om de titelslide, de tijdlijn en het beeldmerk goed te krijgen.title: opblauw(de standaard) staat de echte Npuls-titelachtergrond (Slide15.PNG, blauw met bogen) met het volledige horizontale logo linksboven. De tagline staat dan links onder het logo.npuls: falsezet dat uit.beeldmerk-ring.json) in plaats van het indicatieve SVG.timeline: blokken op schaal met tijden, titel, uitleg, optionele tag-pill en legenda.scripts/edit.py(python-pptx) voor een bestaande pptx:dump,replace(faalt als een zoekterm niet voorkomt),beeldmerk,media,kloon,verplaats. Voor een nieuw deck blijftbuild.mjsde route.SKILL.md: stap 6 en de vaste regels aangepast;slidetypes.mden het voorbeelddeck bevattentimeline.Let op: afhankelijkheid
De logo's en het beeldmerk staan niet in de skill maar in
public/npuls/logos/van clidev-presentaties, net als de illustraties. Die PR moet dus eerst of tegelijk mee: cedanl/clidev-presentaties (branchfeat/npuls-logos). Zonder die map geeftbuild.mjsgeen titelslide met logo.Getest
edit.pyop het originele DAIR-voorstel: tekst vervangen, afbeeldingen vervangen, een slide klonen en verplaatsen; resultaat gerenderd.edit.py beeldmerkvervangt het beeldmerk op 26 slides van het uitvoeringsdeck.Verandert voor bestaande decks
Op de tijdlijn en in de titelslide is de tekstkleur nu die van de toegestane Npuls-combinaties (bijvoorbeeld roze op blauwe blokken). Bestaande decks veranderen niet tot je ze opnieuw bouwt.
🤖 Generated with Claude Code