Conversation
The `project` frontmatter field is parsed, exposed in the filter dropdown and counted in `AdrStatistics::by_project`, with test coverage for the counting. Nothing ever displayed it, so a collection grouped by project could be filtered but never read. - add a sortable Project column to the list view, rendered as plain text next to Author - add Project to the record detail panel - report `By Project` in the stats summary and in the Markdown output, mirroring the existing category and author breakdowns - give the list view `overflow-x: auto` so the extra column cannot push the table past the viewport
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
projectis already a first-class field everywhere except the screen. It is parsed intoFrontmatter, exposed throughAdr::project(), offered in the viewer's Project filter dropdown, and counted inAdrStatistics::by_projectwith test coverage intest_statistics_by_project.Nothing ever renders it. A collection grouped by project can be filtered by project, but the value itself is invisible in the table, in the detail panel, and in
statsoutput. Selecting a project in the dropdown narrows the list, and the reader still cannot tell which project any row belongs to.This surfaces the facet through the paths that already exist for
categoryandauthor.Changes
Projectcolumn, rendered as plain text next to Author. Plain rather than a badge, so it does not compete with the category chip.Projectrow besideCategory.By Projectin the summary and in the Markdown output, mirroring the category and author breakdowns.overflow-x: autoon.view-container, so the seventh column scrolls instead of pushing the table past a narrow viewport.Notes
projectshow-, exactly as they already do for a missing author.test_summary_with_all_fieldsnow sets a project on both fixtures and assertsBy Project:alongside the existingBy Category:andAuthors:assertions.make checkpasses:cargo fmt --check,cargo clippy -- -D warnings, and the full test suite.