Skip to content

feat: improve note viewer with synced insights panel and CI fixes#65

Merged
ddulic merged 3 commits intomainfrom
012-viewer-insights-panel-and-ci-fixes
Mar 18, 2026
Merged

feat: improve note viewer with synced insights panel and CI fixes#65
ddulic merged 3 commits intomainfrom
012-viewer-insights-panel-and-ci-fixes

Conversation

@ddulic
Copy link
Copy Markdown
Owner

@ddulic ddulic commented Mar 18, 2026

Summary

  • Viewer layout: Fixed full-height layout so the left panel scrolls internally (not the whole window), which was breaking the IntersectionObserver used for page tracking. Added scroll-snap (proximity) with consistent top padding per page.
  • Insights panel: Auto-opens when summaries exist; right panel scroll syncs to the active left page with correct gap padding; removed model name badge from AI cards; show all content with scroll-sync instead of filtering to one page.
  • Breadcrumbs: Hidden in viewer mode; folder path shown in FileViewer header instead.
  • CI: Skip test/lint workflows on release-please merge commits to avoid redundant runs after PR merges to main. Squash merging also disabled on the repo.

Test plan

  • Open a note — insights panel should auto-open if summaries exist
  • Scroll through pages — right panel (AI/OCR) scrolls in sync with the active page, with gap padding at top
  • Breadcrumbs and upload/new folder buttons hidden in viewer, visible in file browser
  • FileViewer header shows folder path (e.g. Home / Folder · 5 Pages)
  • Merge a release-please PR — test/lint workflows should not run on the resulting push to main

ddulic and others added 3 commits March 18, 2026 00:15
Viewer layout:
- Fix full-height viewer layout so left panel actually scrolls (was
  scrolling the whole window, breaking IntersectionObserver)
- Hide breadcrumbs/upload buttons in viewer mode; pass folder path into
  FileViewer header as breadcrumb trail
- Add scroll-snap (proximity) with scroll-mt-6 so pages land with
  consistent top padding when scrolling

Insights panel:
- Auto-open panel when summaries exist (v-show + has-insights emit)
- Sync right panel scroll to active left page with 16px top padding so
  only the gap is visible above the incoming card, not previous content
- Remove model name badge from AI segment cards
- Show all segments/OCR pages (scroll-sync) rather than filtering to one

CI:
- Skip test/lint workflows on release-please merge commits to avoid
  redundant runs after PR is merged to main
@ddulic ddulic merged commit 4119513 into main Mar 18, 2026
3 checks passed
@ddulic ddulic deleted the 012-viewer-insights-panel-and-ci-fixes branch March 18, 2026 00:20
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

ddulic added a commit that referenced this pull request Apr 13, 2026
- Revert default port 8000→8080 across all files (Dockerfile, READMEs,
  config-example.yaml, docker-compose.yml, cli, client, server config)
- Restore deleted comments and whitespace in embedding.py
- Raise ValueError on zero-norm query embedding in search.py
- Revert PLAN.md Phase 5 to original historical content
- Move inline UserDO import to top of test_embedding.py
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