feat: add KnowAll AI presentation suite (Company Overview, AI Discovery, Agentic Delivery, CISP)#47
Open
BenGWeeks wants to merge 27 commits into
Open
feat: add KnowAll AI presentation suite (Company Overview, AI Discovery, Agentic Delivery, CISP)#47BenGWeeks wants to merge 27 commits into
BenGWeeks wants to merge 27 commits into
Conversation
- Add knowall-overview.html presentation with 11 slides - Create presentations index page with styled card layout - Presentations accessible at /presentations/ Co-authored-by: Ben Weeks <Ben.Weeks@KnowAll.ai>
- Add :focus-visible/:focus-within styles mirroring hover states on cards, CTA button, and links in both files - Add role="region" and descriptive aria-label to all 12 slides - Mark decorative emoji icons aria-hidden (logo, card, feature-box and contact icons); give the Pennie robot a role="img" label and add accessible labels to the CTA and GitHub links - Add meta description, Open Graph, and Twitter Card tags plus favicon to both files - Pin reveal.js 4.5.0 CDN assets with Subresource Integrity hashes and crossorigin="anonymous" - Fix WCAG AA contrast on green slides: #e0e0e0 on the green gradient was 1.02:1 (white 1.35:1), so text on green areas is now dark (9:1+ against both gradient stops); cards on green get a darker fill - Extract the grid-pattern SVG data URI and repeated magic numbers (heading sizes, card padding/radius, icon sizes, timeline width, focus ring) into CSS custom properties Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Rework the reveal.js overview deck and presentations index to match the KnowAll brand: lime #9DFE0A on charcoal #333732, Playfair Display Black headings with Mulish body (per the house PowerPoint theme), the textured green header band with chevron notch, and the real KnowAll.ai robot logo instead of an emoji placeholder. - Replace emoji icons with inline Lucide SVG icons throughout - Add generated on-brand images: night-time robot title hero and a Pennie mascot illustration (gpt-image-1, stored under public/presentations/images/) - Correct T-Minus-15 to the five house stages: Prep, Design, Engineer, Test, Operate (was four, missing Test) - Update Azure AD reference to Microsoft Entra ID - Make the closing CTA 'Book a Discovery Day' to match the service offering, with mail/web contact row - Restyle /presentations/ index from off-brand purple gradient to the house palette - Refresh docs/screenshots/presentation-*.png with the new slides Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…hots Added padding-right: 200px to h1 and h2 elements to ensure slide titles don't extend into the top-right KnowAll.AI logo area. Regenerated all 12 presentation screenshots at 1280x720 to reflect the fix. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…shots The previous screenshot regeneration captured 404 pages (static server was rooted incorrectly), and padding-right alone did not stop long centred titles from overflowing under the top-right logo. Reduce h1/h2 sizes (4em->3.1em, 3em->2.3em) so titles clear the logo, and regenerate all 12 docs/screenshots from the live deck (verified non-blank, no title/logo overlap). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Reveal.js deck mirroring the corporate "KnowAll AI Overview" PDF (slides 1-19, appendix excluded), reusing the existing branded template style (brand green, wave/green-gradient dividers, top-right logo, page counter). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The Agentic Delivery "Day rates" table and the CISP "Service tiers & rates" and "Response & resolution targets" tables contained invented figures not present in the source deck; remove them, keeping the real fixed prices (EUR 2,000 / 8,000 / 25k-100k) and qualitative tier descriptions. Rename the two overview cards on the index so they are distinguishable: the existing short deck is "Agentic AI Solutions"; the new full corporate deck is "Company Overview". Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
BenGWeeks
marked this pull request as ready for review
July 9, 2026 23:04
The four new suite decks and index were added without running Prettier, which fails the CI Format check (prettier --check .). Reformat them; whitespace-only, verified no rendered content changed. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
Consolidates PR #45 into the presentation suite: adds the Self-Improving Agents and Agent Memory (Neo4j) slides to knowall-overview.html and renumbers to 14 slides. Supersedes #45 and #2 so the whole suite lands as one PR against master. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
3 tasks
There was a problem hiding this comment.
Pull request overview
Adds a new /presentations/ area under public/ containing several self-contained Reveal.js decks plus an index page, establishing the initial “presentation suite” foundation for the KnowAll AI website.
Changes:
- Added a presentations index page at
/presentations/that links to each deck. - Added multiple Reveal.js HTML decks (overview + full overview + service decks) with shared branding/styling and slide configuration.
- Included deck-specific marketing/service content for AI Discovery, Agentic Delivery, and CISP.
Reviewed changes
Copilot reviewed 6 out of 18 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| public/presentations/index.html | Adds a landing page that lists and links out to each presentation deck. |
| public/presentations/knowall-overview.html | Adds a shorter “Agentic AI Solutions” overview Reveal.js deck. |
| public/presentations/knowall-overview-full.html | Adds the full “Company Overview” Reveal.js deck with expanded content sections. |
| public/presentations/ai-discovery.html | Adds the “AI Discovery” Reveal.js service deck. |
| public/presentations/agentic-delivery.html | Adds the “Agentic Delivery” Reveal.js service deck. |
| public/presentations/cisp.html | Adds the “CISP” Reveal.js service deck. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Empty commit to fire a pull_request:synchronize event now that the PR targets master, so the CI workflow (Lint/Format/Type Check/Build/Tests) runs — earlier pushes happened before the base was retargeted. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
Addresses Copilot review: external-target links (target=_blank) now carry rel=noopener noreferrer to prevent reverse-tabnabbing. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
The old docs/screenshots/presentation-*.png were stale: 12 shots for the now-14-slide overview deck (missing the folded-in Self-Improving Agents and Neo4j memory slides) and none for the other four suite decks. Add scripts/screenshot-presentations.mjs (Playwright) to capture every slide of all five decks at 1280x720 with deck-based names, and regenerate the full set: knowall-overview (14), knowall-overview-full (19), ai-discovery (6), agentic-delivery (7), cisp (6). Remove the stale files. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
When the Self-Improving Agents and Neo4j slides were inserted at 6-7, the aria-labels were renumbered to /14 but the <!-- Slide N: --> HTML comment markers below them were not, leaving comments (Security, Microsoft, etc.) two behind their aria-labels. Renumber the six trailing markers so the comments run 1-14 in sync. Addresses Copilot review. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
Uncommitted work rescued from the design session's scratchpad worktree: - knowall-overview.html: +413/-79 further polish on the house-style deck - images/logos/: real partner/vendor logos (Azure, Power Platform, Copilot Studio, Fabric, Dataverse, SharePoint, AI Foundry, Anthropic, Eir, XMA) - images/requirements-analyst.png: generated Pennie artwork - images/tminus15-book.png: T-Minus-15 book cover Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
…to feature/presentation-suite # Conflicts: # docs/screenshots/presentation-01.png # docs/screenshots/presentation-02.png # docs/screenshots/presentation-03.png # docs/screenshots/presentation-04.png # docs/screenshots/presentation-05.png # docs/screenshots/presentation-06.png # docs/screenshots/presentation-07.png # docs/screenshots/presentation-08.png # docs/screenshots/presentation-09.png # docs/screenshots/presentation-10.png # docs/screenshots/presentation-11.png # docs/screenshots/presentation-12.png # public/presentations/index.html # public/presentations/knowall-overview.html
The polished house-style overview (feature/presentations-polish) predated the #45 fold-in, so the Self-Improving Agents and Agent Memory (Neo4j) slides existed only in the old template. Recreate both in the house style (band + chevron, Playfair/Mulish, lime chip cards, github-link pill) after the T-Minus-15 slide, add git-pull-request/database/sprout icons to the Lucide sprite, and renumber aria-labels to a consistent 1-15 (including the previously unnumbered Meet-the-team slide) matching reveal's counter. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
Rebuild knowall-overview-full (19 slides), ai-discovery (6), agentic-delivery (7) and cisp (6) on the polished house template rescued from feature/presentations-polish — lime #9DFE0A on charcoal, Playfair Display + Mulish, textured green band with chevron notch, hero-night cover artwork, Lucide icon cards, stage flows, checklists and fact pills. List all five decks on the house-styled presentations index (with rel=noopener), serve screenshots over a local HTTP server so absolute /images paths resolve, and regenerate all 53 slide screenshots. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
Addresses Copilot review: server.close() is asynchronous, so the script could log completion before the server actually closed. Await it via a promise for deterministic teardown. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
Audit follow-ups from the design-session conversation review: - Waves: the band now uses the real aerial-wave brand asset from 'KnowAll AI Overview.pptx' (Sales and Marketing / Teams), cropped of its baked-in notch and optimised to presentations/images/wave-band.jpg — the site-wide green-bg.png only carried texture in its top third, so the wave motif was invisible in the 250px band. - Benefit stats: slide 13 no longer presents invented percentages as client experience. Cards are grounded in real engagement facts: 4-day Proto-Sprint, the ≈90% straight-through invoice-automation target, 4–8 week case-study deliveries, and 24/7 availability. - Footer: 'Presentations' link added to the company column; /presentations clean URL served via a next.config rewrite (public/ has no directory index; the bare path previously 308→404'd). - Cosmetics: stale 'Meet Pennie' comments renamed to Preppie; orphaned pennie.png removed; master merged in. - All 53 screenshots regenerated with the wave band. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
… tooltips Addresses Copilot review on #47: - /presentations was served via a rewrite, leaving the browser at the extensionless URL where the page's relative deck/asset links resolved against the site root (404). Use a redirect to /presentations/index.html so relatives resolve under /presentations/. Verified end-to-end. - Meet-the-team agent skill tooltips were hover-only; agents are now tabbable (tabindex=0) and the tooltip shows on :focus-visible / :focus-within with a visible focus ring. Verified via Playwright. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K331ToqUrk97AV12Tq4Dsy
Comment on lines
+28
to
+35
| <li> | ||
| <a | ||
| href="/presentations" | ||
| className="text-gray-400 hover:text-white transition-colors" | ||
| > | ||
| Presentations | ||
| </a> | ||
| </li> |
Comment on lines
+41
to
+45
| const path = normalize(decodeURIComponent(new URL(req.url, 'http://x').pathname)); | ||
| const file = join(publicDir, path); | ||
| if (!file.startsWith(publicDir)) throw new Error('traversal'); | ||
| const body = await readFile(file); | ||
| res.writeHead(200, { 'content-type': MIME[extname(file)] || 'application/octet-stream' }); |
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.
What
The full KnowAll AI presentation suite in the house style — lime
#9DFE0Aon charcoal, Playfair Display + Mulish, the textured green band with chevron notch, the hero-night cover artwork and real partner logos. Five decks sharing one template, plus a house-styled presentations index.This consolidates the earlier stacked presentation PRs (#2 overview, #45 agent slides) and the rescued design-session polish (
feature/presentations-polish) into a single PR againstmaster.Decks
knowall-overview.html— Agentic AI Solutions (15 slides), the polished house deck including the Self-Improving Agents and Agent Memory (Neo4j) slides re-created in house style.knowall-overview-full.html— faithful Company Overview (19 slides): about, mission & vision, all five service offerings with rates, two case studies, T-Minus-15, Zaplie, AI agents & Copilots.ai-discovery.html— The AI Discovery (6 slides).agentic-delivery.html— The Agentic Delivery (7 slides).cisp.html— CISP (6 slides).index.html— house-styled index listing all five decks (rel="noopener noreferrer"on all links).Each deck is a self-contained static HTML file.
scripts/screenshot-presentations.mjscaptures every slide over a local HTTP server (so/images/...paths resolve) — all 53 screenshots regenerated.Supersedes
Screenshots
Test plan
npm run dev, open/presentations/, follow each deck link, step through all slides.docs/screenshots/; overflow-checked programmatically (child rects vs container).Fixes #46