adds template + CSS for contact compontent

[markconroy]

Closes #421

Note

This is not ready for testing just yet.

• Wait to get designs from Designer Dave and see about implementing them
• Update the CSS to use custom variables instead of generic ones.

What does this change?

Adds a template + CSS for contact component.

Images

Before

After

---

Thanks to Big Blue Door for sponsoring my time to work on this.

[markconroy]

This got the 👍 from Clear Spot Media, #421 (comment)

[millnut]

@ekes I've re-run the tests only failures are eslint and stylelint which I think have already been fixed on 1.x

[msayoung]

@markconroy - great!

A couple of things:

• the label for the Other team.localgov_contact.URL probably needs fixing
• Repetition of the Office Hours heading and label
• The Contact component title is showing above the component.
• There's a bit more padding above the Phone heading, so its misaligned with Email and Website

[finnlewis]

@markconroy looks like @msayoung has requested some changes, so we'll leave this with you for now.

[markconroy]

To do

• the label for the Other team.localgov_contact.URL probably needs fixing
• Repetition of the Office Hours heading and label
• The Contact component title is showing above the component.
• There's a bit more padding above the Phone heading, so its misaligned with Email and Website

[markconroy]

@msayoung I've completed checkbox 4 above, but the first three are config items not localgov_base items.

the label for the Other team.localgov_contact.URL probably needs fixing
Repetition of the Office Hours heading and label
The Contact component title is showing above the component.

So ... I think this is ready to be approved and councils can remove those field labels showing in their own config and/or we can remove them by default in our other corresponding repos.

[msayoung]

@markconroy - sorry I'm still seeing extra padding above the Phone item. We've also lost space between the titles and fields.

[markconroy]

@msayoung this is ready again.

[msayoung]

styles fine now, thanks.

[msayoung]

One thing I noticed is that we're getting a heading for Email whether or not there is anything in that field.

[msayoung]

I think this is supposed to wrap the block above.

[markconroy]

Fixed, can't believe I made such a rookie mistake!

[msayoung]

I assume you left it to check my QA skills ;)

[markconroy]

@msayoung ready again!

All changes requested have been implemented.