Skip to content

Conversation

rosalieper
Copy link
Contributor

@rosalieper rosalieper commented Aug 1, 2025

refactor the css and update the fonts

Copy link

github-actions bot commented Aug 1, 2025

Deployment previews on netlify for branch refs/pull/1018/merge will be at the following locations (when build is done):

The Segeoe UI on Windows is a very similar to Lato on Linux
Sans-serif is the fallback for all other systems with neither of these fonts
Copy link
Member

@outdooracorn outdooracorn left a comment

Choose a reason for hiding this comment

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

Hey, I've left some comments which I think will make this code simpler by relying on the site-wide defaults more.

It would also be good to update the PR title and description, as this change is no longer about the Lato font. How about "Align font used in the complaint form with the rest of the site" or "Use site-wide font for the complaint form"?

Copy link
Member

@outdooracorn outdooracorn left a comment

Choose a reason for hiding this comment

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

Found some more places to reduce our custom CSS and use Vuetify's defaults! 😁

Did you see my previous comment about the title and description of this PR?

@rosalieper rosalieper changed the title Added fallback font for systems without the Lato font style. Align font used in the complaint form with the rest of the site Aug 12, 2025
Copy link
Member

@outdooracorn outdooracorn left a comment

Choose a reason for hiding this comment

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

Thank you! 😎

@outdooracorn outdooracorn merged commit 6cb7e8b into main Aug 12, 2025
7 checks passed
@outdooracorn outdooracorn deleted the fix-font branch August 12, 2025 13:40
dati18 pushed a commit that referenced this pull request Aug 13, 2025
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