Faster 3D terrain in the starters, and fixes for the terrain replacing the ground

.circleci/config.yml

@@ -33,12 +33,38 @@ jobs:
       - checkout
       - run: npm install
       - run: git clone git@github.com:4ian/GDevelop.git --depth 1
-      - run: cd GDevelop/newIDE/app && npm install
+      - run:
+          name: Generate the cache key of GDevelop dependencies
+          # Computed before the install, which can rewrite the lock files.
+          command: cat GDevelop/newIDE/app/package-lock.json GDevelop/GDJS/package-lock.json | shasum -a 256 > gdevelop-dependencies-checksum.txt
+      - restore_cache:
+          keys:
+            - gdevelop-dependencies-v1-{{ checksum "gdevelop-dependencies-checksum.txt" }}
+            # fallback to using the latest cache if no exact match is found
+            - gdevelop-dependencies-v1-
+      - run: cd GDevelop/newIDE/app && npm install --no-audit --no-fund
+      - save_cache:
+          paths:
+            - GDevelop/newIDE/app/node_modules
+            - GDevelop/GDJS/node_modules
+          key: gdevelop-dependencies-v1-{{ checksum "gdevelop-dependencies-checksum.txt" }}
       - persist_to_workspace:
           root: .
           paths:
             - node_modules # For npm install.
-            - GDevelop/newIDE/app # As we have a dependency on GDevelop sources.
+            # What scripts/lib/GDevelopCoreLoader.js loads from GDevelop (not the
+            # whole newIDE/app: its node_modules are more than 1GB, long to persist
+            # and to attach in every job).
+            # Without it, the files of GDevelop would take the "type" of the
+            # package.json of this repository.
+            - GDevelop/newIDE/app/package.json
+            - GDevelop/newIDE/app/public/libGD.js
+            - GDevelop/newIDE/app/public/libGD.wasm
+            - GDevelop/newIDE/app/resources/GDJS
+            - GDevelop/newIDE/app/src
+            # The packages required by newIDE/app/src/JsExtensionsLoader.
+            - GDevelop/newIDE/app/node_modules/lodash
+            - GDevelop/newIDE/app/node_modules/node-require-function
   tests:
     docker:
       - image: cimg/node:20.13.1
@@ -77,6 +103,11 @@ jobs:
       - attach_workspace:
           at: .
       - aws-cli/setup
+      - run:
+          # The deploy uploads more than a thousand small files: the default of
+          # 10 concurrent requests makes it take minutes.
+          name: Upload more files in parallel
+          command: aws configure set default.s3.max_concurrent_requests 64
       - run: npm run deploy -- --cf-zoneid $CLOUDFLARE_ZONE_ID --cf-token $CLOUDFLARE_TOKEN
   deploy-staging:
     docker:
@@ -94,6 +125,11 @@ jobs:
       - attach_workspace:
           at: .
       - aws-cli/setup
+      - run:
+          # The deploy uploads more than a thousand small files: the default of
+          # 10 concurrent requests makes it take minutes.
+          name: Upload more files in parallel
+          command: aws configure set default.s3.max_concurrent_requests 64
       - run: npm run deploy -- --staging --cf-zoneid $CLOUDFLARE_ZONE_ID --cf-token $CLOUDFLARE_TOKEN
 
   # Run the gameplay tests of the example games with a real GDevelop: the

examples/3d-endless-terrain/3d-endless-terrain.json

@@ -8498,8 +8498,8 @@
       "iconUrl": "data:image/svg+xml;base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4NCjwhLS0gR2VuZXJhdG9yOiBBZG9iZSBJbGx1c3RyYXRvciAyMy4wLjMsIFNWRyBFeHBvcnQgUGx1Zy1JbiAuIFNWRyBWZXJzaW9uOiA2LjAwIEJ1aWxkIDApICAtLT4NCjxzdmcgdmVyc2lvbj0iMS4xIiBpZD0iSWNvbnMiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4Ig0KCSB2aWV3Qm94PSIwIDAgMzIgMzIiIHN0eWxlPSJlbmFibGUtYmFja2dyb3VuZDpuZXcgMCAwIDMyIDMyOyIgeG1sOnNwYWNlPSJwcmVzZXJ2ZSI+DQo8c3R5bGUgdHlwZT0idGV4dC9jc3MiPg0KCS5zdDB7ZmlsbDpub25lO3N0cm9rZTojMDAwMDAwO3N0cm9rZS13aWR0aDoyO3N0cm9rZS1saW5lY2FwOnJvdW5kO3N0cm9rZS1saW5lam9pbjpyb3VuZDtzdHJva2UtbWl0ZXJsaW1pdDoxMDt9DQo8L3N0eWxlPg0KPHBvbHlsaW5lIGNsYXNzPSJzdDAiIHBvaW50cz0iMTksMTkgMjQsMTkgMjQsMjQgIi8+DQo8cG9seWxpbmUgY2xhc3M9InN0MCIgcG9pbnRzPSI2LDIzIDExLDIzIDExLDE4ICIvPg0KPHBhdGggY2xhc3M9InN0MCIgZD0iTTI0LDE5LjRjLTAuNywwLjgtMS40LDEuNi0yLjIsMi40Yy03LDctMTUuMywxMC4yLTE4LjUsN3MtMC4xLTExLjUsNy0xOC41czE1LjMtMTAuMiwxOC41LTcNCgljMS40LDEuNCwxLjYsMy42LDAuOCw2LjMiLz4NCjxwYXRoIGNsYXNzPSJzdDAiIGQ9Ik0xMSwyMi41Yy0wLjMtMC4yLTAuNS0wLjUtMC44LTAuOGMtNy03LTEwLjItMTUuMy03LTE4LjVzMTEuNS0wLjEsMTguNSw3czEwLjIsMTUuMyw3LDE4LjUNCgljLTEuNywxLjctNC44LDEuNi04LjQsMC4xIi8+DQo8L3N2Zz4NCg==",
       "name": "MousePointerLock",
       "previewIconUrl": "https://resources.gdevelop-app.com/assets/Icons/Line Hero Pack/Master/SVG/Virtual Reality/Virtual Reality_360_rotate_vr_movement.svg",
-      "shortDescription": "This behavior removes the limit on the distance the mouse can move and hides the cursor.",
-      "version": "0.3.0",
+      "shortDescription": "Lock and hide mouse pointer for unlimited movement (e.g., FPS mouse look).",
+      "version": "0.3.2",
       "description": [
         "This behavior removes the limit on the distance the mouse can move and hides the cursor.",
         "",
@@ -8547,7 +8547,7 @@
         {
           "name": "TouchSpeedFactor",
           "type": "number",
-          "value": 4
+          "value": 2
         }
       ],
       "eventsFunctions": [
@@ -8795,16 +8795,46 @@
               ]
             },
             {
-              "type": "BuiltinCommonInstructions::JsCode",
-              "inlineCode": [
-                "const { handler } = gdjs._MousePointerLockExtension;\r",
-                "\r",
-                "handler.requestPointerLock();\r",
-                ""
+              "type": "BuiltinCommonInstructions::Comment",
+              "color": {
+                "b": 109,
+                "g": 230,
+                "r": 255,
+                "textB": 0,
+                "textG": 0,
+                "textR": 0
+              },
+              "comment": "Fingers are followed instead of locking the pointer, because mobile browsers give tiny pointer movements."
+            },
+            {
+              "type": "BuiltinCommonInstructions::Standard",
+              "conditions": [
+                {
+                  "type": {
+                    "value": "NumberVariable"
+                  },
+                  "parameters": [
+                    "TouchId",
+                    "<",
+                    "2"
+                  ]
+                }
               ],
-              "parameterObjects": "",
-              "useStrict": true,
-              "eventsSheetExpanded": false
+              "actions": [],
+              "events": [
+                {
+                  "type": "BuiltinCommonInstructions::JsCode",
+                  "inlineCode": [
+                    "const { handler } = gdjs._MousePointerLockExtension;\r",
+                    "\r",
+                    "handler.requestPointerLock();\r",
+                    ""
+                  ],
+                  "parameterObjects": "",
+                  "useStrict": true,
+                  "eventsSheetExpanded": false
+                }
+              ]
             }
           ],
           "parameters": [],
@@ -8951,7 +8981,7 @@
           "functionType": "Condition",
           "name": "isEmulatingPointerLock",
           "private": true,
-          "sentence": "The mouse pointer is lock is emulated",
+          "sentence": "The mouse pointer lock is emulated",
           "events": [
             {
               "type": "BuiltinCommonInstructions::Standard",
@@ -8987,6 +9017,31 @@
                   ]
                 }
               ]
+            },
+            {
+              "type": "BuiltinCommonInstructions::Standard",
+              "conditions": [
+                {
+                  "type": {
+                    "value": "NumberVariable"
+                  },
+                  "parameters": [
+                    "TouchId",
+                    ">=",
+                    "2"
+                  ]
+                }
+              ],
+              "actions": [
+                {
+                  "type": {
+                    "value": "SetReturnBoolean"
+                  },
+                  "parameters": [
+                    "True"
+                  ]
+                }
+              ]
             }
           ],
           "parameters": [],
@@ -9293,6 +9348,18 @@
                   },
                   "comment": "TODO It's probably a bad idea to rotate the object around Y."
                 },
+                {
+                  "type": "BuiltinCommonInstructions::Comment",
+                  "color": {
+                    "b": 109,
+                    "g": 230,
+                    "r": 255,
+                    "textB": 0,
+                    "textG": 0,
+                    "textR": 0
+                  },
+                  "comment": "Both rotations are relative to the screen height because the field of view is vertical. This way, the speed doesn't change with the screen ratio."
+                },
                 {
                   "type": "BuiltinCommonInstructions::Standard",
                   "conditions": [
@@ -9314,7 +9381,7 @@
                       "parameters": [
                         "Object",
                         "+",
-                        "MousePointerLock::MovementX() * HorizontalSpeed / SceneWindowWidth()"
+                        "MousePointerLock::MovementX() * HorizontalSpeed / (SceneWindowHeight() * 16 / 9)"
                       ]
                     },
                     {
@@ -9957,32 +10024,7 @@
               "name": "ShouldLock"
             }
           ],
-          "propertiesFolderStructure": {
-            "folderName": "__ROOT",
-            "children": [
-              {
-                "propertyName": "Object3D"
-              },
-              {
-                "propertyName": "HorizontalSpeed"
-              },
-              {
-                "propertyName": "VerticalSpeed"
-              },
-              {
-                "propertyName": "VerticalAngleMin"
-              },
-              {
-                "propertyName": "VerticalAngleMax"
-              },
-              {
-                "propertyName": "OffsetZ"
-              },
-              {
-                "propertyName": "ShouldLock"
-              }
-            ]
-          }
+          "sharedPropertyDescriptors": []
         }
       ],
       "eventsBasedObjects": []