Skip to content

New landing page design #119

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

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

New landing page design #119

wants to merge 80 commits into from

Conversation

shravanngoswamii
Copy link
Member

@shravanngoswamii shravanngoswamii commented Apr 14, 2025

Continuing #105
Preview the changes: https://turinglang.org/pr-previews/119

Copy link
Contributor

Preview the changes: https://turinglang.org/pr-previews/119
Please avoid using the search feature and navigation bar in PR previews!

@shravanngoswamii
Copy link
Member Author

If anyone is interested, then please feel free to take on this PR!

@shravanngoswamii shravanngoswamii linked an issue Apr 19, 2025 that may be closed by this pull request
simonsteiger and others added 22 commits April 21, 2025 22:54
* very basic redesign

* make buttons interactive
* add dangling code example

* add code example section
* quickfix alignment

* add news and clean up scss
* improve colors

* better link hover color
* paste in svg

* responsive svg header
using the same background colour for the team "cards" as for the non-hoverable cards on the landing page
- added links to discourse and slack to the buttons at the top of the page
- moved link to libraries page to "Learn more" section
@simonsteiger
Copy link
Collaborator

I will definitely have time to finish this in July. Which points would you like to see addressed before this PR can be merged?

@penelopeysm
Copy link
Member

Revisiting this. Is there a reason why the old PR was closed?

Also, I'd personally be happy to merge if some of the placeholder content is removed. Honestly, it's not like the existing front page has any resources, so even a fairly clean slate would be a huge improvement over the current one.

@shravanngoswamii
Copy link
Member Author

Were you thinking about something similar to a logo or a different animation?

No animation, just simple branding header, I used new logo now and its looking good!

The new max width layout looks awesome by the way!

Thank You!

But again, take this with a grain of salt – I absolutely get the visual appeal of hover effects and I'm fine keeping it as is, too. :)

I added some consistent styles for border and hover effect, lets keep it for now!

@yebai yebai requested a review from mhauru July 11, 2025 10:40
Copy link
Member

@mhauru mhauru left a comment

Choose a reason for hiding this comment

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

Looks really neat, good work @shravanngoswamii!

Only thing I would change is that in the News section, I'm not sure how the items that are displayed are picked, but currently it shows e.g. a post from 2019 announcing our blog. Maybe it could instead just show the three newest ones?

@shravanngoswamii
Copy link
Member Author

shravanngoswamii commented Jul 11, 2025

Only thing I would change is that in the News section, I'm not sure how the items that are displayed are picked, but currently it shows e.g. a post from 2019 announcing our blog. Maybe it could instead just show the three newest ones?

We just have to put on_landing: true in metadata of post to make it appear on landing page! I chose 2 latest newletters, 1 GSoC post, 1 research blog, and the blog announcement! Latest 6 post which have on_landing: true appears!

@simonsteiger
Copy link
Collaborator

Great work @shravanngoswamii !

What do you think about changing the justify-content: center property in .panel-wrapper to justify-content: space-between? This would align the outer borders of the section headers like "What is Turing.jl? [...]" with those of the cards following below.

The aligned version is more appealing to my eyes, but once again this is of course subjective to some degree.


With justify-content: center:

image

With justify-content: space-between:

image

@shravanngoswamii
Copy link
Member Author

shravanngoswamii commented Jul 12, 2025

Great work @shravanngoswamii !

All thanks to you @simonsteiger!

What do you think about changing the justify-content: center property in .panel-wrapper to justify-content: space-between? This would align the outer borders of the section headers like "What is Turing.jl? [...]" with those of the cards following below.

I did think of that first, but using justify-content: space-between messes up the layout on some other screen sizes, like see:

justify-content: center justify-content: space-between
image image

We can try something else, but I thought having bit space make the content look nested under titles so I kept it!

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.

Redesign the landing page!
5 participants