freeipa-webui icon indicating copy to clipboard operation
freeipa-webui copied to clipboard

Adapt the `ariaLabel` and `ariaLabelledBy` properties in Certificate headers

Open carma12 opened this issue 11 months ago • 0 comments

The headers shown in the Certificates IPA object (IpaCertificates component) are handled by the getHeaderToggleButtonProps. It has the following structure:

// Get header toggle button props
const getHeaderToggleButtonProps = (cert: CertificateData, idx: number) => {
  return {
    id: "toggle-button-" + idx,
    "aria-label": "Details",
    "aria-labelledby":
      "toggle-button card-" + parseDn(cert.certInfo.issuer).cn,
  };
};

The aria-label and the aria-labelledby properties need to be adapted to have a more descriptive name (TBD by the UX team), not making reference to the issuer (as it should be hidden until expanding the card). E.g., Details for certificate <[cert name | serial number]>.

The same should be applied to the id property:

image

carma12 avatar Mar 06 '24 09:03 carma12