Merge remote-tracking branch 'origin/main' into fetch-readme-parallel

Author: ghostdevv

.github/workflows/remove-needs-review-on-review.yml

@@ -0,0 +1,69 @@
+name: remove-needs-review-on-review
+
+on:
+  pull_request_review:
+    types:
+      - submitted
+
+jobs:
+  remove-needs-review:
+    name: 🏷️ Remove needs review label
+    if: github.repository == 'npmx-dev/npmx.dev'
+    runs-on: ubuntu-slim
+    permissions:
+      contents: read
+      issues: write
+      pull-requests: write
+    steps:
+      - name: 🏷️ Remove needs review label
+        uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
+        with:
+          script: |
+            const pr = context.payload.pull_request;
+            const review = context.payload.review;
+            const LABEL = 'needs review';
+            const reviewer = review?.user?.login;
+            const author = pr.user?.login;
+            const reviewerType = review?.user?.type;
+
+            if (!reviewer) {
+              console.log('No reviewer login found in payload, skipping.');
+              return;
+            }
+
+            if (reviewerType === 'Bot') {
+              console.log(`Skipping bot review from @${reviewer} on PR #${pr.number}.`);
+              return;
+            }
+
+            if (reviewer === author) {
+              console.log(`Skipping self-review from @${reviewer} on PR #${pr.number}.`);
+              return;
+            }
+
+            if (!pr.labels.some(({ name }) => name === LABEL)) {
+              console.log(`PR #${pr.number} does not have the "${LABEL}" label.`);
+              return;
+            }
+
+            const { data: permission } = await github.rest.repos.getCollaboratorPermissionLevel({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              username: reviewer,
+            });
+
+            console.log(`Reviewer @${reviewer} permission is "${permission.permission}".`);
+
+            if (!['admin', 'maintain', 'write'].includes(permission.permission)) {
+              console.log(`Reviewer @${reviewer} is not a maintainer, skipping.`);
+              return;
+            }
+
+            await github.rest.issues.removeLabel({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              issue_number: pr.number,
+              name: LABEL,
+            });
+
+            console.log(`Removed "${LABEL}" from PR #${pr.number}.`);

README.md

@@ -77,7 +77,7 @@ What npmx offers:
 | Organization pages             |    ✅     |    ✅    |
 | Provenance indicators          |    ✅     |    ✅    |
 | Code browser                   |    ✅     |    ✅    |
-| Dark mode                      |    ❌     |    ✅    |
+| Dark mode                      |    ✅     |    ✅    |
 | Outdated dependency warnings   |    ❌     |    ✅    |
 | Module format badges (ESM/CJS) |    ❌     |    ✅    |
 | TypeScript types indicator     |    ✅     |    ✅    |

app/components/CollapsibleSection.vue

@@ -62,8 +62,13 @@ function toggle() {
 }
 
 const ariaLabel = computed(() => {
-  const action = isOpen.value ? 'Collapse' : 'Expand'
-  return props.title ? `${action} ${props.title}` : action
+  if (!props.title) {
+    return isOpen.value ? $t('common.collapse') : $t('common.expand')
+  }
+
+  return isOpen.value
+    ? $t('common.collapse_with_name', { name: props.title })
+    : $t('common.expand_with_name', { name: props.title })
 })
 useHead({
   style: [

app/components/ColorScheme/Img.vue

@@ -0,0 +1,20 @@
+<script setup lang="ts">
+const props = defineProps<{
+  lightSrc: string
+  darkSrc: string
+}>()
+</script>
+
+<template>
+  <img
+    :src="props.darkSrc"
+    class="color-mode-img"
+    :style="`--light-src: url('${props.lightSrc}')`"
+  />
+</template>
+
+<style>
+.light .color-mode-img {
+  content: var(--light-src);
+}
+</style>

app/components/Compare/FacetBarChart.vue

@@ -196,7 +196,7 @@ const config = computed<VueUiHorizontalBarConfig>(() => {
             },
             nameLabels: {
               fontSize: isMobile.value ? 12 : 18,
-              color: colors.value.fgSubtle,
+              color: colors.value.fg,
             },
             underlayerColor: colors.value.bg,
           },