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

Frontpage Alterations #49

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

RichGriff
Copy link
Collaborator

Hey @chrisdburr, I have made the changes you required.

  • Scroll down arrow added
  • No animations when scrolling back through (note: animation only happens on page load and when items come into view)
  • Removed the main.html override as this causes issues for other pages. I managed to remove the class only for the frontpage, so you can still have the full width sections.

Any issues, please let me know.

@RichGriff RichGriff requested a review from chrisdburr March 23, 2023 16:17
@chrisdburr
Copy link
Collaborator

@RichGriff would you mind redoing this PR into the homepage branch please? We have GitHub Actions set up on the main branch to automatically compile and host the site from the main branch so it is only used for production-ready code. I would need to make some changes to this PR before it was ready to go live (e.g. adding proper images and copy).

https://github.com/alan-turing-institute/turing-commons/tree/homepage

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