Skip to content

Conversation

geeknoid
Copy link
Contributor

@geeknoid geeknoid commented Feb 16, 2018

  • Add support for showing the archive date in the footer.

  • Fix invalid HTML in the header which improves the responsive layout. In particular, the down arrow
    for the dropdown menu tends to stay on the same line as the cog more than before.

  • Tone down the blue color of links so it's not so intense. There's still good contrast
    between links and normal text.

  • Remove broken og:image meta tag from each page. It used to point to a logo.png
    file, which we deleted months ago. Turns out it's not intended for web site logos
    anyway.

  • Add og:url and og:site_name per best practices.

  • Remove one more stray border on the community page. I thought I got rid of 'em last time,
    but I missed one.

Staging: https://geeknoid.github.io/istio.github.io/

@geeknoid geeknoid requested a review from ldemailly February 16, 2018 15:42
- Add support for showing the archive date in the footer.

- Fix invalid HTML in the header which improves the responsive layout. In particular, the down arrow
for the dropdown menu tends to stay on the same line as the cog more than before.

- Tone down the blue color of links so it's not so intense. There's still good contrast
between links and normal text.

- Remove broken og:image meta tag from each page. It used to point to a logo.png
file, which we deleted months ago. Turns out it's not intended for web site logos
anyway.

- Add og:url and og:site_name per best practices.

- Remove one more stray border on the community page. I thought I got rid of 'em last time,
but I missed one.
Copy link
Member

@ldemailly ldemailly left a comment

Choose a reason for hiding this comment

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

thanks for all the work on this

@@ -9,7 +9,7 @@ $backgroundColor: $white;
$textColor: #535f61;
$textCodeColor: #d14;

$linkColor: #469BFF;
$linkColor: #469Bdd;
Copy link
Member

Choose a reason for hiding this comment

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

I'd go a bit darker/further to what it used to be but thanks for going in that direction already !

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.

2 participants