diff --git a/d2cli/main.go b/d2cli/main.go index 24a6387d2..7b91422ad 100644 --- a/d2cli/main.go +++ b/d2cli/main.go @@ -372,7 +372,7 @@ func compile(ctx context.Context, ms *xmain.State, plugin d2plugin.Plugin, rende p := pptx.NewPresentation(rootName, description, rootName, username, version.OnlyNumbers()) boardIdToIndex := buildBoardIDToIndex(diagram, nil, nil) - svg, err := renderPPTX(ctx, ms, p, plugin, renderOpts, ruler, outputPath, page, diagram, nil, boardIdToIndex) + svg, err := renderPPTX(ctx, ms, p, plugin, renderOpts, ruler, outputPath, page, diagram, nil, "", boardIdToIndex) if err != nil { return nil, false, err } @@ -785,14 +785,24 @@ func renderPDF(ctx context.Context, ms *xmain.State, plugin d2plugin.Plugin, opt return svg, nil } -func renderPPTX(ctx context.Context, ms *xmain.State, presentation *pptx.Presentation, plugin d2plugin.Plugin, opts d2svg.RenderOpts, ruler *textmeasure.Ruler, outputPath string, page playwright.Page, diagram *d2target.Diagram, boardPath []string, boardIdToIndex map[string]int) ([]byte, error) { - var currBoardPath []string - // Root board doesn't have a name, so we use the output filename +func renderPPTX(ctx context.Context, ms *xmain.State, presentation *pptx.Presentation, plugin d2plugin.Plugin, opts d2svg.RenderOpts, ruler *textmeasure.Ruler, outputPath string, page playwright.Page, diagram *d2target.Diagram, boardPath []pptx.BoardTitle, boardType string, boardIDToIndex map[string]int) ([]byte, error) { + var curr pptx.BoardTitle if diagram.Name == "" { - currBoardPath = append(boardPath, "root") + curr = pptx.BoardTitle{ + Name: "root", + BoardID: "root", + } } else { - currBoardPath = append(boardPath, diagram.Name) + prev := boardPath[len(boardPath)-1] + curr = pptx.BoardTitle{ + Name: diagram.Name, + BoardID: strings.Join([]string{prev.BoardID, boardType, diagram.Name}, "."), + } } + if pageNum, ok := boardIDToIndex[curr.BoardID]; ok { + curr.LinkToSlide = pageNum + 1 + } + currBoardPath := append(boardPath, curr) var svg []byte if !diagram.IsFolderOnly { @@ -867,7 +877,7 @@ func renderPPTX(ctx context.Context, ms *xmain.State, presentation *pptx.Present if err != nil || key.Path[0].Unbox().ScalarString() != "root" { // External link link.ExternalUrl = shape.Link - } else if pageNum, ok := boardIdToIndex[shape.Link]; ok { + } else if pageNum, ok := boardIDToIndex[shape.Link]; ok { // Internal link link.SlideIndex = pageNum + 1 } @@ -875,19 +885,19 @@ func renderPPTX(ctx context.Context, ms *xmain.State, presentation *pptx.Present } for _, dl := range diagram.Layers { - _, err := renderPPTX(ctx, ms, presentation, plugin, opts, ruler, "", page, dl, currBoardPath, boardIdToIndex) + _, err := renderPPTX(ctx, ms, presentation, plugin, opts, ruler, "", page, dl, currBoardPath, LAYERS, boardIDToIndex) if err != nil { return nil, err } } for _, dl := range diagram.Scenarios { - _, err := renderPPTX(ctx, ms, presentation, plugin, opts, ruler, "", page, dl, currBoardPath, boardIdToIndex) + _, err := renderPPTX(ctx, ms, presentation, plugin, opts, ruler, "", page, dl, currBoardPath, SCENARIOS, boardIDToIndex) if err != nil { return nil, err } } for _, dl := range diagram.Steps { - _, err := renderPPTX(ctx, ms, presentation, plugin, opts, ruler, "", page, dl, currBoardPath, boardIdToIndex) + _, err := renderPPTX(ctx, ms, presentation, plugin, opts, ruler, "", page, dl, currBoardPath, STEPS, boardIDToIndex) if err != nil { return nil, err } diff --git a/lib/pptx/pptx.go b/lib/pptx/pptx.go index 9bc885fd8..66d02e317 100644 --- a/lib/pptx/pptx.go +++ b/lib/pptx/pptx.go @@ -16,11 +16,17 @@ import ( "fmt" "image/png" "os" - "strings" "text/template" "time" ) +type BoardTitle struct { + LinkID string + Name string + BoardID string + LinkToSlide int +} + type Presentation struct { Title string Description string @@ -32,8 +38,9 @@ type Presentation struct { Slides []*Slide } + type Slide struct { - BoardPath []string + BoardTitle []BoardTitle Links []*Link Image []byte ImageId string @@ -76,7 +83,7 @@ func NewPresentation(title, description, subject, creator, d2Version string) *Pr } } -func (p *Presentation) AddSlide(pngContent []byte, boardPath []string) (*Slide, error) { +func (p *Presentation) AddSlide(pngContent []byte, titlePath []BoardTitle) (*Slide, error) { src, err := png.Decode(bytes.NewReader(pngContent)) if err != nil { return nil, fmt.Errorf("error decoding PNG image: %v", err) @@ -127,7 +134,7 @@ func (p *Presentation) AddSlide(pngContent []byte, boardPath []string) (*Slide, left := (SLIDE_WIDTH - width) / 2 slide := &Slide{ - BoardPath: make([]string, len(boardPath)), + BoardTitle: make([]BoardTitle, len(titlePath)), ImageId: fmt.Sprintf("slide%dImage", len(p.Slides)+1), Image: pngContent, ImageWidth: width, @@ -137,7 +144,10 @@ func (p *Presentation) AddSlide(pngContent []byte, boardPath []string) (*Slide, ImageScaleFactor: float64(width) / srcWidth, } // it must copy the board path to avoid slice reference issues - copy(slide.BoardPath, boardPath) + for i := 0; i < len(titlePath); i++ { + titlePath[i].LinkID = fmt.Sprintf("navLink%d", i) + slide.BoardTitle[i] = titlePath[i] + } p.Slides = append(p.Slides, slide) return slide, nil @@ -215,11 +225,11 @@ func (p *Presentation) SaveTo(filePath string) error { titles := make([]string, 0, len(p.Slides)) for _, slide := range p.Slides { - titles = append(titles, strings.Join(slide.BoardPath, "/")) + titles = append(titles, slide.BoardTitle[len(slide.BoardTitle)-1].BoardID) } err = addFileFromTemplate(zipWriter, "docProps/app.xml", APP_XML, AppXmlContent{ SlideCount: len(p.Slides), - TitlesOfPartsCount: len(p.Slides) + 3, + TitlesOfPartsCount: len(p.Slides) + 3, // + 3 for fonts and theme D2Version: p.D2Version, Titles: titles, }) @@ -291,6 +301,13 @@ func getSlideXmlRelsContent(imageID string, slide *Slide) RelsSlideXmlContent { }) } + for _, t := range slide.BoardTitle { + content.Links = append(content.Links, RelsSlideXmlLinkContent{ + RelationshipID: t.LinkID, + SlideIndex: t.LinkToSlide, + }) + } + return content } @@ -308,9 +325,14 @@ type SlideLinkXmlContent struct { Height int } +type SlideXmlTitlePathContent struct { + Name string + RelationshipID string +} + type SlideXmlContent struct { Title string - TitlePrefix string + TitlePrefix []SlideXmlTitlePathContent Description string HeaderHeight int ImageID string @@ -323,17 +345,18 @@ type SlideXmlContent struct { } func getSlideXmlContent(imageID string, slide *Slide) SlideXmlContent { - boardPath := slide.BoardPath - boardName := boardPath[len(boardPath)-1] - prefixPath := boardPath[:len(boardPath)-1] - var prefix string - if len(prefixPath) > 0 { - prefix = strings.Join(prefixPath, " / ") + " / " + title := make([]SlideXmlTitlePathContent, len(slide.BoardTitle)-1) + for i := 0; i < len(slide.BoardTitle)-1; i++ { + t := slide.BoardTitle[i] + title[i] = SlideXmlTitlePathContent{ + Name: t.Name, + RelationshipID: t.LinkID, + } } content := SlideXmlContent{ - Title: boardName, - TitlePrefix: prefix, - Description: strings.Join(boardPath, " / "), + Title: slide.BoardTitle[len(slide.BoardTitle)-1].Name, + TitlePrefix: title, + Description: slide.BoardTitle[len(slide.BoardTitle)-1].BoardID, HeaderHeight: HEADER_HEIGHT, ImageID: imageID, ImageLeft: slide.ImageLeft, diff --git a/lib/pptx/template.pptx b/lib/pptx/template.pptx index f87fcd9e2..783f7014c 100644 Binary files a/lib/pptx/template.pptx and b/lib/pptx/template.pptx differ diff --git a/lib/pptx/templates/slide.xml b/lib/pptx/templates/slide.xml index d138670ea..eace2f28e 100644 --- a/lib/pptx/templates/slide.xml +++ b/lib/pptx/templates/slide.xml @@ -75,9 +75,19 @@ - {{if .TitlePrefix}} - {{.TitlePrefix}} - {{end}} + + {{range .TitlePrefix}} + + + + + {{.Name}} + + / + {{end}} + {{.Title}}