fix: always show all CRE links on CRE pages without collapsing (#901) (#909)

The issue was in CommonRequirementEnumeration.tsx — the collapse logic (slicing the list after 5 items and showing a "Show more" button) was being applied to all link types including CREs, with no distinction between CRE links and links to external standards.

The fix adds an allLinksAreCres check that mirrors the same pattern already correctly implemented in DocumentNode.tsx. When all links in a group are of type CRE, the full list is shown without slicing and the "Show more" button is hidden. Links to external standards continue to collapse as before.

Changes are limited to a single file: CommonRequirementEnumeration.tsx

Added DOCUMENT_TYPES import from ../../const
Compute allLinksAreCres before rendering each link group
Use visibleResults (full list for CREs, sliced for others) instead of inline slice
Guard the "Show more" button with !allLinksAreCres

Author: shreeshtripurwarcomp23-coder

application/frontend/src/pages/CommonRequirementEnumeration/CommonRequirementEnumeration.tsx

@@ -8,6 +8,7 @@ import { Icon } from 'semantic-ui-react';
 import { DocumentNode } from '../../components/DocumentNode';
 import { ClearFilterButton, FilterButton } from '../../components/FilterButton/FilterButton';
 import { LoadingAndErrorIndicator } from '../../components/LoadingAndErrorIndicator';
+import { DOCUMENT_TYPES } from '../../const';
 import { useEnvironment } from '../../hooks';
 import { applyFilters, filterContext } from '../../hooks/applyFilters';
 import { Document } from '../../types';
@@ -106,25 +107,30 @@ export const CommonRequirementEnumeration = () => {
                 const sortedResults = links.sort((a, b) =>
                   getDocumentDisplayName(a.document).localeCompare(getDocumentDisplayName(b.document))
                 );
+                const allLinksAreCres =
+                  sortedResults.length > 0 &&
+                  sortedResults.every((link) => link.document.doctype === DOCUMENT_TYPES.TYPE_CRE);
+                const visibleResults =
+                  allLinksAreCres || showAll[type]
+                    ? sortedResults
+                    : sortedResults.slice(0, MAX_LENGTH_FOR_AUTO_EXPAND);
                 return (
                   <div className="cre-page__links" key={type}>
                     <div className="cre-page__links-eader">
                       <b>Which {getDocumentTypeText(type, links[0].document.doctype)}</b>:
                       {/* Risk of mixed doctype in here causing odd output */}
                     </div>
-                    {sortedResults
-                      .slice(0, showAll[type] ? sortedResults.length : MAX_LENGTH_FOR_AUTO_EXPAND)
-                      .map((link, i) => (
-                        <div
-                          key={i}
-                          className="accordion ui fluid styled cre-page__links-container"
-                          style={{ marginBottom: '4px' }}
-                        >
-                          <DocumentNode node={link.document} linkType={type} />
-                          <FilterButton document={link.document} />
-                        </div>
-                      ))}
-                    {sortedResults.length > MAX_LENGTH_FOR_AUTO_EXPAND && (
+                    {visibleResults.map((link, i) => (
+                      <div
+                        key={i}
+                        className="accordion ui fluid styled cre-page__links-container"
+                        style={{ marginBottom: '4px' }}
+                      >
+                        <DocumentNode node={link.document} linkType={type} />
+                        <FilterButton document={link.document} />
+                      </div>
+                    ))}
+                    {!allLinksAreCres && sortedResults.length > MAX_LENGTH_FOR_AUTO_EXPAND && (
                       <button
                         onClick={() => setShowAll((prev) => ({ ...prev, [type]: !prev[type] }))}
                         style={{ marginTop: '8px', cursor: 'pointer' }}