<a href="<?php echo esc_url( home_url( '/' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a>

And please do not use target=“blank" on links as it leaves some users stranded on the new page without any effective way to navigate back to the original page.

</div><!-- #content -->

	<?php get_sidebar( 'footer' ); ?>

	<footer id="colophon" class="site-footer">
		<?php if ( has_nav_menu ( 'social' ) ) : ?>
			<?php wp_nav_menu( array( 'theme_location' => 'social', 'depth' => 1, 'link_before' => '<span class="screen-reader-text">', 'link_after' => '</span>', 'container_class' => 'social-links', ) ); ?>
		<?php endif; ?>

		<div class="site-info"  role="contentinfo">
			<?php
$year = date('Y');
if ($year != 2017){
echo '? 2017 – '.$year.' Musely-WD.com';
} else {
echo '? 2017 Musely-WD.com';
}
?>
		</div><!-- .site-info -->
	</footer><!-- #colophon -->
</div><!-- #page -->

<?php wp_footer(); ?>

</body>
</html>

.social-links ul li {
  margin: 0 5px;
}

I don’t think esmi realized the link is external to the site. While it’s possible (and kinda cool) to add a custom setting field in which to specify the copyright holder’s site, hardcoding external links is a pretty common practice. I’d go ahead and put the hardcoded links back in unless you want to add a settings field. She is right about the new window bit though, don’t do that!

You could have the if()else logic only manage the copyright and year part of the output, then output your site and link outside of the if()else structure so it only needs to occur once. Always strive to never replicate data! In fact, the “? 2017” only needs to occur once, the decision is only whether to output “– $year” or not.