Skip to content

Social icon updates#51

Merged
budparr merged 3 commits intotheNewDynamic:masterfrom
bdashrad:social-icon-updates
Sep 26, 2017
Merged

Social icon updates#51
budparr merged 3 commits intotheNewDynamic:masterfrom
bdashrad:social-icon-updates

Conversation

@bdashrad
Copy link
Contributor

  • Add LinkedIn
  • Fix github svg spacing
  • Update colors to ones from brandcolors.net

</a>
{{ end }}
{{ with .Site.Params.linkedin }}
<a href="{{ . }}" class="link-transition linkedin link dib z-999 pt3 pt0-l mr3" title="LinkedIn link">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've personally found the mr3 spacing is a bit wide.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch. i was just about to push a fix but it was just merged.

@budparr
Copy link
Member

budparr commented Sep 26, 2017

Hi guys. Looks like two competing PRs? I'm going to try to pull these in, but FYI I'm working Ananke 2.0, which will break these changes slightly.

@adambkaplan
Copy link

Yep - competing. @bdashrad and I seem to have wanted to add GH and LinkedIn at the same time 😺

@budparr
Copy link
Member

budparr commented Sep 26, 2017

Is the substantive difference the mr3 mr2thing?

@budparr budparr merged commit eb765fd into theNewDynamic:master Sep 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants