Skip to content

Conversation

@john-sp
Copy link
Member

@john-sp john-sp commented Oct 21, 2025

image image

Any of the authors are welcome to adjust their titles to something more fitting, I just guessed.

Copilot AI review requested due to automatic review settings October 21, 2025 23:07
@github-actions
Copy link

🚀 Draft Preview: https://pr178-draft.lc0.org/

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a new visual design for blog author information, moving from a simple text attribution to a styled author box with avatars and metadata. The changes introduce a centralized author data configuration system and update the blog listing and detail page layouts to display rich author information.

Key Changes:

  • Created a new blog.css file with dedicated styles for blog summaries and author boxes
  • Implemented a YAML-based author data system with avatars, titles, and profile links
  • Updated blog templates to display author information prominently with enhanced styling

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
themes/leela/layouts/_partials/head/css.html Added blog.css to the CSS bundle
themes/leela/assets/css/main.css Removed legacy blog-related CSS (moved to blog.css)
themes/leela/assets/css/blog.css Created new dedicated stylesheet for blog post and author styling
layouts/blog/section.html Updated blog listing to display author names with new layout structure
layouts/blog/page.html Replaced simple author attribution with rich author box component
data/authors.yml Added centralized author configuration with avatars, titles, and URLs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

🚀 Draft Preview: https://pr178-draft.lc0.org/

@github-actions
Copy link

🚀 Draft Preview: https://pr178-draft.lc0.org/

@github-actions
Copy link

🚀 Draft Preview: https://pr178-draft.lc0.org/

@mooskagh
Copy link
Member

mooskagh commented Dec 1, 2025

On the main page: https://pr178-draft.lc0.org/blog/
I'd expect it to be the opposite: show authors for the "guest authors" but hide the authors for the core team.

@github-actions
Copy link

github-actions bot commented Dec 7, 2025

🚀 Draft Preview: https://pr178-draft.lc0.org/

@github-actions
Copy link

github-actions bot commented Dec 7, 2025

🚀 Draft Preview: https://pr178-draft.lc0.org/

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