Replace h1 with div in consent overlay modal

[SteveDMurphy]

Closes FIDES-247

Description Of Changes

Requested by a user that noticed the h1 element used in the consent overlay modal could detract from SEO.

Found that there is an existing style variable for this (--fides-overlay-font-size-title) that was in some cases being overridden by the h1 element if styled.

Switching to a div allowed the expected style to be used

Code Changes

• Replaced the h1 element with a div element in ConsentContent.tsx

Steps to Confirm

• Tested and validated the change to the consent overlay using the expected variable (graphic below)

With styled h1:

Without styled h1:

After change to a div:

This could potentially result in some change in default appearance for the overlay, which should be reviewed (not a one person choice I think)

Pre-Merge Checklist

• All CI Pipelines Succeeded
• Documentation:
  • documentation complete, PR opened in fidesdocs
  • documentation issue created in fidesdocs
• Issue Requirements are Met
• Relevant Follow-Up Issues Created
• Update CHANGELOG.md
• For API changes, the Postman collection has been updated

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
fides-plus-nightly	⬜️ Ignored (Inspect)	Visit Preview		Nov 7, 2023 8:55pm