1681-engine 0.6.3 → 0.6.5

package/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to this package are written here. The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and versions follow [Semantic Versioning](https://semver.org/); before 1.0 a minor version may break the API.
 
+## 0.6.5
+
+### Changed
+
+- `Group.moveAbove` / `moveBelow` do nothing when the child is already directly above / below the other. Taking it out
+  and putting it back marked the render group's structure changed, so Pixi rebuilt its draw instructions that frame:
+  a game ordering a held weapon against its body every frame rebuilt them every frame.
+
 ## 0.6.3
 
 ### Changed

package/dist/core/Group.js

@@ -1,6 +1,6 @@
-import { Container as r } from "pixi.js";
-import { GameObject as i } from "./gameObject.js";
-class a extends i(r) {
+import { Container as i } from "pixi.js";
+import { GameObject as r } from "./gameObject.js";
+class d extends r(i) {
   get list() {
     return this.children;
   }
@@ -14,15 +14,19 @@ class a extends i(r) {
   remove(t, e = !1) {
     return t.parent === this && this.removeChild(t), e && t.destroy({ children: !0, context: !0 }), this;
   }
-  /** Puts `child` directly below `other` in the draw order. */
+  /**
+   * Puts `child` directly below `other` in the draw order. Already there, it does nothing: taking a child out and
+   * putting it back marks the render group's structure changed, and Pixi rebuilds its draw instructions for that
+   * frame — a held weapon ordered against its body every frame did it every frame.
+   */
   moveBelow(t, e) {
-    return t.parent !== this || e.parent !== this || t === e ? this : (this.removeChild(t), this.addChildAt(t, this.getChildIndex(e)), this);
+    return t.parent !== this || e.parent !== this || t === e ? this : this.getChildIndex(t) === this.getChildIndex(e) - 1 ? this : (this.removeChild(t), this.addChildAt(t, this.getChildIndex(e)), this);
   }
-  /** Puts `child` directly above `other` in the draw order. */
+  /** Puts `child` directly above `other` in the draw order; already there, it does nothing (see `moveBelow`). */
   moveAbove(t, e) {
-    return t.parent !== this || e.parent !== this || t === e ? this : (this.removeChild(t), this.addChildAt(t, this.getChildIndex(e) + 1), this);
+    return t.parent !== this || e.parent !== this || t === e ? this : this.getChildIndex(t) === this.getChildIndex(e) + 1 ? this : (this.removeChild(t), this.addChildAt(t, this.getChildIndex(e) + 1), this);
   }
 }
 export {
-  a as Group
+  d as Group
 };

package/dist/types/core/Group.d.ts

@@ -6,9 +6,13 @@ export declare class Group extends Group_base {
     add(child: PixiContainer | PixiContainer[]): this;
     addAt(child: PixiContainer, index: number): this;
     remove(child: PixiContainer, destroy?: boolean): this;
-    /** Puts `child` directly below `other` in the draw order. */
+    /**
+     * Puts `child` directly below `other` in the draw order. Already there, it does nothing: taking a child out and
+     * putting it back marks the render group's structure changed, and Pixi rebuilds its draw instructions for that
+     * frame — a held weapon ordered against its body every frame did it every frame.
+     */
     moveBelow(child: PixiContainer, other: PixiContainer): this;
-    /** Puts `child` directly above `other` in the draw order. */
+    /** Puts `child` directly above `other` in the draw order; already there, it does nothing (see `moveBelow`). */
     moveAbove(child: PixiContainer, other: PixiContainer): this;
 }
 export {};

package/dist/types/core/Group.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"Group.d.ts","sourceRoot":"","sources":["../../../src/core/Group.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,KAAK,SAAS,IAAI,aAAa,EAAE,MAAM,SAAS,CAAC;;AAGrE,yFAAyF;AACzF,qBAAa,KAAM,SAAQ,UAAqB;IAC9C,IAAI,IAAI,IAAI,aAAa,EAAE,CAE1B;IAED,GAAG,CAAC,KAAK,EAAE,aAAa,GAAG,aAAa,EAAE,GAAG,IAAI;IAKjD,KAAK,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI;IAKhD,MAAM,CAAC,KAAK,EAAE,aAAa,EAAE,OAAO,UAAQ,GAAG,IAAI;IAMnD,6DAA6D;IAC7D,SAAS,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,aAAa,GAAG,IAAI;IAO3D,6DAA6D;IAC7D,SAAS,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,aAAa,GAAG,IAAI;CAM5D"}
+{"version":3,"file":"Group.d.ts","sourceRoot":"","sources":["../../../src/core/Group.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,KAAK,SAAS,IAAI,aAAa,EAAE,MAAM,SAAS,CAAC;;AAGrE,yFAAyF;AACzF,qBAAa,KAAM,SAAQ,UAAqB;IAC9C,IAAI,IAAI,IAAI,aAAa,EAAE,CAE1B;IAED,GAAG,CAAC,KAAK,EAAE,aAAa,GAAG,aAAa,EAAE,GAAG,IAAI;IAKjD,KAAK,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI;IAKhD,MAAM,CAAC,KAAK,EAAE,aAAa,EAAE,OAAO,UAAQ,GAAG,IAAI;IAMnD;;;;OAIG;IACH,SAAS,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,aAAa,GAAG,IAAI;IAQ3D,+GAA+G;IAC/G,SAAS,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,aAAa,GAAG,IAAI;CAO5D"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "1681-engine",
-  "version": "0.6.3",
+  "version": "0.6.5",
   "description": "A 2D game engine on PixiJS 8 for any art style: scenes and a fast ECS, a fixed-step loop, asset loading, Tiled maps, sprite animation, input actions, camera follow, audio and deterministic, server-replayable movement, plus Dither, an optional pixel-art module (pixel-perfect effects, dithered lighting).",
   "type": "module",
   "types": "./dist/types/index.d.ts",