Skip to content

Conversation

UnicornDevCraft
Copy link
Contributor

Hi @benjaoming and @thibaudcolas,

Regarding the landing page:

  1. I didn’t touch the navbar (though I feel a transparent one could give more space).
  2. The content remains inside a container.
  3. I used a 550px breakpoint for font size, let me know if that feels unnecessary.

Please let me know if this works for you or if you’d like further adjustments.

On intro.html from PR #54:
If the illustrations and “cubic waves” are approved, I can draft the intro section. Just a note: keeping it inside the container won’t look quite the same.

On the table from PR #72:
@benjaoming, I saw you had follow-up ideas for small visual improvements - happy to work on those together.

@benjaoming
Copy link
Contributor

Thanks so much @UnicornDevCraft - I will have a look at this in the coming 2-3 days. I'm currently busy with a Friday deadline and a tight weekend, but I will certainly try to get this without obstructions (as always, focusing on over-all improvements and just looking for obvious bugs)

@benjaoming benjaoming marked this pull request as ready for review August 22, 2025 21:03
Copy link
Contributor

@benjaoming benjaoming left a comment

Choose a reason for hiding this comment

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

Tested in different viewports, and this works incredibly well and much better than before! Love that the illustration gets its well-deserved spotlight.

I made it show again as background on the other pages, and I think that also works equally well still.

THANKS @UnicornDevCraft for the amazing craft!

@benjaoming benjaoming merged commit 2228215 into django:main Aug 22, 2025
2 checks passed
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