Skip to content

Conversation

@Wituareard
Copy link
Collaborator

@netlify
Copy link

netlify bot commented May 21, 2025

Deploy Preview for pauseai ready!

Name Link
🔨 Latest commit 7819a46
🔍 Latest deploy log https://app.netlify.com/projects/pauseai/deploys/6890b7fa2507fa0008580e64
😎 Deploy Preview https://deploy-preview-338--pauseai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Wituareard
Copy link
Collaborator Author

Wituareard commented May 21, 2025

For some reason, this also fixes some <strong> tags in /proposal not working fixed on main

@Wituareard
Copy link
Collaborator Author

@tom-bibby99 What do you think?

@Wituareard
Copy link
Collaborator Author

Wituareard commented May 22, 2025

The home page elements might need to stay bigger, but a heavier font doesn't work with the current size for body text imo and the heavier font definitely makes things more readable.

@joepio
Copy link
Collaborator

joepio commented Aug 2, 2025

I personally don't like the font weight change. With 400 weight everything feels too "bold" with this font. The contrast between body and headers also suffers IMO, which is especially obvious in the footer (see example below).

Screenshot 2025-08-02 at 13 19 36 Screenshot 2025-08-02 at 13 19 48 Screenshot 2025-08-02 at 13 21 39 Screenshot 2025-08-02 at 13 21 28

@Wituareard
Copy link
Collaborator Author

Wituareard commented Aug 2, 2025

I think smaller + heavier is a lot more readable (also closer to common size guidelines), only smaller was too thin in absolute terms iirc. But the contrast to the headings is a good point

@Wituareard Wituareard marked this pull request as ready for review August 2, 2025 22:06
@Wituareard
Copy link
Collaborator Author

oops that was accidental

@Wituareard Wituareard marked this pull request as draft August 7, 2025 14:40
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