Repository navigation
feat(annotations): filterable annotation summary with CSV and PDF report (#383) - #769
Closed
oscar981-op wants to merge 1 commit into
Closed
oscar981-op wants to merge 1 commit into
oscar981-op wants to merge 1 commit into
Conversation
…ort (OpenAEC-Foundation#383) - Comments panel: filter bar for author, type, page range and date range (across all pages, alongside the existing status filter), clear-filters action, and bulk status change for the selection or all listed comments in one undo step. - Sorting/grouping logic moved into a pure module shared by list and exports. - CSV export now carries all fields: id, created/modified dates, status set by/at, reply count and reply text, layer and bounds. - "Summarize Comments" builds a PDF report of the listed comments: page number, type, author, date, status, text, replies and a crop of the comment's location with a marker frame. - Translations for all locales; unit tests for filtering, sorting, bulk status, crop geometry, CSV and the PDF report builder. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Merged
Collaborator
|
Thank you, @oscar981-op! This change is included in release 2026.44.1 through the release PR #841, as one squashed commit with you as the author, so this PR is closed rather than merged. |
philip-tam
pushed a commit
to philip-tam/open-pdf-studio
that referenced
this pull request
Nov 1, 2026
…ort (OpenAEC-Foundation#769) The comments panel gets a filter bar for author, type, page range and date range, alongside the existing status filter, plus a clear-filters action and a bulk status change for the selection or all listed comments in one undo step. Sorting and grouping move into a pure module shared by the list and the exports. The CSV export now carries all fields (ids, dates, status history, replies, layer and bounds), and "Summarize Comments" builds a PDF report of the listed comments with a crop of each comment's location. Fixes OpenAEC-Foundation#383 Review: included as is; no changes were needed on top. Note that the CSV column set has changed.
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.
Fixes #383.
Builds on the existing comments panel (grouping and sorting, status filter, click to navigate, XFDF, CSV export).
Added
js/annotations/opmerkingen-overzicht.js. The list, the CSV and the PDF use the same order.js/annotations/opmerkingen-rapport.js.summarizeComingSoonandcsvDate.Tests
New and extended tests:
Unit suite: 5279 passed, 0 failed.
Not yet checked in the running app / known limits
🤖 Generated with Claude Code