Merge branch 'develop' into clairep/fix_deploy_to_production_trigger

Author: clairep94

.github/workflows/deploy-staging.yml

@@ -14,6 +14,7 @@ env:
   IMAGE: ${{ secrets.DOCKER_USERNAME }}/p5.js-web-editor-staging
 jobs:
   push_to_registry:
+    if: ${{ github.event.workflow_run.conclusion == 'success' }}
     environment: staging
     name: Push Docker image to Docker Hub
     runs-on: ubuntu-latest

.github/workflows/release-reminder.yml

@@ -0,0 +1,32 @@
+name: Weekly release reminder
+
+on:
+  # Tuesdays 15:30 UTC
+  schedule:
+    - cron: "30 15 * * 2"
+  workflow_dispatch: {}
+
+permissions:
+  issues: write
+
+jobs:
+  remind:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Open release reminder issue
+        uses: actions/github-script@v7
+        with:
+          script: |
+            await github.rest.issues.create({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              title: `Weekly release - ${new Date().toISOString().slice(0, 10)}`,
+              body: [
+                "1. Decide on any remaining PRs to merge into `develop`. Check the [Approved Candidates for Release](https://github.com/processing/p5.js-web-editor/pulls?q=is%3Apr+is%3Aopen+review%3Aapproved)",
+                "2. Review what's merged into `develop` since the last release.",
+                "3. Decide the version bump (patch/minor/major).",
+                "4. Run the [Create a release](../actions/workflows/release.yml) workflow with that version.",
+                "",
+                "No release needed this week? Just close this issue."
+              ].join("\n")
+            });

.github/workflows/test.yml

@@ -1,6 +1,12 @@
 name: Test
 
-on: [push, pull_request, workflow_call]
+on:
+  pull_request:
+  push:
+    branches:
+      - develop
+      - release
+  workflow_call:
 
 jobs:
   test:

client/modules/IDE/components/FileNode.jsx

@@ -231,6 +231,9 @@ const FileNode = ({
     if (!canEdit) {
       return;
     }
+    if (!isOptionsOpen) {
+      fileOptionsRef.current?.focus();
+    }
     setIsOptionsOpen(!isOptionsOpen);
   };
 

client/styles/components/_toast.scss

@@ -5,24 +5,32 @@
     color: getThemifyVariable('toast-text-color');
     background-color: getThemifyVariable('toast-background-color');
   }
-  padding: #{math.div(10, $base-font-size)}rem;
+  padding: #{math.div(10, $base-font-size)}rem #{math.div(20, $base-font-size)}rem;
   position: fixed;
-  top: 0;
+  top: #{math.div(20, $base-font-size)}rem; 
   left: 50%;
-  transform: translate(-50%, 0);
-  min-width: #{math.div(375, $base-font-size)}rem; //500px
-  font-size: #{math.div(20, $base-font-size)}rem;
+  transform: translateX(-50%);
+  min-width: #{math.div(360, $base-font-size)}rem; 
+  font-size: #{math.div(16, $base-font-size)}rem;
+  border-radius: #{math.div(8, $base-font-size)}rem;
   display: flex;
-  z-index: 9999;
   align-items: center;
   justify-content: space-between;
+  z-index: 9999;
+  transition: all 0.3s ease-in-out;
   box-shadow: 0 6px 6px 0 rgba(0,0,0,0.10);
 }
 
 .toast__close {
   @include themify() {
     @extend %icon-toast;
   }
-  padding-top: #{math.div(10, $base-font-size)}rem;
-  margin-left: #{math.div(40, $base-font-size)}rem;
-}
+  padding-top: #{math.div(6, $base-font-size)}rem;
+  margin-left: #{math.div(24, $base-font-size)}rem;
+  cursor: pointer;
+  opacity: 0.7;
+
+  &:hover {
+    opacity: 1;
+  }
+}

common/p5Versions.js

@@ -1,11 +1,13 @@
-export const currentP5Version = '2.3.2';
+export const currentP5Version = '2.3.4';
 
 // Generated from https://www.npmjs.com/package/p5?activeTab=versions
 // Run this in the console:
 // JSON.stringify([...document.querySelectorAll('._132722c7')].map(n => n.innerText), null, 2)
 // TODO: use their API for this to grab these at build time?
 export const p5Versions = [
-  { version: '2.3.2', label: '(Latest)' },
+  { version: '2.3.4', label: '(Latest)' },
+  '2.3.2',
+  '2.3.2',
   '2.3.1',
   '2.3.0',
   '2.2.3',