Skip to content

feat(slidedeck): rich text, links and nested lists - #35

Merged
TheGreatAxios merged 2 commits into
cl-11886-build-a-native-powerpoint-writer-and-deck-tools-for-buildersfrom
cl-11889-add-rich-text-links-and-nested-lists-to-the-powerpoint
Nov 1, 2026
Merged

TheGreatAxios merged 2 commits into
cl-11886-build-a-native-powerpoint-writer-and-deck-tools-for-buildersfrom
cl-11889-add-rich-text-links-and-nested-lists-to-the-powerpoint

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Closes CL-11889. Stacked on #34.

What changed

  • Text runs: bold, italic, underline, strike, colour (theme name or hex), size, and a link to a URL (http/https/mailto, written percent-encoded) or to another slide. Plain strings still work anywhere runs do.
  • Lists: bulleted or numbered, nested to 5 levels, styled from the master (indents, accent bullets, 1. / a. / i. numbering). Adjacent numbered lists are separated so each starts a new sequence.
  • Validation, all with slide and path:
    • bad URLs and missing slide targets;
    • lists deeper than 5, checked without recursion (10,000 levels returns an issue, not a crash).
  • Line breaks keep their run's formatting. Text boxes take their list style as a parameter, ready for free-form slides.

Verified

  • lint, format:check, typecheck, test and pins:verify pass. The fixture covers every feature; identical bytes, well-formed parts and resolved relationships all pass.
  • python-pptx readback: run properties, list levels and numbering, separator paragraphs, URL and slide link targets.
  • Quick Look shows the formatting.
  • Not verified: that adjacent numbered lists restart in real PowerPoint/Keynote. Quick Look ignores the separator, and no PowerPoint or Keynote is available here.
  • Independent review; all findings fixed.

@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

CL-11889

@corbits-triage

corbits-triage Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Stacked on #34: only this pull request's own changes were checked.

Triage will approve, request review.

@corbits-triage corbits-triage Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Triage requests changes:

  • Conventions: added code matches a code pattern

@corbits-triage
corbits-triage Bot dismissed their stale review October 31, 2026 20:31

Nothing triage asked to change still fails, so this request no longer stands.

@TheGreatAxios
TheGreatAxios force-pushed the cl-11889-add-rich-text-links-and-nested-lists-to-the-powerpoint branch from a71cb77 to 0f19e22 Compare October 31, 2026 23:11
@TheGreatAxios
TheGreatAxios force-pushed the cl-11889-add-rich-text-links-and-nested-lists-to-the-powerpoint branch from a526ce0 to 51ef4b0 Compare November 1, 2026 18:31
@TheGreatAxios TheGreatAxios changed the title feat(pptx): rich text, links and nested lists feat(slidedeck): rich text, links and nested lists Nov 1, 2026
corbits-triage[bot]
corbits-triage Bot previously approved these changes Nov 1, 2026
@TheGreatAxios
TheGreatAxios added this pull request to stack #50 November 1, 2026 18:46
Text takes styled runs (bold, italic, underline, strike, theme or hex
colour, size, links to URLs or slides). Lists nest to five levels,
bulleted or numbered, styled from the master. Validation reports bad
URLs, missing slide targets and too-deep lists without recursion.
@TheGreatAxios
TheGreatAxios added this pull request to the merge queue Nov 1, 2026
Merged via the queue into main with commit d7155e5 Nov 1, 2026
8 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant