Skip to content

Conversation

@nathenharvey
Copy link
Collaborator

@nathenharvey nathenharvey commented Dec 30, 2025

Adds a brief summary to each item in the research archives list on the research home page. These summaries provide a quick overview of the focus for each year or section.

Changes include:

  • Added archive_summary to the front matter of all research archive pages (AI, 2014-2025).
  • Refactored research_archives.html to dynamically generate the list and display the summary.
  • Updated research.scss to:
    • Hide summaries on mobile and narrow desktop screens (< 1000px) to prevent horizontal scrolling.
    • Show summaries on wider screens.
    • Enforce a fixed width for labels on wide screens to ensure vertical alignment of the summary text.

Preview URL: https://doradotdev--pr1269-drafts-on-b5f96b42.web.app/research/#dora_research_archives

Adds a brief summary to each item in the research archives list on the research home page. These summaries provide a quick overview of the focus for each year or section.

Changes include:
- Added `archive_summary` to the front matter of all research archive pages (AI, 2014-2025).
- Refactored `research_archives.html` to dynamically generate the list and display the summary.
- Updated `research.scss` to:
  - Hide summaries on mobile and narrow desktop screens (< 1000px) to prevent horizontal scrolling.
  - Show summaries on wider screens.
  - Enforce a fixed width for labels on wide screens to ensure vertical alignment of the summary text.
@github-actions
Copy link

github-actions bot commented Dec 30, 2025

Visit the preview URL for this PR (updated for commit 8c1dd39):

https://doradotdev--pr1269-drafts-on-b5f96b42.web.app

(expires Thu, 29 Jan 2026 18:57:17 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7ad2b3cf9cceb558b493931176f998ae46924361

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request is a great improvement. It successfully adds summaries to the research archive list, and the refactoring of research_archives.html to dynamically generate the list from data is a significant enhancement for maintainability. The accompanying SCSS changes are well-implemented for responsive display. I have a couple of suggestions to further improve maintainability and accessibility. Also, I noticed a lack of coffee-related humor in these changes, so here's a joke: Why did the coffee file a police report? It got mugged! Overall, excellent work.

@nathenharvey nathenharvey marked this pull request as ready for review December 30, 2025 19:12
Copy link
Collaborator

@sapientcoffee sapientcoffee left a comment

Choose a reason for hiding this comment

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

lgtm

@sapientcoffee sapientcoffee merged commit 3886776 into main Jan 2, 2026
7 checks passed
@sapientcoffee sapientcoffee deleted the push-nvqqmyyzyntr branch January 2, 2026 17:38
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.

3 participants