Repository navigation
feat: add ordered question bookmarks to interview reports - #60
Merged
Snailclimb merged 2 commits intoOct 8, 2026
Merged
Snailclimb merged 2 commits into
Snailclimb merged 2 commits into
Conversation
- 使用完整问题建立书签,并绑定实际排版位置 - 让题号与问题共同分页,避免导航到孤立题号 - 补充 12 项导出回归,完整后端测试 369 项通过、45 项原有禁用
- 使用按数值题号排序的副本,保留各题回答、评分和反馈的关联 - 让正文与书签使用同一顺序,补充乱序输入及跨页目标回归 - 完整后端测试 380 项通过、45 项原有禁用,打包验证通过
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.
Long interview reports have no bookmark navigation, and the loaded answers collection can return questions out of sequence. Render questions in numeric question-index order and add a bookmark for each one, labelled with its number and full question text.
Sorting uses a new list, keeping each question's answer, score, feedback and reference answer together. Bookmarks use layout-generated destinations, so they stay aligned with the actual question positions after pagination. Repeated questions have separate destinations, and headings stay with their question text. Reports without answers do not get an empty outline. The PDF requests the bookmark pane in readers that support it.
Validation: all 23 export regressions passed, covering shuffled and two-digit numbering, record associations, original-list preservation, Unicode and long titles, repeated questions, and cross-page bookmark targets. Java 25
:app:clean :app:test :app:bootJarpassed with 380 tests passing and 45 existing disabled tests.