imbajin commented on code in PR #418:
URL:
https://github.com/apache/incubator-hugegraph-doc/pull/418#discussion_r2385861367
##########
assets/scss/_variables_project.scss:
##########
@@ -4,3 +4,179 @@ Add styles or override variables from the theme here.
*/
+// Self-hosted font declarations
+// Open Sans font family
+@font-face {
+ font-family: 'Open Sans';
+ font-style: normal;
+ font-weight: 300;
+ font-display: swap;
+ src: url('/assets/fonts/OpenSans-Light.woff2') format('woff2'),
Review Comment:
**Should Fix**: Path consistency issue with font URLs
The font paths use absolute URLs (`/assets/fonts/`) which could break in
subdirectory deployments or development environments. Consider using relative
paths or Hugo's asset pipeline:
```suggestion
src: url('../fonts/OpenSans-Light.woff2') format('woff2'),
url('../fonts/OpenSans-Light.woff') format('woff');
```
This applies to all 38 @font-face declarations. Using relative paths would
ensure the fonts load correctly regardless of the deployment path.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
