Skip to content

Conversation

@eleino
Copy link
Contributor

@eleino eleino commented Jan 15, 2026

πŸ“„ Pull Request Overview

Closes #611

πŸ”§ Changes Made

  1. Added the Marko One font and changed layout.tsx to replace SedgwickAve as title font.

βœ… Checklist Before Submission

  • Functionality: I have tested my code, and it works as expected.
  • JSDoc: I have added or updated JSDoc comments for all relevant code.
  • Debugging: No console.log() or other debugging statements are left.
  • Clean Code: Removed commented-out or unnecessary code.
  • Tests: Added new tests or updated existing ones for the changes made.
  • Documentation: Documentation has been updated (if applicable).

πŸ“ Additional Information

Provide any additional context or information that reviewers may need to know:

  • Screenshots:
kuva kuva kuva
  • Dependencies: [Mention any new dependencies or breaking changes]
  • Known Issues: [List any known issues or limitations]

@eleino eleino self-assigned this Jan 15, 2026
@eleino eleino linked an issue Jan 15, 2026 that may be closed by this pull request
Copy link
Contributor

@Rutjake Rutjake left a comment

Choose a reason for hiding this comment

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

Good work! The font change in Layout looks good, but it seems to be only partially implemented. The main.scss and global CSS variables (like --heading-font) should also be updated to ensure the new Marko One font is correctly inherited by all components, such as the DescriptionCard and ModularCard.

@eleino
Copy link
Contributor Author

eleino commented Jan 15, 2026

Good work! The font change in Layout looks good, but it seems to be only partially implemented. The main.scss and global CSS variables (like --heading-font) should also be updated to ensure the new Marko One font is correctly inherited by all components, such as the DescriptionCard and ModularCard.

Thanks, fixed those now, no more files still using Sedgwick Ave.

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.

Update Font Styles to Match Figma Designs

3 participants