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

Blog Styling needs to be cleaned up #21

Open
akozlik opened this issue Feb 6, 2022 · 3 comments
Open

Blog Styling needs to be cleaned up #21

akozlik opened this issue Feb 6, 2022 · 3 comments
Milestone

Comments

@akozlik
Copy link
Collaborator

akozlik commented Feb 6, 2022

  • Left align blog posts
  • "In the News" title is too close to the header. We need to determine our vertical spacing here.

Screen Shot 2022-02-06 at 12 45 46 PM

@akozlik akozlik added this to the v1.0 milestone Feb 6, 2022
@jonkurtis
Copy link
Contributor

I think the page layouts in general need to be cleaned up and de-duplicated. There seems to be a layout for every page that builds on the default layout but essentially just ads the title to an H1 and puts the content on the page. Instead we should have a general page, post, event layout which would include default spacing at the top of the page. I think it is better to add default spacing as margin top of the page layout instead of margin bottom to the header itself to give us the flexibility to add an element directly up against the header like a hero image.

@akozlik
Copy link
Collaborator Author

akozlik commented Feb 18, 2022

Yeah I definitely agree. We should pick a single layout per page type and stick wtih that. What do you need to move forward with that?

@jonkurtis
Copy link
Contributor

Nothing I need. I will work on this.

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

No branches or pull requests

2 participants