Remove support for care card optional visually hidden text

Author: colinrotherham

src/components/navigation/card/__tests__/Card.test.tsx

@@ -225,28 +225,6 @@ describe('Card', () => {
         );
       });
     });
-
-    describe('hidden text', () => {
-      it('renders without hidden text', () => {
-        const { container } = render(
-          <Card cardType="urgent">
-            <Card.Heading visuallyHiddenText={false}>Heading</Card.Heading>
-          </Card>,
-        );
-
-        expect(container.querySelector('.nhsuk-u-visually-hidden')).toBeFalsy();
-      });
-
-      it('renders with hidden text', () => {
-        const { container } = render(
-          <Card cardType="urgent">
-            <Card.Heading visuallyHiddenText="Custom">Heading</Card.Heading>
-          </Card>,
-        );
-
-        expect(container.querySelector('.nhsuk-u-visually-hidden')?.textContent).toEqual('Custom');
-      });
-    });
   });
 
   describe('Card.Group', () => {

src/components/navigation/card/components/CardHeading.tsx

@@ -8,26 +8,22 @@ import { cardTypeIsCareCard } from '@util/types/TypeGuards';
 
 export interface CardHeadingProps extends HTMLProps<HTMLHeadingElement> {
   headingLevel?: HeadingLevelType;
-  visuallyHiddenText?: false | string;
 }
 
 const genHiddenText = (cardType: CareCardType): string => {
   switch (cardType) {
-    case 'non-urgent':
-      return 'Non-urgent advice: ';
-    case 'urgent':
-      return 'Urgent advice: ';
     case 'emergency':
       return 'Immediate action required: ';
+    case 'urgent':
+      return 'Urgent advice: ';
     default:
-      return '';
+      return 'Non-urgent advice: ';
   }
 };
 
 const CareHeading: FC<CardHeadingProps & { careType: CareCardType }> = ({
   className,
   children,
-  visuallyHiddenText,
   careType,
   headingLevel = 'h2',
   role = 'text',
@@ -41,11 +37,7 @@ const CareHeading: FC<CardHeadingProps & { careType: CareCardType }> = ({
         {...rest}
       >
         <span role={role}>
-          {visuallyHiddenText !== false ? (
-            <span className="nhsuk-u-visually-hidden">
-              {visuallyHiddenText || genHiddenText(careType)}
-            </span>
-          ) : null}
+          <span className="nhsuk-u-visually-hidden">{genHiddenText(careType)}</span>
           {children}
         </span>
       </HeadingLevel>

stories/Navigation/Card.stories.tsx

@@ -236,39 +236,3 @@ export const EmergencyCareCard: Story = {
     </Card>
   ),
 };
-
-/**
- * By default, CareCard components prepend hidden text before the title. These are:
- *
- * - ("non-urgent") Non-urgent advice:
- * - ("urgent") Urgent advice:
- * - ("immediate") Immediate action required:
- *
- * If you wish to disable this behaviour, pass the prop `visuallyHiddenText={false}` to the `CareCard.Heading` component or specify your own visually hidden text by using `visuallyHiddenText="Custom"`.
- *
- * You can change the heading type (i.e. `h1`, `h2` and so on) of the title by passing the prop `headingLevel="<headingLevel>"` to the `CareCard.Heading`.
- */
-export const WithCustomVisuallyHiddenText: Story = {
-  args: { cardType: 'non-urgent' },
-  render: (args) => (
-    <Card cardType={args.cardType}>
-      <Card.Heading visuallyHiddenText="Custom visually hidden text">
-        Speak to a GP if:
-      </Card.Heading>
-      <Card.Content>
-        <ul>
-          <li>you&apos;re not sure it&apos;s chickenpox</li>
-          <li>the skin around the blisters is red, hot or painful (signs of infection)</li>
-          <li>
-            your child is <a href="">dehydrated</a>
-          </li>
-          <li>you&apos;re concerned about your child or they get worse</li>
-        </ul>
-        <p>
-          Tell the receptionist you think it&apos;s chickenpox before going in. They may recommend a
-          special appointment time if other patients are at risk.
-        </p>
-      </Card.Content>
-    </Card>
-  ),
-};