Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consistent header and footer for blog #53

Closed
wants to merge 4 commits into from
Closed

Consistent header and footer for blog #53

wants to merge 4 commits into from

Conversation

mosabua
Copy link
Member

@mosabua mosabua commented Nov 4, 2019

  • externalized header and footer from default
  • added usage for blog related layouts
  • added youtube channel link to footer (and image)
  • added highlighting for blog link in nav

Signed-off-by: Manfred Moser mmoser@apache.org
Signed-off-by: Manfred Moser manfred@simpligility.com

- externalized header and footer from default
- added usage for blog related layouts
- added youtube channel link to footer (and image)
- added highlighting for blog link in nav

Signed-off-by: Manfred Moser <mmoser@apache.org>
Signed-off-by: Manfred Moser <manfred@simpligility.com>
@cla-bot cla-bot bot added the cla-signed label Nov 4, 2019
@mosabua
Copy link
Member Author

mosabua commented Nov 4, 2019

Screen Shot 2019-11-04 at 14 04 08

preview image from local run of site

@mosabua
Copy link
Member Author

mosabua commented Nov 4, 2019

@electrum @martint @dain @findepi .. feel free to have a look and merge away.


<header class="hero clearfix">
<div class="width">
<h1>Blog</h1>
Copy link
Member

Choose a reason for hiding this comment

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

The font for this doesn't match the font for the header in other pages.

Copy link
Member Author

@mosabua mosabua Nov 5, 2019

Choose a reason for hiding this comment

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

hm ... probably a side effect of the separate css for blog .. let me debug this some more. Might be a bigger cleanup needed. Fingers crossed I can just remove the usage of the blog css or override locally

Copy link
Member Author

Choose a reason for hiding this comment

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

Ouch ... side effects galore.

Copy link
Member Author

Choose a reason for hiding this comment

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

Found and pushed a fix. Reordering css imports ftw!

@martint
Copy link
Member

martint commented Nov 5, 2019

There's something about the website css that's causing the layout in at least one blog post to break. Take a look at:

https://deploy-preview-53--prestosql.netlify.com/blog/2019/05/21/optimizing-the-casts-away.html#putting-it-all-together
https://prestosql.io/blog/2019/05/21/optimizing-the-casts-away.html#putting-it-all-together

See how the formula numbers on the right no longer line up.

- in https://prestosql.io/blog/2019/05/21/optimizing-the-casts-away.html
- needed to remove extra spacing for li on left to allow formula to fit
- in the long run we will make the blog same width as other pages

Signed-off-by: Manfred Moser <mmoser@apache.org>
Signed-off-by: Manfred Moser <manfred@simpligility.com>
@mosabua
Copy link
Member Author

mosabua commented Nov 5, 2019

@martint Pushed a fix for the alignment. In the long run I think we want to make the blog use the same style as the rest of the site .. maybe even the same for the docs. Atm I am just trying to fix the big issues= while I work towards understanding all the setup and such.

@mosabua
Copy link
Member Author

mosabua commented Nov 7, 2019

Also made it so that the width of the blog matches the width of the other pages on the site now. Can we get this merged?

@martint
Copy link
Member

martint commented Nov 7, 2019

Merged, thanks!

@martint martint closed this Nov 7, 2019
@mosabua mosabua deleted the shared-footer branch November 8, 2019 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants