Skip to content

Conversation

@urmauur
Copy link
Member

@urmauur urmauur commented Sep 3, 2025

Describe Your Changes

This pull request adds global styling to the website by introducing a new custom font and updating the configuration to include the global CSS file. The most notable changes are the addition of multiple font weights for the Studio Feixen Sans font and the application of this font throughout the site.

Styling and font integration:

  • Added a global.css file that defines several @font-face rules for the Studio Feixen Sans font family, covering a range of weights from ultralight to bold. This ensures consistent typography across the site.
  • Set CSS variables for the main and monospace fonts in the :root selector, and applied the main font to the html and body elements for site-wide usage.

Configuration update:

  • Updated the Astro site configuration to include the new global.css file as a custom CSS asset, enabling the new font and styling to take effect.

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

Important

Introduces Studio Feixen Sans font across the site with new global.css and updates astro.config.mjs to apply it globally.

  • Styling and Font Integration:
    • Added global.css with @font-face rules for Studio Feixen Sans covering weights from ultralight to bold.
    • Set CSS variables for main and monospace fonts in :root and applied to html and body.
  • Configuration Update:
    • Updated astro.config.mjs to include global.css for site-wide font application.

This description was created by Ellipsis for 8db764c. You can customize this summary. It will automatically update as commits are pushed.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2025

Preview URL Astro Docs: https://78749efe.astro-docs-14t.pages.dev

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Caution

Changes requested ❌

Reviewed everything up to 8db764c in 1 minute and 46 seconds. Click for details.
  • Reviewed 124 lines of code in 10 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. website/src/styles/global.css:17
  • Draft comment:
    Verify that the unconventional font-weight value (350) for 'Studio Feixen Sans-Book' is intentional.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None
2. website/src/styles/global.css:57
  • Draft comment:
    Typo: The ':root' selector should be written with a single colon. Replace '::root' with ':root'.
  • Reason this comment was not posted:
    Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 0% vs. threshold = 50% The comment appears to be incorrect - looking at the actual code, line 57 already uses ':root' with a single colon, not '::root'. The comment is suggesting a fix for a problem that doesn't exist. This is a new file being added, and the code is already correct. Could I be misreading the line? Should I double-check CSS specifications to verify the correct syntax? I've clearly verified the code shows ':root' with a single colon, and I'm confident about CSS selector syntax. The comment is definitely incorrect about the current state of the code. The comment should be deleted because it's suggesting a fix for a problem that doesn't exist - the code already uses the correct ':root' syntax.

Workflow ID: wflow_az6i5TSEVlw2rCtU

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@urmauur urmauur requested a review from Minh141120 September 3, 2025 05:56
Copy link
Member

@Minh141120 Minh141120 left a comment

Choose a reason for hiding this comment

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

LGTM!

@urmauur urmauur merged commit 08e756f into dev Sep 3, 2025
2 checks passed
@urmauur urmauur deleted the docs/update-web-font branch September 3, 2025 06:22
@github-project-automation github-project-automation bot moved this to QA in Jan Sep 3, 2025
@github-actions github-actions bot added this to the v0.7.0 milestone Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants