Merge branch 'bump/primer-upstream-ref' into bump/primer-upstream

opf · Nov 18, 2024 · a3ea4cd · a3ea4cd
2 parents ebdab21 + a2398ab
commit a3ea4cd
Show file tree

Hide file tree

Showing 422 changed files with 10,337 additions and 15,413 deletions.
diff --git a/.changeset/afraid-teachers-battle.md b/.changeset/afraid-teachers-battle.md
@@ -0,0 +1,5 @@
+---
+"@openproject/primer-view-components": minor
+---
+
+Delete Primer::Alpha::ImageCrop
diff --git a/.changeset/bright-eggs-return.md b/.changeset/bright-eggs-return.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': patch
+---
+
+Moves the loading and "no results" messages on the SelectPanel to the top of the container.
diff --git a/.changeset/chatty-apples-hope.md b/.changeset/chatty-apples-hope.md
@@ -0,0 +1,6 @@
+---
+'@openproject/primer-view-components': minor
+---
+
+[SelectPanel] replace loading label using parameter loading_label
+[SelectPanel] show additional loading content with loading_description
diff --git a/.changeset/cuddly-games-prove.md b/.changeset/cuddly-games-prove.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': patch
+---
+
+Allow setting custom values on submit buttons.
diff --git a/.changeset/curvy-apes-judge.md b/.changeset/curvy-apes-judge.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': minor
+---
+
+[Forms] Fix position of caption for checkbox and radio groups
diff --git a/.changeset/four-steaks-shake.md b/.changeset/four-steaks-shake.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': minor
+---
+
+Add option to Dialog to permit/disable page scrolling
diff --git a/.changeset/healthy-phones-love.md b/.changeset/healthy-phones-love.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': minor
+---
+
+Bump primer/css
diff --git a/.changeset/heavy-camels-hear.md b/.changeset/heavy-camels-hear.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': patch
+---
+
+Fix problem introduced by #3141 that causes template captions not to render for certain input types if the input is configured with a `value:`.
diff --git a/.changeset/hip-deers-rhyme.md b/.changeset/hip-deers-rhyme.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': minor
+---
+
+Allow form groups to accept system arguments
diff --git a/.changeset/lemon-kiwis-learn.md b/.changeset/lemon-kiwis-learn.md
@@ -0,0 +1,5 @@
+---
+"@openproject/primer-view-components": patch
+---
+
+Add "action" to the list of data attributes that accept multiple values in the `merge_data` helper
diff --git a/.changeset/many-readers-own.md b/.changeset/many-readers-own.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': minor
+---
+
+[SelectPanel] Use checkboxes for multi-select
diff --git a/.changeset/new-nails-walk.md b/.changeset/new-nails-walk.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': patch
+---
+
+Adjusts config for DetailsMenuMigration linter to take in an array of strings instead of array of regexp
diff --git a/.changeset/quiet-fans-repeat.md b/.changeset/quiet-fans-repeat.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': patch
+---
+
+[SelectPanel] Fix issue causing no error banner to appear when there are no matches
diff --git a/.changeset/shy-schools-grow.md b/.changeset/shy-schools-grow.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': minor
+---
+
+Introduce the Stack and StackItem components.
diff --git a/.changeset/sweet-donkeys-allow.md b/.changeset/sweet-donkeys-allow.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': patch
+---
+
+Clear input on SelectPanel when dialog closed
diff --git a/.changeset/three-readers-switch.md b/.changeset/three-readers-switch.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': minor
+---
+
+[SelectPanel] Adds banner scheme as param.
diff --git a/.changeset/tough-actors-wait.md b/.changeset/tough-actors-wait.md
@@ -0,0 +1,5 @@
+---
+"@openproject/primer-view-components": patch
+---
+
+Fix bug where checkboxes were broken with prefers-reduced-motion
diff --git a/.changeset/tricky-foxes-train.md b/.changeset/tricky-foxes-train.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': patch
+---
+
+Increase space between form elements
diff --git a/.changeset/wicked-bananas-tie.md b/.changeset/wicked-bananas-tie.md
@@ -0,0 +1,5 @@
+---
+'@openproject/primer-view-components': patch
+---
+
+Remove dialog-scrollgutter setting from hot-path, for improved performance
diff --git a/.devcontainer/devcontainer.json b/.devcontainer/devcontainer.json
@@ -13,25 +13,29 @@
   },
   "onCreateCommand": ".devcontainer/on-create-command.sh",
   "postCreateCommand": ".devcontainer/post-create-command.sh",
-  "settings": {
-    "files.watcherExclude": {
-      "**/vendor": true,
-      "**/.git": true,
-      "**/tmp": true,
-      "**/node_modules": true
-    }
+  "customizations": {
+    "settings": {
+      "files.watcherExclude": {
+        "**/vendor": true,
+        "**/.git": true,
+        "**/tmp": true,
+        "**/node_modules": true
+      }
+    },
+    "extensions": [
+      "castwide.solargraph",
+      "kaiwood.endwise",
+      "manuelpuyol.erb-linter",
+      "misogi.ruby-rubocop",
+      "rebornix.ruby",
+      "wingrunr21.vscode-ruby",
+      "mutantdino.resourcemonitor",
+      "vunguyentuan.vscode-postcss",
+      "DavidAnson.vscode-markdownlint",
+      "stylelint.vscode-stylelint",
+      "hex-ci.stylelint-plus"
+    ]
   },
-  "extensions": [
-    "castwide.solargraph",
-    "kaiwood.endwise",
-    "manuelpuyol.erb-linter",
-    "misogi.ruby-rubocop",
-    "rebornix.ruby",
-    "wingrunr21.vscode-ruby",
-    "mutantdino.resourcemonitor",
-    "vunguyentuan.vscode-postcss",
-    "DavidAnson.vscode-markdownlint"
-  ],
   "forwardPorts": [4000],
   "appPort": [4000],
   "portsAttributes": {

diff --git a/.erb-lint.yml b/.erb-lint.yml
@@ -27,7 +27,7 @@ linters:
         Enabled: false
         Exclude:
           - app/components/primer/layout_component.html.erb
-      Primer/Migrations/IconButtonComponent:
+      Migrations/IconButtonComponent:
         Enabled: true
-      Primer/Migrations/TruncateComponent:
+      Migrations/TruncateComponent:
         Enabled: true
diff --git a/.github/release_template.md b/.github/release_template.md
@@ -21,4 +21,8 @@ Please ensure these items are checked before merging.
 - [ ] No noticeable regressions have been introduced as a result of changes in this release
 - [ ] Release notes accurately describe the changes made
 
+### 🚢 Post ship reminders
+
+- [ ] Deploy lookbook via chat ops in the #primer-rails-ops channel. `.deploy primer-lookbook to production`
+
 Please also leave any testing notes as a comment on this pull request. In particular, describing any issues encountered during your testing. This is helpful in providing historical context to maintainers.
diff --git a/.github/workflows/demo-preview-cleanup.yml b/.github/workflows/demo-preview-cleanup.yml
@@ -27,9 +27,9 @@ jobs:
         with:
           # excluding a client secret here will cause a login via OpenID Connect (OIDC),
           # which prevents us from having to rotate client credentials, etc
-          client-id: "5ad1a188-b944-40eb-a2f8-cc683a6a65a0"
-          tenant-id: "398a6654-997b-47e9-b12b-9515b896b4de"
-          subscription-id: "550eb99d-d0c7-4651-a337-f53fa6520c4f"
+          client-id: ${{ secrets.AZURE_CLIENT_ID }}
+          tenant-id: ${{ secrets.AZURE_TENANT_ID }}
+          subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
 
       # Do this before repo checkout to prevent running bundle install
       - uses: ruby/setup-ruby@v1

diff --git a/.github/workflows/demo-preview-destroy.yml b/.github/workflows/demo-preview-destroy.yml
@@ -30,9 +30,9 @@ jobs:
         with:
           # excluding a client secret here will cause a login via OpenID Connect (OIDC),
           # which prevents us from having to rotate client credentials, etc
-          client-id: "5ad1a188-b944-40eb-a2f8-cc683a6a65a0"
-          tenant-id: "398a6654-997b-47e9-b12b-9515b896b4de"
-          subscription-id: "550eb99d-d0c7-4651-a337-f53fa6520c4f"
+          client-id: ${{ secrets.AZURE_CLIENT_ID }}
+          tenant-id: ${{ secrets.AZURE_TENANT_ID }}
+          subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
 
       - name: Check out repo
         uses: actions/checkout@v4

diff --git a/.github/workflows/demo-production-deploy.yml b/.github/workflows/demo-production-deploy.yml
@@ -32,9 +32,9 @@ jobs:
       with:
         # excluding a client secret here will cause a login via OpenID Connect (OIDC),
         # which prevents us from having to rotate client credentials, etc
-        client-id: "5ad1a188-b944-40eb-a2f8-cc683a6a65a0"
-        tenant-id: "398a6654-997b-47e9-b12b-9515b896b4de"
-        subscription-id: "550eb99d-d0c7-4651-a337-f53fa6520c4f"
+        client-id: ${{ secrets.AZURE_CLIENT_ID }}
+        tenant-id: ${{ secrets.AZURE_TENANT_ID }}
+        subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
     - name: Purge tags
       run: |
         # only delete tags that aren't "latest" or "latest-assets"
@@ -84,13 +84,16 @@ jobs:
       with:
         # excluding a client secret here will cause a login via OpenID Connect (OIDC),
         # which prevents us from having to rotate client credentials, etc
-        client-id: "5ad1a188-b944-40eb-a2f8-cc683a6a65a0"
-        tenant-id: "398a6654-997b-47e9-b12b-9515b896b4de"
-        subscription-id: "550eb99d-d0c7-4651-a337-f53fa6520c4f"
+        client-id: ${{ secrets.AZURE_CLIENT_ID }}
+        tenant-id: ${{ secrets.AZURE_TENANT_ID }}
+        subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
     - name: Deploy
       env:
         RAILS_MASTER_KEY: ${{ secrets.RAILS_MASTER_KEY }}
         AZURE_ACR_PASSWORD: ${{ secrets.AZURE_ACR_PASSWORD }}
+        AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
+        AZURE_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }}
+        AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }}
       run: |
-        export AZURE_ACCESS_TOKEN=$(az account get-access-token --subscription 550eb99d-d0c7-4651-a337-f53fa6520c4f | jq -r .accessToken)
+        export AZURE_ACCESS_TOKEN=$(az account get-access-token --subscription ${{ secrets.AZURE_SUBSCRIPTION_ID }} | jq -r .accessToken)
         bin/kuby -e production deploy
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -12,7 +12,7 @@ jobs:
     - uses: actions/checkout@v4
     - name: Get specific changed files
       id: changed-files
-      uses: tj-actions/changed-files@v45.0.1
+      uses: tj-actions/changed-files@v45.0.2
       with:
         files: |
           docs/**/*.md
@@ -25,7 +25,6 @@ jobs:
     - uses: actions/setup-node@v4
       if: steps.changed-files.outputs.any_changed == 'true'
       with:
-        node-version: 16
         cache: 'npm'
     - name: Lint with Markdownlint
       if: steps.changed-files.outputs.any_changed == 'true'
@@ -39,57 +38,50 @@ jobs:
     - uses: actions/checkout@v4
     - name: Get specific changed files
       id: changed-files
-      uses: tj-actions/changed-files@v45.0.1
+      uses: tj-actions/changed-files@v45.0.2
       with:
         files: |
           app/**/*.rb
           lib/**/*.rb
           demo/**/*.rb
           test/**/*.rb
     - uses: ruby/setup-ruby@v1
-      if: steps.changed-files.outputs.any_changed == 'true'
       with:
         ruby-version: '3.3'
         bundler-cache: true
     - name: Lint with Rubocop
-      if: steps.changed-files.outputs.any_changed == 'true'
-      run: |
-        bundle exec rubocop ${{ steps.changed-files.outputs.all_changed_files }} --format github
+      run: script/rubocop --format github
 
   erblint:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v4
     - name: Get changed files
       id: changed-files
-      uses: tj-actions/changed-files@v45.0.1
+      uses: tj-actions/changed-files@v45.0.2
       with:
         files: |
           app/components/**/*.erb
     - uses: ruby/setup-ruby@v1
-      if: steps.changed-files.outputs.any_changed == 'true'
       with:
         ruby-version: '3.3'
         bundler-cache: true
     - name: Lint with ERB Lint
-      if: steps.changed-files.outputs.any_changed == 'true'
-      run: |
-        bundle exec erblint ${{ steps.changed-files.outputs.all_changed_files }} --format github
+      run: script/erblint --format github
 
   eslint:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v4
     - name: Get changed files
       id: changed-files
-      uses: tj-actions/changed-files@v45.0.1
+      uses: tj-actions/changed-files@v45.0.2
       with:
         files: |
           app/components/**/*.ts
     - uses: actions/setup-node@v4
       if: steps.changed-files.outputs.any_changed == 'true'
       with:
-        node-version: 16
         cache: 'npm'
     - name: Lint with ESLint
       if: steps.changed-files.outputs.any_changed == 'true'
@@ -103,18 +95,17 @@ jobs:
     - uses: actions/checkout@v4
     - name: Get changed files
       id: changed-files
-      uses: tj-actions/changed-files@v45.0.1
+      uses: tj-actions/changed-files@v45.0.2
       with:
         files: |
           app/components/**/*.pcss
           lib/postcss_mixins/*.pcss
     - uses: actions/setup-node@v4
       if: steps.changed-files.outputs.any_changed == 'true'
       with:
-        node-version: 16
         cache: 'npm'
     - name: Lint with Stylelint
       if: steps.changed-files.outputs.any_changed == 'true'
       run: |
         npm ci
-        npx stylelint ${{ steps.changed-files.outputs.all_changed_files }} -f "github"
+        npx stylelint --rd -q ${{ steps.changed-files.outputs.all_changed_files }} --custom-formatter=stylelint-actions-formatters
diff --git a/.github/workflows/preview-deploy.yml b/.github/workflows/preview-deploy.yml
@@ -64,9 +64,9 @@ jobs:
       with:
         # excluding a client secret here will cause a login via OpenID Connect (OIDC),
         # which prevents us from having to rotate client credentials, etc
-        client-id: "5ad1a188-b944-40eb-a2f8-cc683a6a65a0"
-        tenant-id: "398a6654-997b-47e9-b12b-9515b896b4de"
-        subscription-id: "550eb99d-d0c7-4651-a337-f53fa6520c4f"
+        client-id: ${{ secrets.AZURE_CLIENT_ID }}
+        tenant-id: ${{ secrets.AZURE_TENANT_ID }}
+        subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
 
     - name: Get preview app info
       run: ./.github/workflows/demo-preview-app-info.sh
@@ -98,7 +98,7 @@ jobs:
       uses: azure/arm-deploy@a1361c2c2cd398621955b16ca32e01c65ea340f5
       with:
         resourceGroupName: primer
-        subscriptionId: 550eb99d-d0c7-4651-a337-f53fa6520c4f
+        subscriptionId: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
         template: ./.github/workflows/demo-preview-template.json
         deploymentName: ${{env.DEPLOYMENT_NAME}}
         parameters: appName="${{env.APP_NAME}}"