GSA / cto-website

Tech at GSA website
https://tech.gsa.gov/
Other
45 stars 66 forks source link

Removes `<br>` from hero #859

Closed brentryanjohnson closed 3 months ago

brentryanjohnson commented 4 months ago

I meant to include this in #857, but this removes the <br> and the accompanying (excessive) space in between the h1 and description.

brentryanjohnson commented 3 months ago

@jfredrickson This change appears to be throwing two pa11y errors, but neither of them seem related to this change. Do you know what's going on with the check?

jfredrickson commented 3 months ago

@brentryanjohnson I think it's an issue with pa11y-ci itself. It's complaining about some sort of iframe, but we don't use those anywhere on the site. I'm puzzled why it's coming up now all of a sudden. I'm going to temporarily disable the required pa11y-ci test to merge this, because that iframe error is the only issue.