Skip to content

Conversation

@josihoppe
Copy link
Collaborator

  • adjusted the layout of the "what can offgridplanner do" section: aligned numbers with pictures, aligned text with numbers, made text bigger, made pictures smaller
  • adjusted the "who is it for" section, made text bigger

@josihoppe josihoppe marked this pull request as ready for review December 28, 2025 13:25
Copy link
Collaborator

@paulapreuss paulapreuss left a comment

Choose a reason for hiding this comment

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

Looks good! I made the text over the image cards dark blue instead of black to match the rest of the page and fixed some syntax errors that were highlighted in the html. Feel free to merge :)

@josihoppe
Copy link
Collaborator Author

Looks good! I made the text over the image cards dark blue instead of black to match the rest of the page and fixed some syntax errors that were highlighted in the html. Feel free to merge :)

Good catch! :D

You also made these texts smaller, now they have a diffrent size then the "who is it for"-text. Was that intended? I tried to have these both with the same font size, but this way the who is it for seems more present.

@paulapreuss
Copy link
Collaborator

Looks good! I made the text over the image cards dark blue instead of black to match the rest of the page and fixed some syntax errors that were highlighted in the html. Feel free to merge :)

Good catch! :D

You also made these texts smaller, now they have a diffrent size then the "who is it for"-text. Was that intended? I tried to have these both with the same font size, but this way the who is it for seems more present.

Yes, I did do that on purpose. I thought the text was a bit too big compared with the other text in the cards above it, I didn't really catch the text below it 😬 You can reverse the change if you'd like or see about making the other text a bit smaller aswell.

@josihoppe josihoppe merged commit 119226d into main Jan 6, 2026
@josihoppe josihoppe deleted the fix/minor_design_changes branch January 6, 2026 15:33
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.

3 participants