Skip to content

feat: improve file explorer page numbering and insights UI (v1.3.0)#57

Merged
ddulic merged 1 commit intomainfrom
006-ui-file-explorer-improvements
Mar 17, 2026
Merged

feat: improve file explorer page numbering and insights UI (v1.3.0)#57
ddulic merged 1 commit intomainfrom
006-ui-file-explorer-improvements

Conversation

@ddulic
Copy link
Copy Markdown
Owner

@ddulic ddulic commented Mar 17, 2026

Summary

  • Fix page numbering: pages in the file viewer were 0-based (Page 0, Page 1…); now correctly 1-based (Page 1, Page 2…)
  • OCR scroll sync: when the Insights sidebar is open on the OCR tab, it automatically scrolls to match the page currently visible in the main scroll area, using IntersectionObserver
  • AI tab cleanup: the AI tab now shows only AI-generated summaries; OCR text entries (dataSource = "OCR") are filtered out and remain exclusively in the OCR tab

Test plan

  • Open a multi-page note — confirm pages are labelled starting from Page 1
  • Open Insights → OCR tab, scroll through pages — confirm sidebar tracks the visible page
  • Open Insights → AI tab — confirm no OCR text entries appear, only AI summaries

- Fix page numbering to be 1-based (was 0-based) in the file viewer
- Sync OCR sidebar scroll position to the currently visible page using IntersectionObserver
- Filter OCR entries from the AI insights tab so it shows only AI summaries
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@ddulic ddulic merged commit 842704e into main Mar 17, 2026
5 checks passed
@ddulic ddulic deleted the 006-ui-file-explorer-improvements branch March 17, 2026 22:22
@ddulic ddulic restored the 006-ui-file-explorer-improvements branch March 17, 2026 22:43
@ddulic ddulic deleted the 006-ui-file-explorer-improvements branch March 17, 2026 22:43
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.

1 participant