Build: Scalafmt - De-indent extends to get superclass constructor p…

…arams at the right indent

.scalafmt.conf

@@ -20,7 +20,7 @@ indent.binPackDefnSite = 2
 indent.ctorSite = 2
 indent.matchSite = 2
 indent.caseSite = 2
-indent.extendSite = 2
+indent.extendSite = 0
 indent.withSiteRelativeToExtends = 0
 indent.commaSiteRelativeToExtends = 2 # 0 is not allowed
 indent.extraBeforeOpenParenDefnSite = 0
@@ -76,6 +76,7 @@ newlines.afterInfix = keep
 newlines.afterInfixBreakOnNested = false # ??? https://scalameta.org/scalafmt/docs/configuration.html#newlinesafterinfixbreakonnested
 newlines.avoidForSimpleOverflow = [tooLong, punct, slc]
 newlines.avoidInResultType = true
+newlines.avoidAfterYield = false
 newlines.sometimesBeforeColonInMethodReturnType = false
 # newlines.beforeOpenParenDefnSite = keep # TODO not sure https://scalameta.org/scalafmt/docs/configuration.html#newlinesbeforeopenparenxxxsite
 # newlines.beforeOpenParenCallSite = keep # TODO only supported for Scala 3. Also, not sure... https://scalameta.org/scalafmt/docs/configuration.html#newlinesbeforeopenparenxxxsite

src/main/scala/com/raquo/laminar/api/Laminar.scala

@@ -19,32 +19,32 @@ import org.scalajs.dom
 // @TODO[Performance] Check if order of traits matters for quicker access (given trait linearization). Not sure how it's encoded in JS.
 
 trait Laminar
-  extends HtmlTags
-  with HtmlAttrs
-  with HtmlProps
-  with GlobalEventProps
-  with StyleProps
-  with ComplexHtmlKeys
-  with MountHooks // onMountFocus, onMountSet, onMountBind, OnMountCallback, OnUnmountCallback, etc.
-  with AirstreamAliases
-  with LaminarAliases
-  with Implicits {
+extends HtmlTags
+with HtmlAttrs
+with HtmlProps
+with GlobalEventProps
+with StyleProps
+with ComplexHtmlKeys
+with MountHooks // onMountFocus, onMountSet, onMountBind, OnMountCallback, OnUnmountCallback, etc.
+with AirstreamAliases
+with LaminarAliases
+with Implicits {
 
   /** Contains Helpers like `style.px(12)` // returns "12px" */
   object style
-    extends StyleUnitsApi
+  extends StyleUnitsApi
 
   /** Contains ARIA attrs for accessibility */
   object aria
-    extends AriaAttrs
+  extends AriaAttrs
 
   // @TODO[API] Add GlobalEventProps to SVG as well?
   //  Regular event props work fine, but they might be hard to import if you also import svg._
   /** Contains SVG tags and attrs */
   object svg
-    extends SvgTags
-    with SvgAttrs
-    with ComplexSvgKeys
+  extends SvgTags
+  with SvgAttrs
+  with ComplexSvgKeys
 
   //
   // Document & window events

src/main/scala/com/raquo/laminar/api/StyleUnitsApi.scala

@@ -5,10 +5,10 @@ import com.raquo.laminar.defs.styles.units
 import com.raquo.laminar.keys.DerivedStyleBuilder
 
 trait StyleUnitsApi extends DerivedStyleBuilder[String, StyleEncoder]
-  with units.Color[String, StyleEncoder]
-  with units.Length[StyleEncoder, Int]
-  with units.Time[StyleEncoder]
-  with units.Url[StyleEncoder] {
+with units.Color[String, StyleEncoder]
+with units.Length[StyleEncoder, Int]
+with units.Time[StyleEncoder]
+with units.Url[StyleEncoder] {
 
   override protected def styleSetter(value: String): String = value
 

src/main/scala/com/raquo/laminar/nodes/ChildNode.scala

@@ -7,8 +7,8 @@ import scala.annotation.tailrec
 import scala.scalajs.js
 
 trait ChildNode[+Ref <: dom.Node]
-  extends ReactiveNode[Ref]
-  with Modifier[ReactiveElement[dom.Element]] {
+extends ReactiveNode[Ref]
+with Modifier[ReactiveElement[dom.Element]] {
 
   private var _maybeParent: Option[ParentNode.Base] = None
 

src/main/scala/com/raquo/laminar/nodes/ReactiveElement.scala

@@ -13,8 +13,8 @@ import org.scalajs.dom
 import scala.scalajs.js
 
 trait ReactiveElement[+Ref <: dom.Element]
-  extends ChildNode[Ref]
-  with ParentNode[Ref] {
+extends ChildNode[Ref]
+with ParentNode[Ref] {
 
   // @Warning[Fragile] deactivate should not need an isActive guard.
   //  If Laminar starts to cause exceptions here, we need to find the root cause.

src/test/scala/com/raquo/laminar/utils/AsyncUnitSpec.scala

@@ -9,10 +9,10 @@ import scala.scalajs.js
 import scala.util.Try
 
 class AsyncUnitSpec
-  extends AsyncFunSpec
-  with LaminarSpec
-  with AsyncMountSpec
-  with Utils {
+extends AsyncFunSpec
+with LaminarSpec
+with AsyncMountSpec
+with Utils {
 
   // @TODO[Test] Extract this to a more generic place
   def delay[V](value: => V): Future[V] = {

src/test/scala/com/raquo/laminar/utils/LaminarSpec.scala

@@ -12,9 +12,9 @@ import com.raquo.laminar.tags.Tag
 import org.scalactic
 
 trait LaminarSpec
-  extends MountOps
-  with RuleImplicits[Tag.Base, CommentNode, HtmlProp, HtmlAttr, SvgAttr, StyleProp]
-  with EventSimulator {
+extends MountOps
+with RuleImplicits[Tag.Base, CommentNode, HtmlProp, HtmlAttr, SvgAttr, StyleProp]
+with EventSimulator {
   // === On nullable variables ===
   // `root` is nullable because if it was an Option it would be too easy to
   // forget to handle the `None` case when mapping or foreach-ing over it.

src/test/scala/com/raquo/laminar/utils/UnitSpec.scala

@@ -5,8 +5,8 @@ import com.raquo.domtestutils.scalatest.{Matchers, MountSpec}
 import org.scalatest.funspec.AnyFunSpec
 
 class UnitSpec
-  extends AnyFunSpec
-  with LaminarSpec
-  with MountSpec
-  with Matchers
-  with Utils
+extends AnyFunSpec
+with LaminarSpec
+with MountSpec
+with Matchers
+with Utils