update facet drawer height
closes #6229
I think @dbranchini may have intentionally added the close button in the larger design?
@taylor-steve Thanks I missed that. I am assuming this should look the same on medium screens as well?
Is medium the full size side drawer? That seems fine to me but Darcy would the one to say for sure.
done and screenshots added
Should the headings be h1? We've got a visually-hidden h1 right above it. Site improve thinks it's fine, WAVE doesn't like it. I don't know if drawers have special considerations?
Per SODA, WAVE complains with more false positives than the other scanning tools because it scans hidden elements. We should use the same pattern here as we do for modals, and we use H2s for modals.
As far as medium sized, this is what I designed and it seems that was implemented, so we shouldn't have drawers on that size.