fix bolding

ci/release/changelogs/next.md

@@ -35,6 +35,8 @@
   So you can run `go install oss.terrastruct.com/d2@latest` to install from source
   now.
   [#290](https://github.com/terrastruct/d2/pull/290)
+- Container default font styling is no longer bold. Everything used to look too bold.
+  [#358](https://github.com/terrastruct/d2/pull/358)
 - `BROWSER=0` now works to disable opening a browser on `--watch`.
   [#311](https://github.com/terrastruct/d2/pull/311)
 

d2exporter/export.go

@@ -95,6 +95,8 @@ func toShape(obj *d2graph.Object, theme *d2themes.Theme) d2target.Shape {
 	shape.Height = int(obj.Height)
 
 	text := obj.Text()
+	shape.Bold = text.IsBold
+	shape.Italic = text.IsItalic
 	shape.FontSize = text.FontSize
 
 	applyStyles(shape, obj)

d2graph/d2graph.go

@@ -336,7 +336,7 @@ func (obj *Object) GetFill(theme *d2themes.Theme) string {
 	} else if obj.IsSequenceDiagramNote() {
 		return theme.Colors.Neutrals.N7
 	} else if obj.IsSequenceDiagramGroup() {
-		sd := obj.outerSequenceDiagram()
+		sd := obj.OuterSequenceDiagram()
 		// Alternate
 		if (level-int(sd.Level()))%2 == 0 {
 			return theme.Colors.Neutrals.N7
@@ -428,10 +428,14 @@ func (obj *Object) AbsIDArray() []string {
 }
 
 func (obj *Object) Text() *d2target.MText {
+	isBold := !obj.IsContainer()
 	fontSize := d2fonts.FONT_SIZE_M
-	if obj.IsContainer() && !obj.Parent.IsSequenceDiagram() {
-		// sequence diagram children (aka, actors) shouldn't have the container font size
-		fontSize = obj.Level().LabelSize()
+	if obj.OuterSequenceDiagram() == nil {
+		if obj.IsContainer() {
+			fontSize = obj.Level().LabelSize()
+		}
+	} else {
+		isBold = false
 	}
 	if obj.Attributes.Style.FontSize != nil {
 		fontSize, _ = strconv.Atoi(obj.Attributes.Style.FontSize.Value)
@@ -443,7 +447,7 @@ func (obj *Object) Text() *d2target.MText {
 	return &d2target.MText{
 		Text:     obj.Attributes.Label.Value,
 		FontSize: fontSize,
-		IsBold:   !obj.IsContainer(),
+		IsBold:   isBold,
 		IsItalic: false,
 		Language: obj.Attributes.Language,
 		Shape:    obj.Attributes.Shape.Value,
@@ -742,7 +746,7 @@ func (obj *Object) Connect(srcID, dstID []string, srcArrow, dstArrow bool, label
 	src := srcObj.EnsureChild(srcID)
 	dst := dstObj.EnsureChild(dstID)
 
-	if src.outerSequenceDiagram() != dst.outerSequenceDiagram() {
+	if src.OuterSequenceDiagram() != dst.OuterSequenceDiagram() {
 		return nil, errors.New("connections within sequence diagrams can connect only to other objects within the same sequence diagram")
 	}
 

d2graph/seqdiagram.go

@@ -6,7 +6,7 @@ func (obj *Object) IsSequenceDiagram() bool {
 	return obj != nil && obj.Attributes.Shape.Value == d2target.ShapeSequenceDiagram
 }
 
-func (obj *Object) outerSequenceDiagram() *Object {
+func (obj *Object) OuterSequenceDiagram() *Object {
 	for obj != nil {
 		obj = obj.Parent
 		if obj.IsSequenceDiagram() {
@@ -19,7 +19,7 @@ func (obj *Object) outerSequenceDiagram() *Object {
 // groups are objects in sequence diagrams that have no messages connected
 // and does not have a note as a child (a note can appear within a group, but it's a child of an actor)
 func (obj *Object) IsSequenceDiagramGroup() bool {
-	if obj.outerSequenceDiagram() == nil {
+	if obj.OuterSequenceDiagram() == nil {
 		return false
 	}
 	for _, e := range obj.Graph.Edges {
@@ -38,7 +38,7 @@ func (obj *Object) IsSequenceDiagramGroup() bool {
 
 // notes are descendant of actors with no edges and no children
 func (obj *Object) IsSequenceDiagramNote() bool {
-	if obj.outerSequenceDiagram() == nil {
+	if obj.OuterSequenceDiagram() == nil {
 		return false
 	}
 	return !obj.hasEdgeRef() && !obj.ContainsAnyEdge(obj.Graph.Edges) && len(obj.ChildrenArray) == 0

d2layouts/d2sequence/layout_test.go

@@ -6,6 +6,7 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
+
 	"oss.terrastruct.com/d2/d2compiler"
 	"oss.terrastruct.com/d2/d2graph"
 	"oss.terrastruct.com/d2/d2layouts/d2sequence"