[6.0] Other declaration pills should be clickable when there are changes data

src/components/DocumentationTopic/PrimaryContent/Declaration.vue

@@ -20,7 +20,6 @@
     <template v-else>
       <DeclarationList
         v-for="(declaration, i) in declarations"
-        :class="changeClasses"
         :key="i"
         :declaration="declaration"
         :shouldCaption="hasPlatformVariants"

src/components/DocumentationTopic/PrimaryContent/DeclarationDiff.vue

@@ -17,7 +17,6 @@
         :key="i"
         :declaration="declaration"
         :should-caption="currentDeclarations.length > 1"
-        :changeType="changeType"
       />
     </div>
     <div class="declaration-diff-previous">
@@ -27,7 +26,6 @@
         :key="i"
         :declaration="declaration"
         :should-caption="previousDeclarations.length > 1"
-        :changeType="changeType"
       />
     </div>
   </div>
@@ -50,14 +48,6 @@ export default {
       type: Object,
       required: true,
     },
-    /**
-     * The applied change type to the diff.
-     * @type {"added"|"deprecated"|"modified"}
-     */
-    changeType: {
-      type: String,
-      required: true,
-    },
   },
   computed: {
     previousDeclarations: ({ changes }) => changes.declaration.previous || [],

src/components/DocumentationTopic/PrimaryContent/DeclarationGroup.vue

@@ -0,0 +1,164 @@
+<!--
+  This source file is part of the Swift.org open source project
+
+  Copyright (c) 2024 Apple Inc. and the Swift project authors
+  Licensed under Apache License v2.0 with Runtime Library Exception
+
+  See https://swift.org/LICENSE.txt for license information
+  See https://swift.org/CONTRIBUTORS.txt for Swift project authors
+-->
+
+<template>
+  <div
+    class="declaration-group"
+    :class="classes"
+    ref="apiChangesDiff"
+  >
+    <p v-if="shouldCaption" class="platforms">
+      <strong>{{ caption }}</strong>
+    </p>
+    <Source
+      :tokens="declaration.tokens"
+      :language="interfaceLanguage"
+    />
+  </div>
+</template>
+
+<script>
+import DeclarationSource from 'docc-render/components/DocumentationTopic/PrimaryContent/DeclarationSource.vue';
+import Language from 'docc-render/constants/Language';
+import { APIChangesMultipleLines } from 'docc-render/mixins/apiChangesHelpers';
+
+/**
+ * Renders a code source with an optional caption.
+ */
+export default {
+  name: 'DeclarationGroup',
+  components: {
+    Source: DeclarationSource,
+  },
+  mixins: [APIChangesMultipleLines],
+  inject: {
+    languages: {
+      default: () => new Set(),
+    },
+    interfaceLanguage: {
+      default: () => Language.swift.key.api,
+    },
+    symbolKind: {
+      default: () => undefined,
+    },
+  },
+  props: {
+    declaration: {
+      type: Object,
+      required: true,
+    },
+    /**
+     * Whether to show the caption or not.
+     * Usually if there is more than Declaration group.
+     */
+    shouldCaption: {
+      type: Boolean,
+      default: false,
+    },
+    /**
+     * The type of code change.
+     * @type {"added"|"deprecated"|"modified"}
+     */
+    changeType: {
+      type: String,
+      required: false,
+    },
+  },
+  computed: {
+    classes: ({
+      changeType,
+      multipleLinesClass,
+      displaysMultipleLinesAfterAPIChanges,
+    }) => ({
+      [`declaration-group--changed declaration-group--${changeType}`]: changeType,
+      [multipleLinesClass]: displaysMultipleLinesAfterAPIChanges,
+    }),
+    caption() {
+      return this.declaration.platforms.join(', ');
+    },
+  },
+};
+</script>
+
+<style scoped lang="scss">
+@import 'docc-render/styles/_core.scss';
+
+.platforms {
+  @include font-styles(body-reduced);
+
+  margin-bottom: 0.45rem;
+  margin-top: var(--spacing-stacked-margin-xlarge);
+
+  .changed & {
+    padding-left: $code-source-spacing;
+  }
+
+  &:first-of-type {
+    margin-top: 1rem;
+  }
+}
+
+.source {
+  transition: margin 0.3s linear;
+
+  .platforms + & {
+    margin: 0;
+  }
+}
+
+// don't highlight tokens in initial declaration until the user has explicitly
+// expanded a list of overloaded declarations — this rule could be simplified
+// in the future if the HTML is restructured to have an expanded state class for
+// the whole list instead of having it on each declaration
+.declaration-pill:not(.declaration-pill--expanded) {
+  .source:deep(.highlighted) {
+    background: unset;
+    font-weight: normal;
+  }
+}
+
+// only applicable for when other declaration list is expanded
+.declaration-pill--expanded {
+  $docs-declaration-source-border-width: 1px;
+
+  .source {
+    border-width: $docs-declaration-source-border-width;
+
+    // ensure links are not clickable, when expanded
+    :deep(a) {
+      pointer-events: none;
+    }
+  }
+
+  &.selected-declaration {
+    .source {
+      border-color: var(--color-focus-border-color, var(--color-focus-border-color));
+    }
+  }
+
+  &:not(.selected-declaration) {
+    .source {
+      background: none;
+    }
+  }
+}
+
+@include changedStyles {
+  .source {
+    // background should also be applied over changed icon over whole pill
+    background: none;
+    border: none;
+    margin-top: 0;
+    margin-bottom: 0;
+    margin-left: $change-icon-occupied-space;
+    padding-left: 0;
+  }
+}
+</style>