Merge pull request #586 from alixander/empty-text

fix
This commit is contained in:
Alexander Wang 2023-01-01 01:49:12 -08:00 committed by GitHub
commit 711b7c47a9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -1014,10 +1014,6 @@ func getMarkdownDimensions(mtexts []*d2target.MText, ruler *textmeasure.Ruler, t
return d2target.NewTextDimensions(width, height), nil
}
if strings.TrimSpace(t.Text) == "" {
return d2target.NewTextDimensions(100, 100), nil
}
return nil, fmt.Errorf("text not pre-measured and no ruler provided")
}
@ -1071,6 +1067,22 @@ func (g *Graph) SetDimensions(mtexts []*d2target.MText, ruler *textmeasure.Ruler
if obj.Attributes.Height != nil {
desiredHeight, _ = strconv.Atoi(obj.Attributes.Height.Value)
}
if obj.Attributes.Label.Value == "" &&
obj.Attributes.Shape.Value != d2target.ShapeImage &&
obj.Attributes.Shape.Value != d2target.ShapeSQLTable &&
obj.Attributes.Shape.Value != d2target.ShapeClass {
obj.Width = DEFAULT_SHAPE_PADDING
obj.Height = DEFAULT_SHAPE_PADDING
if desiredWidth != 0 {
obj.Width = float64(desiredWidth)
}
if desiredHeight != 0 {
obj.Height = float64(desiredHeight)
}
continue
}
shapeType := strings.ToLower(obj.Attributes.Shape.Value)
labelDims, err := obj.GetLabelSize(mtexts, ruler, fontFamily)